Skip to content

Playwright parallelization#3937

Merged
bznein merged 5 commits intoBitBoxSwiss:masterfrom
bznein:playwright-parallelization
Apr 8, 2026
Merged

Playwright parallelization#3937
bznein merged 5 commits intoBitBoxSwiss:masterfrom
bznein:playwright-parallelization

Conversation

@bznein
Copy link
Copy Markdown
Collaborator

@bznein bznein commented Mar 19, 2026

This PR aims at reducing the total runtime of playwright tests by performing the following:

  • Separate Chromium and Webkit projects into different tests execution, effectively halving the runtime (biggest improvement)
  • Build the frontend once and reuse it in all tests
  • Only install the necessary browser for each test
  • Cache NPM dependencies

There is also a separate commit that fixes some logic for handling retries in regtest tests. It is not strictly needed but it doesn't hurt. Open to moving it to a different PR if anyone fells strongly about it :D

Before asking for reviews, here is a check list of the most common things you might need to consider:

  • updating the Changelog
  • writing unit tests
  • checking if your changes affect other coins or tokens in unintended ways
  • testing on multiple environments (Qt, Android, ...)
  • having an AI review your changes

@bznein bznein requested a review from Beerosagos March 19, 2026 11:26
Copy link
Copy Markdown
Collaborator

@Beerosagos Beerosagos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI and playwright aren't exactly my area of expertise, but the changes look good to me and Codex 😬

@bznein bznein force-pushed the playwright-parallelization branch from b8a6568 to a8e8c66 Compare April 8, 2026 15:26
@bznein bznein merged commit ca1f8de into BitBoxSwiss:master Apr 8, 2026
5 checks passed
@bznein bznein deleted the playwright-parallelization branch April 8, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants