feat: enable shared parallel frontend E2E execution with single Solo/back-end setup - #2577
Merged
steven-sheehy merged 2 commits intoApr 7, 2026
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2577 +/- ##
=======================================
Coverage 99.95% 99.95%
=======================================
Files 183 184 +1
Lines 7410 7446 +36
Branches 1464 1479 +15
=======================================
+ Hits 7407 7443 +36
Misses 3 3
🚀 New features to boost your workflow:
|
yasenltd
force-pushed
the
refactor_parallel_execution
branch
from
April 3, 2026 09:36
69369e5 to
1ac9942
Compare
yasenltd
marked this pull request as ready for review
April 3, 2026 10:25
svienot
previously approved these changes
Apr 7, 2026
svienot
left a comment
Contributor
There was a problem hiding this comment.
This looks good to me.
I have not been able to test locally because of some instabilities which also manifest on main branch for me, so these are not caused by the changes in this PR.
Since tests all pass in CI and locally for @yasenltd I think we are good to go.
yasenltd
force-pushed
the
refactor_parallel_execution
branch
from
April 7, 2026 14:31
7631c12 to
3a29d90
Compare
Member
|
Need to merge from main and change imports to use hiero-ledger/sdk |
…back-end setup Signed-off-by: yasenltd <yasenltd@gmail.com>
yasenltd
force-pushed
the
refactor_parallel_execution
branch
from
April 7, 2026 17:22
e6b38df to
2f80d02
Compare
Signed-off-by: yasenltd <yasenltd@gmail.com>
steven-sheehy
approved these changes
Apr 7, 2026
Kolpic
pushed a commit
that referenced
this pull request
Apr 15, 2026
…back-end setup (#2577) This PR refactors the frontend automation execution model so the heavier Playwright/Electron suites can run as one shared parallel E2E slice in CI, instead of splitting them into multiple infrastructure-heavy jobs. The main goal is to provision Solo and the back end once in CI and reuse that environment across @local-transactions, @organization-basic, and @organization-advanced, while keeping @local-basic as a lightweight standalone slice. To make that safe, the PR adds worker isolation for Electron/Playwright and updates suite hooks so shared-env runs rely on per-worker isolation instead of destructive resets. - Switched CI automation selection from file-based commands to suite tags plus `--grep` - Split automation execution into: - `Automation | Local Basic` for `@local-basic` - `Automation | Shared E2E` for `@local-transactions|@organization-basic|@organization-advanced` - Configured the shared E2E job to start Solo and the back end once and reuse them for all shared suites in that job - Added Playwright/Electron worker isolation using worker-specific: - `userData` directories - session partitions - remote debugging ports - single-instance-lock bypass when needed - Updated front-end runtime code to honor those isolation settings - Updated automation hooks to activate/cleanup shared-env isolation and avoid destructive resets when `PLAYWRIGHT_SHARED_ENV=true` - Added targeted deflake fixes in organization/settings flows: - more deterministic observer selection - seeded organization transaction cache reset - safer negative account-ID test setup - toast assertions by variant instead of generic `.v-toast__text` - more reliable dismissal of the startup “Important Note” modal - Kept a few unstable advanced organization scenarios explicitly skipped instead of letting them fail nondeterministically - Updated `automation/README.md` to document local/CI execution, hooks, and launch/attach limitations - Included small supporting front-end Prisma/test config fixes needed for this branch --------- Signed-off-by: yasenltd <yasenltd@gmail.com>
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
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.
Summary
This PR refactors the frontend automation execution model so the heavier Playwright/Electron suites can run as one shared parallel E2E slice in CI, instead of splitting them into multiple infrastructure-heavy jobs.
The main goal is to provision Solo and the back end once in CI and reuse that environment across
@local-transactions,@organization-basic, and@organization-advanced, while keeping@local-basicas a lightweight standalone slice. To make that safe, the PR adds worker isolation for Electron/Playwright and updates suite hooks so shared-env runs rely on per-worker isolation instead of destructive resets.Why
The previous setup had a few problems:
What Changed
--grepAutomation | Local Basicfor@local-basicAutomation | Shared E2Efor@local-transactions|@organization-basic|@organization-advanceduserDatadirectoriesPLAYWRIGHT_SHARED_ENV=true.v-toast__textautomation/README.mdto document local/CI execution, hooks, and launch/attach limitationsWhat To Review
The most important areas to review are:
.github/workflows/test-frontend.yamlautomation/utils/playwrightIsolation.tsautomation/utils/sharedTestEnvironment.tsautomation/utils/electronAppLauncher.tsThe README and Prisma-related changes are supporting changes, not the main review focus.
Related issue(s):
Fixes #2594