pmem / pmemstream

Other
9 stars 13 forks source link

tests: add check for region re-allocation order #177

Closed lukaszstolarczuk closed 2 years ago

lukaszstolarczuk commented 2 years ago

This change is Reviewable

codecov-commenter commented 2 years ago

Codecov Report

Merging #177 (1e74f09) into master (972f029) will decrease coverage by 5.00%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
- Coverage   91.66%   86.66%   -5.00%     
==========================================
  Files          13       13              
  Lines         780      735      -45     
==========================================
- Hits          715      637      -78     
- Misses         65       98      +33     
Flag Coverage Δ
tests_clang_debug_cpp17 ?
tests_gcc_debug_cpp17 86.66% <ø> (-0.02%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/libpmemstream_internal.h 88.88% <0.00%> (-11.12%) :arrow_down:
src/mpmc_queue.c 86.95% <0.00%> (-10.23%) :arrow_down:
src/region.c 83.47% <0.00%> (-10.13%) :arrow_down:
src/iterator.c 88.07% <0.00%> (-9.25%) :arrow_down:
src/id_manager.c 82.85% <0.00%> (-6.19%) :arrow_down:
src/span.c 86.95% <0.00%> (-4.35%) :arrow_down:
src/thread_id.c 77.77% <0.00%> (-2.66%) :arrow_down:
src/libpmemstream.c 89.35% <0.00%> (-0.57%) :arrow_down:
src/region_allocator/region_allocator.c 84.90% <0.00%> (-0.17%) :arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more