marcinslusarz commented 7 years ago

Should help with code coverage fluctuations with each build.

This change is

krzycz commented 7 years ago

Reviewed 3 of 3 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

codecov-io commented 7 years ago

Codecov Report

Merging #329 into master will increase coverage by 0.1%. The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #329     +/-   ##
=========================================
+ Coverage   78.18%   78.29%   +0.1%     
=========================================
  Files          90       90             
  Lines       13827    13836      +9     
  Branches     1996     1996             
=========================================
+ Hits        10811    10833     +22     
+ Misses       2294     2283     -11     
+ Partials      722      720      -2

Impacted Files	Coverage Δ
tests/posix/dirs/dirs.cpp	`92.8% <100%> (+0.02%)`	:arrow_up:
tests/posix/mt/mt.cpp	`87.11% <100%> (+0.65%)`	:arrow_up:
tests/posix/basic/basic.cpp	`94.39% <100%> (+0.09%)`	:arrow_up:
src/libpmemfile-posix/unlink.c	`92.4% <0%> (-2.54%)`	:arrow_down:
src/libpmemfile-posix/utils.c	`71.87% <0%> (-1.57%)`	:arrow_down:
src/libpmemfile/path_resolve.c	`88.88% <0%> (+1.23%)`	:arrow_up:
src/libpmemfile-posix/dir.c	`85.25% <0%> (+1.47%)`	:arrow_up:
src/libpmemfile-posix/file.c	`68.95% <0%> (+1.92%)`	:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 01eeaa0...a079ea0. Read the comment docs.

pmem / pmemfile

Small code coverage improvements. #329

Codecov Report