Skip to content
This repository has been archived by the owner on Jan 6, 2025. It is now read-only.

Add test using vfork #50

Merged
merged 1 commit into from
Sep 13, 2017
Merged

Conversation

GBuella
Copy link
Contributor

@GBuella GBuella commented Sep 11, 2017

This change is Reviewable

Ref: #3

@codecov-io
Copy link

codecov-io commented Sep 11, 2017

Codecov Report

Merging #50 into master will increase coverage by 0.57%.
The diff coverage is 71.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
+ Coverage   69.71%   70.28%   +0.57%     
==========================================
  Files          19       20       +1     
  Lines        1433     1454      +21     
  Branches      391      397       +6     
==========================================
+ Hits          999     1022      +23     
+ Misses        235      234       -1     
+ Partials      199      198       -1
Flag Coverage Δ
#pmemfile_tests 65.67% <ø> (ø) ⬆️
#regular_tests 57.15% <71.42%> (+0.76%) ⬆️
Impacted Files Coverage Δ
test/prog_using_vfork.c 71.42% <71.42%> (ø)
src/intercept_util.c 54.65% <0%> (+1.05%) ⬆️
src/intercept.c 68.05% <0%> (+1.38%) ⬆️

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 e15d384...1743bd8. Read the comment docs.

@sarahjelinek
Copy link
Contributor

:lgtm:


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


Comments from Reviewable

@GBuella GBuella force-pushed the add_test_with_vfork branch from 1ce001a to 594bbe6 Compare September 12, 2017 08:42
@krzycz
Copy link
Contributor

krzycz commented Sep 12, 2017

:lgtm:


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


Comments from Reviewable

@GBuella GBuella force-pushed the add_test_with_vfork branch from 594bbe6 to d399463 Compare September 12, 2017 18:17
@GBuella GBuella force-pushed the add_test_with_vfork branch from d399463 to 1743bd8 Compare September 13, 2017 09:27
@sarahjelinek sarahjelinek merged commit 6070cf9 into pmem:master Sep 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants