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

Adding E2E testing with Playwright #3349

Merged
merged 13 commits into from
May 15, 2022
Merged

Adding E2E testing with Playwright #3349

merged 13 commits into from
May 15, 2022

Conversation

tony-sull
Copy link
Contributor

@tony-sull tony-sull commented May 11, 2022

Changes

Includes Playwright for E2E testing. The proof of concept migrates over our Tailwind tests to better validate rendered CSS styles, but with a few extra test utils we'll also be able to do proper HMR testing 🎉

Testing

That's all it is!

Docs

None, test coverage only

@changeset-bot
Copy link

changeset-bot bot commented May 11, 2022

⚠️ No Changeset found

Latest commit: ac62f7d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label May 11, 2022
@matthewp
Copy link
Contributor

Can you update the CONTRIBUTING.md and explain how to run these. Also an explanation of how to run an individual test (rather than all of them) would be great.

@github-actions github-actions bot added the 🚨 action Modifies GitHub Actions label May 11, 2022
@tony-sull tony-sull force-pushed the test/playwright-e2e branch 4 times, most recently from 9ee980a to 1cccce8 Compare May 12, 2022 20:58
@tony-sull tony-sull changed the title WIP: adding E2E testing with Playwright Adding E2E testing with Playwright May 12, 2022
@tony-sull tony-sull marked this pull request as ready for review May 12, 2022 22:19
Copy link
Contributor

@matthewp matthewp left a comment

Choose a reason for hiding this comment

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

Ready for this to be merged!

@tony-sull tony-sull marked this pull request as draft May 15, 2022 20:37
@tony-sull tony-sull marked this pull request as ready for review May 15, 2022 20:40
@tony-sull tony-sull merged commit 2b622b5 into main May 15, 2022
@tony-sull tony-sull deleted the test/playwright-e2e branch May 15, 2022 21:47
florian-s-code pushed a commit to florian-s-code/astro that referenced this pull request May 16, 2022
* adding Tailwind E2E tests with Playwright

* package.json updates

* adding e2e tests to CI workflow

* using e2e for dev tests, mocha for build tests

* refactor: sharing test-utils helpers

* chore: update lockfile

* Adding contributing docs

* Revert "refactor: sharing test-utils helpers"

This reverts commit 48496f4.

* refactor: simpler solution to resolving e2e test fixtures

* chore: updating lockfile

* refactor: cleaning up how URLs are resolved in e2e tests

* install playwright deps in CI

* trying pnpm playwright install to fix version mismatch
SiriousHunter pushed a commit to SiriousHunter/astro that referenced this pull request Feb 3, 2023
* adding Tailwind E2E tests with Playwright

* package.json updates

* adding e2e tests to CI workflow

* using e2e for dev tests, mocha for build tests

* refactor: sharing test-utils helpers

* chore: update lockfile

* Adding contributing docs

* Revert "refactor: sharing test-utils helpers"

This reverts commit 48496f4.

* refactor: simpler solution to resolving e2e test fixtures

* chore: updating lockfile

* refactor: cleaning up how URLs are resolved in e2e tests

* install playwright deps in CI

* trying pnpm playwright install to fix version mismatch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 action Modifies GitHub Actions pkg: astro Related to the core `astro` package (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants