-
-
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
test: Appium separate and optimize app launch time measurements #11213
test: Appium separate and optimize app launch time measurements #11213
Conversation
…+ log in to wallet view
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.
Left some comments
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11213 +/- ##
==========================================
+ Coverage 55.24% 55.76% +0.51%
==========================================
Files 1558 1580 +22
Lines 37153 37627 +474
Branches 4406 4501 +95
==========================================
+ Hits 20527 20981 +454
- Misses 16155 16156 +1
- Partials 471 490 +19 ☔ View 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.
LGTM
Quality Gate passedIssues Measures |
Description
This PR refactors the app launch times measurement to capture the time each screen appears, improving granularity. Previously, both cold start tests were in a single feature file, making it difficult to isolate specific test sessions in BrowserStack. Now, the two cold start scenarios are split into separate feature files, allowing for easier identification and tracking of individual test sessions in browserstack.
NOTE: The baseline measures were generated based on ten consecutive runs.
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist