-
Notifications
You must be signed in to change notification settings - Fork 359
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.
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
test: add projects tests [CM-467] #9928
Conversation
✅ Deploy Preview for determined-ui canceled.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9928 +/- ##
==========================================
- Coverage 59.18% 54.06% -5.12%
==========================================
Files 751 442 -309
Lines 104462 75881 -28581
Branches 3597 3599 +2
==========================================
- Hits 61824 41027 -20797
+ Misses 42506 34722 -7784
Partials 132 132
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'll give the approve, but please use safeName to avoid conflicts. CI could possibly conflict since the tests run 3 times each. each worker gets it's own user, but if you're ever scoped to all users, not using that util will become a problem
Ticket
CM-467
Description
Adds e2e tests to cover filtering, sorting, and toggling the view on for WorkspaceProjects page.
Adds testing for moving a project to another workspace.
Splits workspaces and projects tests into separate files.
Test Plan
No additional release party testing required. Automated tests should pass.
Checklist
docs/release-notes/
See Release Note for details.