-
Notifications
You must be signed in to change notification settings - Fork 80
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
test(e2e): update tests to run against tab navigator #5140
Conversation
@@ -9,6 +9,9 @@ describe('Account', () => { | |||
}) | |||
|
|||
describe('Support', Support) |
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.
not doing a describe.each here because not all tests need to run against both navs, see Support.js for tests running against both variants
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5140 +/- ##
==========================================
+ Coverage 85.70% 85.71% +0.01%
==========================================
Files 729 729
Lines 29833 29854 +21
Branches 5154 5160 +6
==========================================
+ Hits 25568 25589 +21
Misses 4030 4030
Partials 235 235 see 8 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
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 and also doesn't look like it added that much time to the e2e tests!
### Description Updates the following tests - AccountManagement.spec.js - updates to run against both variants - Pin.spec.js - updates to run against both variants - Verify.spec.js - updated to run against tab navigator (no need for drawer here since there's sufficient coverage against drawer settings) - WalletConnect.spec.js - still runs agains the drawer (hard to override statsig gate since app launched from deeplink in some cases) but reuses the `navigateToSettings` helper so its easier to update when tab is rolled out ### Test plan CI ### Related issues - Part of ACT-1113 ### Backwards compatibility N/A ### Network scalability N/A
Description
Updates the following tests
navigateToSettings
helper so its easier to update when tab is rolled outTest plan
CI
Related issues
Backwards compatibility
N/A
Network scalability
N/A