Skip to content

fix(coverage): add TESTFLAGS support and fix grep typo in scripts/coverage.sh #30953

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Victorthedev
Copy link

Enhanced build-test.yml to:

  • Add flaky test detection step with -shuffle=on -count=5 -v.
  • Upload test output as an artifact for debugging.

reference Issue #30892 and PR #30951

What this PR does / why we need it:

Special notes for your reviewer:

If applicable:

  • this PR contains user facing changes (the docs needed label should be applied if so)
  • this PR contains unit tests
  • this PR has been tested for backwards compatibility

…erage.sh, Enhanced build-test.yml to:

- Add flaky test detection step with -shuffle=on -count=5 -v.
- Upload test output as an artifact for debugging.

Signed-off-by: Victor <ubahakweemeka@gmail.com>
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 3, 2025
@Victorthedev
Copy link
Author

@TerryHowe @gjenkins8 Please review

@robertsirc robertsirc added the v4.x Issues and Pull Requests related to the major version v4 label Jun 3, 2025
@gjenkins8
Copy link
Member

For the PR title, I think this would be more correct:

fix(coverage): add scripts/coverage.sh TESTFLAGS support

?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files. v4.x Issues and Pull Requests related to the major version v4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants