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
build: enable Cypress tests for visualizations #10206
Conversation
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.
one question, otherwise lgtm
// Number of circles are pretty unstable when there are a lot of circles | ||
// Since main functionality is already covered in fitler test below, | ||
// skip this test untill we find a solution. |
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.
if this test is unstable and we already cover main functionality, should we delete the test? Seems like there's not much point in keeping it around tbh
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.
Part of me still wants to find the root cause of this instability, even though I may never find time to do it, but it doesn't hurt to keep this here as a reminder?
SUMMARY
#10158 Tried to re-enable Cypress tests for visualizations, but missed one critical step: update the CI to include them.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A
TEST PLAN
Make sure the visualization tests show up in CI run logs.
ADDITIONAL INFORMATION