You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Since introducing Cypress E2E testing, we've largely added tests as we introduce new features, and we haven't really taken stock of what our critical flows are. For example, we don't have any E2E tests for the onboarding flow, despite this arguably being one of the most important user flows in the app.
I'm creating this issue more as a 'suggestion', and whoever takes it forward can perhaps lead a discussion within the engineering team of how we want to do things, but I would suggest we create a folder of "smoke tests" - i.e. the minimal set of tests that give us early indication of something critical being broken (e.g. user sign up, sign in, post creation).
This would help give us confidence in refactors, but we could also consider in the future whether we want to only run the smoke tests on each PR build (to speed up build times).
The text was updated successfully, but these errors were encountered:
Thanks for the issue, we will take it into consideration! Our team of engineers is busy working on many types of features, please give us time to get back to you.
Feature requests that require more discussion may be closed. Read more about our feature request process on forem.dev.
To our amazing contributors: issues labeled bug are always up for grabs, but for feature requests, please wait until we add a ready for dev before starting to work on it.
To claim an issue to work on, please leave a comment. If you've claimed the issue and need help, please ping @forem-team. The OSS Community Manager or the engineers on OSS rotation will follow up.
For full info on how to contribute, please check out our contributors guide.
Since introducing Cypress E2E testing, we've largely added tests as we introduce new features, and we haven't really taken stock of what our critical flows are. For example, we don't have any E2E tests for the onboarding flow, despite this arguably being one of the most important user flows in the app.
I'm creating this issue more as a 'suggestion', and whoever takes it forward can perhaps lead a discussion within the engineering team of how we want to do things, but I would suggest we create a folder of "smoke tests" - i.e. the minimal set of tests that give us early indication of something critical being broken (e.g. user sign up, sign in, post creation).
This would help give us confidence in refactors, but we could also consider in the future whether we want to only run the smoke tests on each PR build (to speed up build times).
The text was updated successfully, but these errors were encountered: