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
First take of applying assertion funcs to shared tests #6666
Conversation
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
Codecov Report
@@ Coverage Diff @@
## master #6666 +/- ##
=======================================
Coverage 62.09% 62.09%
=======================================
Files 408 408
Lines 32055 32055
=======================================
Hits 19906 19906
Misses 9351 9351
Partials 2798 2798 |
What type of PR is this?
Other/Tests Refactoring
What does this PR do? Why is it needed?
First take of applying testutil/assert and testutil/require to part of shared tests. Please note that some may be missed but I think i was able to cover up to 80% of the statements. Let's merge this in first so we can open another one for the final clean up (in #7003).
Which issues(s) does this PR fix?
Part of #6590