Setup e2e tests [WPB-25890]#9644
Merged
Merged
Conversation
Sadly there can only be one instance of the Wire App running at a time so the benefits of a composable function extinct. Instead a fixture makes the cleanup easier and prevents devs from attempting to launch a second instance.
|
markbrockhoff
approved these changes
Jun 3, 2026
5 tasks
thisisamir98
reviewed
Jun 3, 2026
| @@ -0,0 +1,2 @@ | |||
| WEBAPP_URL=op://Test Automation/BackendConnection staging/webappUrl | |||
| BACKEND_URL=op://Test Automation/BackendConnection staging/backendUrl No newline at end of file | |||
Contributor
Author
There was a problem hiding this comment.
Tagging @markbrockhoff, if he can resolve it in next PRs.
thisisamir98
approved these changes
Jun 3, 2026
zskhan
pushed a commit
that referenced
this pull request
Jun 22, 2026
* install playwright & set up very basic test running the app * resolve eslint issues by introducing a tsconfig for playwright * Create "app" fixture Sadly there can only be one instance of the Wire App running at a time so the benefits of a composable function extinct. Instead a fixture makes the cleanup easier and prevents devs from attempting to launch a second instance. * Always launch the app with locale en * Add script to package.json to execute e2e tests * Use temporary user data dir to isolate app state between runs * Move temp dir creation into fixture to ensure cleanup * Move example spec into specs folder * use env vars for global setup * add readme for e2e test setup * exclude e2e tests from jest config --------- Co-authored-by: Mark Brockhoff <mark.brockhoff@digits.schwarz>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 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.
Pull Request
Summary
Add basic setup for e2e tests for the desktop app, including:
.envsetup + docsThis is the first PR for the new goal of fully automated e2e tests for desktop.
Duplicate for: #9643
Security Checklist (required)
Standards Acknowledgement (required)
Notes for reviewers