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(testing): add @nx/playwright plugin #17975

Merged
merged 4 commits into from
Jul 11, 2023

Conversation

barbados-clemens
Copy link
Contributor

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

Current Behavior

no first party playwright plugin for nx.

Expected Behavior

@nx/playwright plugin

Related Issue(s)

  • feat(testing): add playwright plugin
  • feat(testing): add playwright executor
  • chore(testing): add playwright e2e tests
  • chore(testing): add legacy-playwright so plugin can be installed

Note:
assumed generator setup
playwright.config.ts in the project root with testDir set from the project root, and reporter options set to output in the top level 'dist/playwright/', only tested with html reporter.

I didn't look into what the eslint settings would look like nor tsconfig settings.

Might be able to take inspo from nxkit playwright plugin

@vercel
Copy link

vercel bot commented Jul 5, 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 Jul 10, 2023 9:03pm

@barbados-clemens barbados-clemens force-pushed the feat/playwrite-plugin branch 2 times, most recently from fb185d1 to 51ac6b1 Compare July 5, 2023 21:52
@nx-cloud
Copy link

nx-cloud bot commented Jul 5, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3a3115e. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@barbados-clemens barbados-clemens self-assigned this Jul 6, 2023
@barbados-clemens barbados-clemens added the scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx label Jul 6, 2023
@barbados-clemens barbados-clemens force-pushed the feat/playwrite-plugin branch from c11b9d7 to 829454e Compare July 6, 2023 15:34
@barbados-clemens barbados-clemens marked this pull request as ready for review July 6, 2023 15:35
@mandarini
Copy link
Member

Nice!!!! :D

e2e/playwright/src/playwright.test.ts Show resolved Hide resolved
packages/playwright/project.json Outdated Show resolved Hide resolved
packages-legacy/playwright/.eslintrc.json Outdated Show resolved Hide resolved
packages/nx/package.json Outdated Show resolved Hide resolved
packages/playwright/src/executors/playwright/playwright.ts Outdated Show resolved Hide resolved
@barbados-clemens barbados-clemens merged commit 2128f8e into nrwl:master Jul 11, 2023
@github-actions
Copy link

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 Jul 17, 2023
@barbados-clemens barbados-clemens deleted the feat/playwrite-plugin branch July 17, 2023 18:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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.

6 participants