-
Notifications
You must be signed in to change notification settings - Fork 7
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
Create spike for Cypress E2E testing #2568
Conversation
Yay! It works! It takes quite some time to run cc: @petertgiles |
dde633a
to
23763ac
Compare
Going to back out of the reporting goal, as it's a little fraught. Examples:
both reporters took awhile to debug because the check results are showing up under "Bundlewatch" check for some reason, which is perhaps due to "Bundlewatch" action doing something non-standard. Also, there's some confusing work to make GitHub check reports work for forks: Also, cypress has an issue with mocha reporters can't have their output locations configured right now: tl;dr - too many rabbit holes here! |
@esizer @yonikid15 you seem like you'd get a kick out of e2e tests. Care to review when you get a sec? :) |
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.
Looks great! Just noticed a small change to the footer test descriptions
Added an line comment: Some of the footer checks are skipped until this bug is fixed |
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.
Sweet! 🍭
Resolves #1762
frontend/
enable junit xml reporter for tests and report via github actionswont-do