-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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
chore: E2E tests for the Drill to detail modal #21187
Conversation
Codecov Report
@@ Coverage Diff @@
## master #21187 +/- ##
===========================================
- Coverage 66.40% 54.71% -11.70%
===========================================
Files 1783 1784 +1
Lines 68087 68132 +45
Branches 7261 7261
===========================================
- Hits 45215 37280 -7935
- Misses 21007 28987 +7980
Partials 1865 1865
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Storybook has completed and can be viewed at |
…e/e2e-tests-drilltodetail-modal
Storybook has completed and can be viewed at |
}); | ||
|
||
it('opens the modal from the context menu', () => { | ||
cy.get( |
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.
Can we extract the code that opens the modal to a function? Specifically this part:
cy.get(
"[data-test-viz-type='big_number_total'] [aria-label='More Options']",
).click();
cy.get('.ant-dropdown')
.not('.ant-dropdown-hidden')
.find("[role='menu'] [role='menuitem']")
.eq(5)
.should('contain', 'Drill to detail')
.click();
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.
+1
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.
Sure. Updated
Storybook has completed and can be viewed at |
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
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
…e/e2e-tests-drilltodetail-modal
SUMMARY
This PR adds E2E tests for the drill to detail modal originally implemented in this PR #20728
Related RTL tests: #21148
It also adds a new ECharts Dashboard sample with some sample ECharts (as shown in the screenshot). The goal of this Dashboard is to grow and eventually show as many ECharts samples as possible. I'll continue enhancing this sample dashboard in separate PRs.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N.A.
TESTING INSTRUCTIONS
All the tests should succeed
ADDITIONAL INFORMATION