-
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: User Management, Sort and Filter [CM-469] #10158
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for determined-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10158 +/- ##
==========================================
- Coverage 54.27% 54.26% -0.01%
==========================================
Files 1259 1259
Lines 157285 157291 +6
Branches 3644 3642 -2
==========================================
Hits 85359 85359
- Misses 71793 71799 +6
Partials 133 133
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@@ -150,6 +150,10 @@ class UserRow extends Row { | |||
parent: this, | |||
selector: '[data-testid="user"]', | |||
}); | |||
readonly username = new BaseComponent({ | |||
parent: this, | |||
selector: '[class^="Nameplate_text"] div:last-child', |
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.
webui/react/src/e2e/models/common/hew/Nameplate.ts
await userManagementPage.table.table.headRow[columnId].pwLocator.click(); | ||
}, | ||
), | ||
).toPass(); |
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.
might want to consider a timeout on these. i dont remember the default but it might be infinite
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.
requesting some small changes but very good work. merge when you're ready
Ticket
CM-469
Description
Adds test for Sorting and Filters users on User Management page. Also includes:
Test Plan
No manual testing required. Automated tests should pass.
Checklist
docs/release-notes/
See Release Note for details.