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

Add test prefix to all UI Tests #6432

Closed
pixlwave opened this issue Jul 14, 2022 · 0 comments · Fixed by #6433
Closed

Add test prefix to all UI Tests #6432

pixlwave opened this issue Jul 14, 2022 · 0 comments · Fixed by #6433
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks

Comments

@pixlwave
Copy link
Member

pixlwave commented Jul 14, 2022

GitHub actions don't report failed tests if they come from the defaultTestSuite property as compared to explicitly having a test prefix on the name.

The one downside to this is that the compiler won't remind us to add tests when a new mock screen state case is added to the enum. However having knowledge of failing tests is much more valuable and when developing we will be able to run a single test instead of being required to all tests for a specific screen even if only 1 test was changed.

@pixlwave pixlwave added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant