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.

Sign up for GitHub

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

Jump to bottom

Install Playwright for E2E testing #39

Merged
merged 2 commits into from
Jun 15, 2024
Merged

Install Playwright for E2E testing #39

merged 2 commits into from
Jun 15, 2024

Conversation

ann-kilzer
Copy link
Collaborator

@ann-kilzer ann-kilzer commented Jun 15, 2024 •
edited
Loading

Resolves #4

What changed 🧐

  • Install Playwright
  • GH action for main branch
  • Simple test on various devices

How did you test it? 🧪

Run npm run test:e2e and npm run test:e2e:ui

The action must be tested in prod

@ann-kilzer ann-kilzer requested a review from sirbully as a code owner June 15, 2024 07:59
@ann-kilzer ann-kilzer self-assigned this Jun 15, 2024
@ann-kilzer ann-kilzer changed the title 4 playwright Install Playwright for E2E testing Jun 15, 2024
sirbully
sirbully approved these changes Jun 15, 2024
View reviewed changes
Copy link
Contributor

@sirbully sirbully left a comment

Choose a reason for hiding this comment

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

LGTM 🙇‍♀️

@ann-kilzer ann-kilzer merged commit 4034512 into main Jun 15, 2024
6 checks passed
@ann-kilzer ann-kilzer deleted the 4-playwright branch June 15, 2024 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sirbully sirbully sirbully approved these changes

Assignees

@ann-kilzer ann-kilzer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Install Playwright
2 participants
@ann-kilzer @sirbully