ci: add Jenkinsfile and config for e2e mobile tests #19163
+740
−296
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
Jenkins BuildsClick to see older builds (74)
|
jakubgs
requested changes
Nov 7, 2025
Base automatically changed from
e2e_appium/add-browserstack-tests
to
e2e_appium/add-browserstack
November 7, 2025 13:36
glitchminer
force-pushed
the
e2e_appium/add-browserstack
branch
from
42e0569 to
220c950
Compare
- wallet locators - app lifecycle management - toast message handling (visibility checks and logging). - saved addresses page
…ertions - Replaced instances of LambdaTest reporting with Cloud reporting - Assertion messages for clarity - Updated test methods for toast handling
glitchminer
force-pushed
the
ci/test-e2e-mobile
branch
from
162d5a5 to
a21a910
Compare
jakubgs
requested changes
Nov 27, 2025
Member
jakubgs
left a comment
jakubgs
left a comment
There was a problem hiding this comment.
Much better, but needs some more work. You make a few weird assumptions about availability of env variables.
ci/Jenkinsfile.test-e2e.android
Outdated
| def runId = env.E2E_RUN_ID?.trim() | ||
| def reportsPattern = runId ? "test/e2e_appium/reports/${runId}/**/*.xml" : "test/e2e_appium/reports/**/*.xml" | ||
| junit allowEmptyResults: true, testResults: reportsPattern | ||
| def archivePattern = runId ? "test/e2e_appium/reports/${runId}/**/*" : "test/e2e_appium/reports/**/*" |
Member
There was a problem hiding this comment.
Why is this pattern wider than the one above? How are those artifacts useful?
Contributor
Author
There was a problem hiding this comment.
The junit pattern is for only the final pytest results xml.
The wider pattern captures the reports folder for the run for further analysis when needed, investigating failures etc. It can include screenshots (on fail or when configured), page dumps and logs. It could be possible to use only BrowserStack for this but would have to consider access by wider team if necessary.
glitchminer
force-pushed
the
ci/test-e2e-mobile
branch
from
0bba4c4 to
0511399
Compare
jakubgs
reviewed
Dec 4, 2025
Member
There was a problem hiding this comment.
This is looking pretty good right now, but I just noticed that you didn't actually run it. We don't yet have a job for this.
So you've been just dry-writing this the whole time? Haha. You will have to ask one of my engineers, for example @markoburcul , to help you set up a job for this. And you're also missing a way to trigger it.
Contributor
Author
|
Thanks for the feedback @jakubgs Yeah, just based on the other e2e files to start with. I had briefly mentioned to @siddarthkay a while back that this was being worked on but there were dependencies to resolve in the framework itself and the priority has been the recent releases. There's an issue for setting up a job here if that's any help: https://github.com/status-im/infra-ci/issues/222 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does the PR do
ci/Jenkinsfile.test-e2e.androidso Jenkins can run Android smoke tests on BrowserStack (in preparation for PR checks and nightly runs)Successfully ran with simulated config locally but still to be tested from Jenkins directly