We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Checklist (Please check before submitting)
Is your feature request related to a problem? Please describe. Likely related to random number use in unit tests (#228
Failed coverage here: https://github.com/skliper/CF/runs/6581159060?check_suite_focus=true
Overall coverage rate: lines......: 99.9% (2678 of 2681 lines) functions..: 100.0% (258 of 258 functions) branches...: 99.2% (1056 of 1065 branches) 9 branches missed, 8 allowed 3 lines missed, 2 allowed
Note it passed in the very next run w/ no code changes (was just a requirements update)
Describe the solution you'd like Remove whatever random test implementation is causing this (all others eventually to fix #86)
Describe alternatives you've considered None
Additional context #86
Requester Info Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered:
Bad run RANDOM_VALUES_SEED = 1653424171 Good run = 1653424499
Sorry, something went wrong.
Additional missing line/branch:
232 : 6 : CF_ChunkIdx_t CF_Chunks_FindSmallestSize(const CF_ChunkList_t *chunks) 233 : : { 234 : : CF_ChunkIdx_t i; 235 : 6 : CF_ChunkIdx_t smallest = 0; 236 : : 237 [ + + ]: 35 : for (i = 1; i < chunks->count; ++i) 238 : : { 239 [ - + ]: 29 : if (chunks->chunks[i].size < chunks->chunks[smallest].size) 240 : : { 241 : 0 : smallest = i; 242 : : } 243 : : } 244 : : 245 : 6 : return smallest; 246 : : }
Fix nasa#249, Resolve UT variable coverage results
cfb2046
Merge pull request #250 from skliper/fix249-ut_coverage_variability
f713109
Fix #249, Resolve UT variable coverage results
Successfully merging a pull request may close this issue.
Checklist (Please check before submitting)
Is your feature request related to a problem? Please describe.
Likely related to random number use in unit tests (#228
Failed coverage here: https://github.com/skliper/CF/runs/6581159060?check_suite_focus=true
Note it passed in the very next run w/ no code changes (was just a requirements update)
Describe the solution you'd like
Remove whatever random test implementation is causing this (all others eventually to fix #86)
Describe alternatives you've considered
None
Additional context
#86
Requester Info
Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered: