Skip to content
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

feat(angular): add playwright to e2eTestRunner option #18163

Merged

Conversation

barbados-clemens
Copy link
Contributor

@barbados-clemens barbados-clemens commented Jul 18, 2023

Note: this shouldn't be in a released until playwright plugin is published.

Blocked By: #18190 and #18202

Current Behavior

angular app generator only support cypress|none.

Expected Behavior

angular app generator supports playwright|cypress|none

Related Issue(s)

Fixes #

@vercel
Copy link

vercel bot commented Jul 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 1, 2023 3:26pm

@barbados-clemens barbados-clemens force-pushed the feat/playwright-angular branch from 5a4fdda to 391a226 Compare July 18, 2023 14:32
@barbados-clemens barbados-clemens self-assigned this Jul 18, 2023
@barbados-clemens barbados-clemens added scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx scope: angular Issues related to Angular support in Nx labels Jul 18, 2023
@@ -74,5 +74,5 @@
},
"lint": {}
},
"implicitDependencies": ["workspace", "cypress", "jest"]
"implicitDependencies": ["workspace", "playwright", "cypress", "jest"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

See if these implicitDependencies are really needed.

@@ -198,9 +198,20 @@ async function addE2ETestRunner(
): Promise<GeneratorCallback> {
switch (options.e2eTestRunner) {
case E2eTestRunner.Cypress:
const { cypressInitGenerator } = ensurePackage<
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this because cypress is installed

@barbados-clemens barbados-clemens merged commit 1dcb80d into nrwl:master Aug 2, 2023
@barbados-clemens barbados-clemens deleted the feat/playwright-angular branch August 2, 2023 21:22
@github-actions
Copy link

github-actions bot commented Aug 8, 2023

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope: angular Issues related to Angular support in Nx scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants