-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Uptime] Added tests for pages #56736
[Uptime] Added tests for pages #56736
Conversation
Pinging @elastic/uptime (Team:uptime) |
@justinkambic this is also ready for review !! |
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.
This looks good - I had a recommendation on how we can reduce the footprint of the router test helpers this introduces.
x-pack/legacy/plugins/uptime/public/pages/__tests__/page_header.test.tsx
Show resolved
Hide resolved
@justinkambic took care of PR feedback |
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 - resolve conflicts and WFG
💛 Build succeeded, but was flaky
History
To update your PR or re-run it, just comment with: |
* master: [Canvas] Move sample data and feature registration to canvas np plugin (elastic#56564) instrument task manager with apm transactions (elastic#55356) displays Alert Instance state on Alert Details page (elastic#56842) Adding the Accessibility Statement to docs (elastic#57153) [Uptime] Remove redundant adapter function (elastic#56980) [SIEM][Detection Engine] Backend end-to-end tests [Uptime] Added tests for pages (elastic#56736) Updating to [email protected] (elastic#57367)
* added test for pages * fixed types * update types * update snap * PR feedback * PR feedback
Summary
Fixes: elastic/uptime#136
Added test for missing page components.