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

Tests for CLI #11

Merged
merged 1 commit into from May 10, 2019

Conversation

Projects
None yet
3 participants
@luizirber
Copy link
Contributor

commented May 10, 2019

For now they only check if it ran without errors, still need to write more checks for the results!

@codecov-io

This comment has been minimized.

Copy link

commented May 10, 2019

Codecov Report

Merging #11 into master will increase coverage by 51.04%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #11       +/-   ##
===========================================
+ Coverage    3.98%   55.03%   +51.04%     
===========================================
  Files           6        6               
  Lines        1154     1152        -2     
  Branches      386      386               
===========================================
+ Hits           46      634      +588     
+ Misses       1087      365      -722     
- Partials       21      153      +132
Impacted Files Coverage Δ
KEGGDecoder/KEGG_decoder.py 65.91% <100%> (+60.82%) ⬆️
KEGGDecoder/Plotly_viz.py 83.33% <100%> (+83.33%) ⬆️
KEGGDecoder/KEGG_clustering.py 83.87% <0%> (+83.87%) ⬆️

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 62eaa11...9b5a830. Read the comment docs.

@luizirber luizirber force-pushed the luizirber:tests_cli branch from f6a388d to 5592076 May 10, 2019

@luizirber

This comment has been minimized.

Copy link
Contributor Author

commented May 10, 2019

Ready for review @bjtully

@luizirber luizirber force-pushed the luizirber:tests_cli branch from 5592076 to deef7ee May 10, 2019

@luizirber luizirber force-pushed the luizirber:tests_cli branch from f726bc0 to 9b5a830 May 10, 2019

@bjtully bjtully merged commit a2293ca into bjtully:master May 10, 2019

1 check passed

Travis CI - Pull Request 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.