Skip to content

fix(testing): handle undefined options in playwright e2e preset#34741

Open
yjaaidi wants to merge 2 commits intonrwl:masterfrom
yjaaidi:fix/playwright-e2e-preset-options
Open

fix(testing): handle undefined options in playwright e2e preset#34741
yjaaidi wants to merge 2 commits intonrwl:masterfrom
yjaaidi:fix/playwright-e2e-preset-options

Conversation

@yjaaidi
Copy link
Copy Markdown
Contributor

@yjaaidi yjaaidi commented Mar 6, 2026

Calling Playwright's nxE2EPreset(__filename)(without options) fails.

P.S.: I just stumbled upon #34302 & #34301
I guess it's better if we let Playwright handle the default value for openHtmlReport

@yjaaidi yjaaidi requested a review from a team as a code owner March 6, 2026 14:59
@yjaaidi yjaaidi requested a review from JamesHenry March 6, 2026 14:59
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 6, 2026

Deploy Preview for nx-docs canceled.

Name Link
🔨 Latest commit 346775f
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69aaebe07e2239000863e1f7

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 6, 2026

Deploy Preview for nx-dev canceled.

Name Link
🔨 Latest commit 346775f
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/69aaebe01015ff00084f249a

yjaaidi added a commit to marmicode/pragmatic-angular-testing that referenced this pull request Mar 6, 2026
yjaaidi added a commit to marmicode/pragmatic-angular-testing that referenced this pull request Mar 6, 2026
* build: 📦 upgrade swc-angular to avoid nx crash...

...when loading jest with incompatible swc version
during nx migration

* build: 📦 nx migrate latest

* build: 📦 remove migrations.json

Made-with: Cursor

* build: 📦 update vitest to 4.0.18

* ci: 🤖 upgrade github actions

* test: ✅ fix testronaut config

cf. nrwl/nx#34741
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