-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[ML] Migrate SelectInterval/SelectSeverity unit tests from enzyme
to react-testing-lib
#153321
Conversation
e92a213
to
f140df4
Compare
f140df4
to
50ccf09
Compare
Pinging @elastic/ml-ui (:ml) |
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.
Code LGTM
enzyme
to react-testing-lib
enzyme
to react-testing-lib
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.
LGTM, couple of minor suggestions
...k/plugins/ml/public/application/components/controls/select_interval/select_interval.test.tsx
Outdated
Show resolved
Hide resolved
...k/plugins/ml/public/application/components/controls/select_severity/select_severity.test.tsx
Show resolved
Hide resolved
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @walterra |
Summary
Part of #153288.
Migrates the following React component tests from
enzyme
toreact-testing-lib
.SelectInterval
SelectSeverity
Checklist