-
Notifications
You must be signed in to change notification settings - Fork 356
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
feat: e2e add experiment row "kill" action tests #10148
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10148 +/- ##
==========================================
- Coverage 54.35% 51.22% -3.13%
==========================================
Files 1259 1098 -161
Lines 157337 144890 -12447
Branches 3644 3643 -1
==========================================
- Hits 85517 74220 -11297
+ Misses 71687 70537 -1150
Partials 133 133
Flags with carried forward coverage won't be shown. Click here to find out more.
|
✅ Deploy Preview for determined-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…opdown with move modal integration.
…d ExperimentStopModal component
5b016db
to
fd15cac
Compare
- Introduce a time metric for search with a max time of 937 batches - Simplify experiment creation validation logic - Modify kill experiment functionality to check for cancellation status after kill
cb2dc7f
to
c33c77e
Compare
f3e7d95
to
9fa73d0
Compare
9fa73d0
to
7fe0f4f
Compare
just a few small comments. merge when you're ready |
7fe0f4f
to
34e28eb
Compare
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.
looks like e2e is failing
34e28eb
to
3bde7fc
Compare
Closing PR due to company restructuring. |
Ticket
ET-662
Description
This Pr intends to add tests for the "kill" experiment row action for the DataGrid.
Test Plan
'Row Actions'
test from thesrc/e2e/tests/experimentList.spec.ts
test file.Checklist
docs/release-notes/
See Release Note for details.