New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(testing): ensure baseUrl is not passed to playwright cli #21943
fix(testing): ensure baseUrl is not passed to playwright cli #21943
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 72b739c. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 6 targets
Sent with 💌 from NxCloud. |
415dcbd
to
72c51ef
Compare
packages/playwright/src/executors/playwright/playwright.impl.ts
Outdated
Show resolved
Hide resolved
packages/playwright/src/migrations/update-18-1-0/remove-baseUrl-from-project-json.ts
Outdated
Show resolved
Hide resolved
72c51ef
to
6385a6e
Compare
6385a6e
to
72b739c
Compare
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
Playwright does not support
--base-url
as a CLI argument.Some of our generators have generated
baseUrl
as an option to the@nx:/playwright:playwright
executor.When executed, this causes an error.
Expected Behavior
Ensure that
baseUrl
does not get added to the CLI args if it exists in the project.jsonAdd a migration to remove
baseUrl
if it already exists in the project.jsonRelated Issue(s)
Fixes #