sadpandajoe commented on code in PR #31796:
URL: https://github.com/apache/superset/pull/31796#discussion_r1920662959
##########
superset-frontend/cypress-base/cypress/e2e/chart_list/list.test.ts:
##########
@@ -78,20 +81,15 @@ describe('Charts list', () => {
cy.wait('@get');
});
- it('should show the newly added dashboards in a tooltip', () => {
+ it.only('should show the newly added dashboards in a tooltip', () => {
Review Comment:
@mistercrunch actually on second thought, we should probably add a it.skip
to skip the test entirely with the old code. We should leave a comment above
the test to say we skipped it because mouseover does not trigger the tooltip in
cypress but does do it manually. Currently the code that is removed is the meat
of showing the tooltip, so if we aren't going to validate this, we probably
don't need the tests to run all the steps before the validation.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
