Skip to content
New issue

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

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

@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

@sarahjelinek sarahjelinek merged commit 6070cf9 into pmem:master Sep 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants