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

e2e: Add tests for common user account flows #13199

Open
Parkreiner opened this issue May 7, 2024 · 2 comments
Open

e2e: Add tests for common user account flows #13199

Parkreiner opened this issue May 7, 2024 · 2 comments
Labels
chore Non-customer facing refactors, cleanup, or technical debt. site Area: frontend dashboard

Comments

@Parkreiner
Copy link
Contributor

Parkreiner commented May 7, 2024

Spinning this off into a milestone issue per @Kira-Pilot's recommendation.
This is spurred by issue #13130

Problem

As we keep using the Go server to seed more and more metadata directly into the HTML file, we have an increased risk of drift between our JS-based unit tests, and anything that uses the Go server, because the JS tests will never have access to that metadata, and won't ever be able to test for it

Proposed solution

We should have our E2E tests go through core user account flows, to make sure we don't kick up any runtime errors in the UI. It should cover things like:

  • Signing out

Notes

  • I have the beginnings of something here, but I'm not sure if there's anything else critical that needs to be added (particularly for the different types of logins we support)
@Parkreiner Parkreiner added site Area: frontend dashboard chore Non-customer facing refactors, cleanup, or technical debt. labels May 7, 2024
@Parkreiner Parkreiner added this to the Code Stability milestone May 7, 2024
@coder-labeler coder-labeler bot added the must-do Issues that must be completed by the end of the Sprint. Or else. label May 7, 2024
@bpmct bpmct removed the must-do Issues that must be completed by the end of the Sprint. Or else. label May 9, 2024
@BrunoQuaresma
Copy link
Collaborator

I'm missing what core user account flows are because it is VERY open.

@Parkreiner
Copy link
Contributor Author

Yeah, sorry – going to refine the requirements for this later on today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Non-customer facing refactors, cleanup, or technical debt. site Area: frontend dashboard
Projects
None yet
Development

No branches or pull requests

3 participants