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

t/kvs/dtree: Add non-kvs classic functions mkdir test #1764

Merged
merged 2 commits into from Oct 26, 2018

Conversation

Projects
None yet
3 participants
@chu11
Copy link
Contributor

chu11 commented Oct 25, 2018

Rename --mkdir to --mkdir-classic and remake --mkdir into a test that uses non-deprecated kvs classic functions. This is simply to prepare for when kvs_classic is removed and we won't have to update this test later.

chu11 added some commits Oct 22, 2018

t/kvs/dtree: Add test w/o kvs-classic functions
Rename --mkdir to --mkdir-classic and remake --mkdir into a test
that uses non-deprecated kvs classic functions.
@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Oct 25, 2018

Codecov Report

Merging #1764 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1764   +/-   ##
=======================================
  Coverage   79.58%   79.58%           
=======================================
  Files         185      185           
  Lines       34459    34459           
=======================================
  Hits        27423    27423           
  Misses       7036     7036
Impacted Files Coverage Δ
src/cmd/flux-module.c 85.28% <0%> (-0.31%) ⬇️
src/common/libflux/message.c 81.02% <0%> (-0.25%) ⬇️
src/modules/barrier/barrier.c 78.62% <0%> (+2.06%) ⬆️

@garlick garlick merged commit 94a3595 into flux-framework:master Oct 26, 2018

3 checks passed

codecov/patch Coverage not affected when comparing bd23bac...9c51994
Details
codecov/project 79.58% remains the same compared to bd23bac
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.