-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix: Minimize skipping tests #8375
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8375 +/- ##
=======================================
Coverage 40.44% 40.44%
=======================================
Files 1239 1239
Lines 29975 29975
Branches 2875 2875
=======================================
Hits 12122 12122
Misses 17156 17156
Partials 697 697 ☔ View full report in Codecov by Sentry. |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/85b192ee-2060-43e9-a5e4-4fcb71570c6d |
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.
🚀
add spacing for readability Co-authored-by: LeoTM <1881059+leotm@users.noreply.github.com>
add spacing to improve readability Co-authored-by: LeoTM <1881059+leotm@users.noreply.github.com>
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.
should be good to update branch now ^
|
Description
Running tests based on TEST_SUITE only requires skipping a lot of tests which takes significant time for the testrun to complete. Isolating tests suite within the tests folder will only skip tests within that folder significantly reducing tests run times.
This change applies to SmokeConfirmations and SmokeSwaps. SmokeCore tests are located in multiple folders and will still skip tests that are not SmokeCore.
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist