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

Testing translation page #109

Closed
mishmanners opened this issue Jul 19, 2024 · 2 comments · Fixed by #113
Closed

Testing translation page #109

mishmanners opened this issue Jul 19, 2024 · 2 comments · Fixed by #113
Assignees
@mishmanners
Labels
website all things for the website

Comments

@mishmanners
Copy link
Owner

Once we've implemented this feature: #108, we'll need to test to ensure translation is working correctly.

Can test for multiple scenarios.
@mishmanners mishmanners added the website all things for the website label Jul 19, 2024
@mishmanners mishmanners self-assigned this Jul 19, 2024
@mishmanners
Copy link
Owner Author

We can write tests using GitHub Copilot, and then use SIRV to test it in real time.

@mishmanners mishmanners pinned this issue Jul 19, 2024
@mishmanners
Copy link
Owner Author

Started PR here: #112

@maraisr maraisr mentioned this issue Aug 1, 2024
Merged
@mishmanners mishmanners mentioned this issue Aug 9, 2024
Closed
mishmanners added a commit that referenced this issue Aug 9, 2024
@mishmanners
Merge pull request #118 from mr/playwright
bbcd7de 
Playwright is a very popular end-to-end testing suite. We also have a need to get our functionality tested.

This pull request sets the project up with playwright 🎉 , that sets us up for future initiatives — and confidence that we continue to deliver a good customer experience.

What you'll find is:

Brand new tests folder that will house our end-to-end tests
A script added to run them: npm run e2e
Our GitHub actions updated, so as contributions come in GitHub can help us verify things wont regress.
What's next; get more testing coverage, and profit!?

fixes: #109
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mishmanners mishmanners

Labels
website all things for the website
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds some base playwright tests
1 participant
@mishmanners