Skip to content

Optimize test job in CI by caching Playwright browsers - #236

Open
mcmire wants to merge 1 commit into
mainfrom
cache-playwright-browsers
Open

mcmire wants to merge 1 commit into
mainfrom
cache-playwright-browsers

Conversation

@mcmire

@mcmire mcmire commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Each time the test job in CI runs, we end up downloading browsers that the Playwright tests rely upon. This is wasteful and error-prone. Instead, download all browsers up front and cache them for future jobs.

(Most of the code here was copied from the Extension: https://github.com/MetaMask/metamask-extension/blob/73f296fae151c8edf966290381a2324fa3cdd6c3/.github/actions/install-playwright-browsers/action.yml)

@mcmire
mcmire marked this pull request as ready for review November 3, 2026 21:22
@mcmire
mcmire requested a review from a team as a code owner November 3, 2026 21:22
@mcmire
mcmire added this pull request to stack #238 November 3, 2026 23:16
Each time the `test` job in CI runs, we end up downloading browsers that
the Playwright tests rely upon. This is wasteful and error-prone. Cache
the browsers so that we no longer need to do this.
@mcmire
mcmire force-pushed the cache-playwright-browsers branch from 0d91db8 to a94148e Compare November 3, 2026 23:16
@mcmire
mcmire enabled stacked auto-merge (squash) November 3, 2026 23:50

This branch has not been deployed

No deployments
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.

1 participant