Skip to content

Orchestrators tests#71

Merged
thejmazz merged 33 commits intodevfrom
orchestrators_tests
Aug 10, 2017
Merged

Orchestrators tests#71
thejmazz merged 33 commits intodevfrom
orchestrators_tests

Conversation

@thejmazz
Copy link
Member

duplicate of #70

@thejmazz thejmazz requested a review from tiagofilipe12 August 10, 2017 20:01
@codecov-io
Copy link

Codecov Report

Merging #71 into dev will increase coverage by 1.38%.
The diff coverage is 95%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev      #71      +/-   ##
==========================================
+ Coverage   80.49%   81.88%   +1.38%     
==========================================
  Files          36       37       +1     
  Lines         841      861      +20     
  Branches      101      102       +1     
==========================================
+ Hits          677      705      +28     
+ Misses        164      156       -8
Impacted Files Coverage Δ
lib/constants/default-task-state.js 100% <ø> (ø) ⬆️
lib/reducers/collection.js 85.55% <0%> (ø) ⬆️
lib/watermill.js 90.47% <100%> (ø) ⬆️
lib/orchestrators/join.js 96.49% <100%> (+1.03%) ⬆️
lib/task.js 100% <100%> (ø) ⬆️
lib/lifecycle/generate-uid.js 100% <100%> (ø) ⬆️
lib/orchestrators/orchestrator-uid-generator.js 100% <100%> (ø)
lib/orchestrators/fork.js 100% <100%> (+50%) ⬆️
lib/orchestrators/junction.js 89.28% <75%> (+0.39%) ⬆️

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 20d56f3...8cdd6f5. Read the comment docs.

@thejmazz thejmazz merged commit 7359ced into dev Aug 10, 2017
@bmpvieira bmpvieira added the test label Aug 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants