-
-
Notifications
You must be signed in to change notification settings - Fork 144
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
E2E Testing #468
Comments
Working on this one over the weekend @NiallJoeMaher @John-Paul-Larkin Any particular tests you would like to see added? I have the GH auth flow working in playwright so we can test authenticated portions of the application now like Creating a post, commenting on a post etc |
Hey @JohnAllenTech I have not done any testing, so I am eager to learn. I was reluctant to make issues until I have a better understanding myself.
Long term, we should cover as much as possible, but for now the plan is to break this up in to specific issue.
Im really just guessing here, but I can look into it properly later and we can make a proper plan on Mondays livestream? @NiallJoeMaher |
Yes @JohnAllenTech. We can make a heap of issues from this so we can get contributors helping out too.
|
Context
Right now, we have no e2e testing but Playwright setup.
We should have end-to-end testing enabled to make sure people are more comfortable changing things without fear of breaking things.
The first thing would be some input on the appropriate testing.
Then, once we have decided on some appropriate areas, we can split the testing into issues to share the workload.
Expected Behavior
Test all major functionality for the application in e2e tests.
Current Behavior
No testing 😢
The text was updated successfully, but these errors were encountered: