Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Expand test coverage for Feature List dialog #581

Closed
justin-bits opened this issue May 31, 2019 · 1 comment · Fixed by #777
Closed

Expand test coverage for Feature List dialog #581

justin-bits opened this issue May 31, 2019 · 1 comment · Fixed by #777
Labels
cypress tests

Comments

@justin-bits
Copy link
Collaborator

justin-bits commented May 31, 2019 •
edited
Loading

#567

Expand coverage to fill in the gaps for Layer Features testing. This also appears to be the first table version of slick-grid and will serve as a good full test.

Dialog UI:

  • Show Selected Only checkbox
  • Row height slider

Context Menu:

  • Sort Selected
  • Export
  • Go To

Mouse/Keyboard Interactions:

  • Ctrl+click selections
  • Shift+click selections
  • Sort via clicking on header
  • Moving columns via drag

Additional:

  • Multiple instances of the dialog with different layers
  • Smoke tests for additional types of data (e.g. csv)
@justin-bits justin-bits mentioned this issue May 31, 2019
Merged
bradh pushed a commit to bradh/opensphere that referenced this issue Jun 6, 2019
@smithtb
Merge pull request ngageoint#581 in WV/opensphere from ~SMITHT/opensp…
12c0939 
…here:THIN-12526 to master

* commit '0e9cc0f3585abf86d3c6d69f69c8ae7c94d87692':
  feat(watchboxForm): Notification strategy choice should be clearer - back to pre-bs4 nav-pills
@justin-bits justin-bits changed the title Expand test coverage for Layer Features dialog Expand test coverage for Feature List dialog Jul 10, 2019
@justin-bits
Copy link
Collaborator Author

justin-bits commented Jul 11, 2019 •
edited
Loading

Ctrl / shift selections, sort selected, export, are done here: #677

@justin-bits justin-bits mentioned this issue Jul 17, 2019
Merged
@justin-bits justin-bits mentioned this issue Oct 30, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cypress tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test(cypress): adds tests for toolbar, sorting, show selected only justin-bits/opensphere
2 participants
@wallw-teal @justin-bits