test: random_matching_test.cpp modernized, made self-contained and assertive - #555
test: random_matching_test.cpp modernized, made self-contained and assertive#555Becheler wants to merge 1 commit into
random_matching_test.cpp modernized, made self-contained and assertive#555Conversation
|
Boost dependency footprint vs Header-inclusion weights (graph files pulling each direct dependency in): No header-inclusion-weight changes. Transitive Boost modules: 68 → 68 (0) |
random_matching_test.cpp modernized, made self-contained and assertive
|
Compiler-warning counts vs
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #555 +/- ##
===========================================
+ Coverage 88.22% 92.82% +4.60%
===========================================
Files 392 392
Lines 27412 28166 +754
Branches 7997 7994 -3
===========================================
+ Hits 24183 26146 +1963
- Misses 1881 1914 +33
+ Partials 1348 106 -1242
... and 288 files with indirect coverage changes Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
Code coverage is very unstable, I am fixing unit tests, and many are not asserting much and only printing to cout, which is never checked: failed test still show as PASS.
Before submitting
developbranch.Type of change
Does this PR introduce a breaking change?
What this PR does
n/mas constexpr, droppedargc/argv+ usage message; removed args from Jamfile.Motivation
Our coverage is not great, code coverage flickers in CI
Testing
Checklist
b2in thetest/directory).