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

Make root:Packages:igortest:FailedProcWave less verbose #441

Open
Garados007 opened this issue Mar 20, 2023 · 0 comments
Open

Make root:Packages:igortest:FailedProcWave less verbose #441

Garados007 opened this issue Mar 20, 2023 · 0 comments

Comments

@Garados007
Copy link
Member

This wave is used to collect a short message from each failed assertion during the test run and output them at the end as a small summary. As we do only add new information to the wave there are some circumstances now that the wave contains no longer needed information:

  • After the introduction of re-running flaky tests the wave contains the output of previous failed runs. I think we don't need them in the summary and therefore these can be removed.
  • The test experiment TestResultsTests has many intentionally failed tests. There is no need for these errors in the summary.

This can be fixed by storing the number of entries in FailedProcWave at the start of each test case and add a internal function that resets the wave to these number of entries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant