Skip to content

Commit 73e4172

Browse files
committed
fix: postinstall trying to install playwright
1 parent 673b023 commit 73e4172

3 files changed

Lines changed: 8 additions & 3 deletions

File tree

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,6 @@ jobs:
3737
fail-fast: false
3838
with:
3939
os: ${{ matrix.os }}
40-
command: yarn test:nuts
40+
command: npx playwright install --with-deps && yarn test:nuts
4141
retries: 5
4242
secrets: inherit

README.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,13 @@ If a new org is required for NUTs tests, these are the steps to create and confi
149149

150150
## Running NUTs (integration tests) locally
151151

152-
NUTs (e2e integration tests) run the plugin against a real org and, for component-preview tests, a real browser (Playwright). To run them locally:
152+
NUTs (e2e integration tests) run the plugin against a real org and, for component-preview tests, a real browser (Playwright). Playwright browsers are installed automatically on CI but must be installed manually for local runs:
153+
154+
```bash
155+
npx playwright install --with-deps
156+
```
157+
158+
To run NUTs locally:
153159

154160
1. **Environment variables**
155161
Copy `.env.template` to `.env` and set the values for your Dev Hub org and test setup:

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,6 @@
102102
"link-check": "wireit",
103103
"link-lwr": "yarn link @lwrjs/api @lwrjs/app-service @lwrjs/asset-registry @lwrjs/asset-transformer @lwrjs/auth-middleware @lwrjs/base-view-provider @lwrjs/base-view-transformer @lwrjs/client-modules @lwrjs/config @lwrjs/core @lwrjs/dev-proxy-server @lwrjs/diagnostics @lwrjs/esbuild @lwrjs/everywhere @lwrjs/fs-asset-provider @lwrjs/fs-watch @lwrjs/html-view-provider @lwrjs/instrumentation @lwrjs/label-module-provider @lwrjs/lambda @lwrjs/legacy-npm-module-provider @lwrjs/loader @lwrjs/lwc-module-provider @lwrjs/lwc-ssr @lwrjs/markdown-view-provider @lwrjs/module-bundler @lwrjs/module-registry @lwrjs/npm-module-provider @lwrjs/nunjucks-view-provider @lwrjs/o11y @lwrjs/resource-registry @lwrjs/router @lwrjs/security @lwrjs/server @lwrjs/shared-utils @lwrjs/static @lwrjs/tools @lwrjs/types @lwrjs/view-registry lwr",
104104
"lint": "wireit",
105-
"postinstall": "npx playwright install --with-deps",
106105
"postpack": "sf-clean --ignore-signing-artifacts",
107106
"prepack": "sf-prepack",
108107
"prepare": "sf-install",

0 commit comments

Comments
 (0)