Fix #958, Convert some UT_GetStubCount to UtAssert_STUB_COUNT #1377
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
Describe the contribution
UT_GetStubCount()
which were just being checked viaUtAssert_True()
have been converted to useUtAssert_STUB_COUNT()
, which automatically writes the expected and actual stub count to the test log.Testing performed
GitHub CI actions all passing successfully.
Local testing with standard cFS suite shows all tests passing and no change in coverage.
Expected behavior changes
No change to code.
Tests also essentially identical, just a bit simpler, clearer and more useful for debugging.
System(s) tested on
Intel(R) Celeron(R) N4100 CPU @ 1.10GHz x86_64
Debian GNU/Linux 11 (bullseye)
Current main branch of cFS.
Contributor Info
Avi Weiss @thnkslprpt