-
Notifications
You must be signed in to change notification settings - Fork 919
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
Introduce code coverage collection to Cypress runs #9084
base: main
Are you sure you want to change the base?
Conversation
Also: * Add coverage reporting to Cypress CI
…running against Also: * Turn on test isolation when security is disabled Signed-off-by: Miki <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9084 +/- ##
==========================================
- Coverage 60.93% 51.87% -9.07%
==========================================
Files 3809 3525 -284
Lines 91270 87361 -3909
Branches 14414 13796 -618
==========================================
- Hits 55618 45321 -10297
- Misses 32097 38993 +6896
+ Partials 3555 3047 -508
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Miki <[email protected]>
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, although looks like some ci are failing
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.
would this fix #975?
I think https://github.com/AMoo-Miki/Compatibility-Tester and https://tests.aws.barahmand.com/ deal with that better. |
Description
Changelog
Check List
yarn test:jest
yarn test:jest_integration