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

Improving test coverage #92

Merged
merged 3 commits into from Jan 4, 2017

Conversation

Projects
None yet
3 participants
@tejasranade
Member

tejasranade commented Dec 23, 2016

Description

This PR improves the unit test coverage by adding new tests to query module.

Changes

query.test.js contains new tests

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Jan 3, 2017

Current coverage is 73.35% (diff: 100%)

Merging #92 into master will increase coverage by 1.76%

@@             master        #92   diff @@
==========================================
  Files            75         75          
  Lines          6636       6636          
  Methods         892        892          
  Messages          0          0          
  Branches       1034       1034          
==========================================
+ Hits           4751       4868   +117   
+ Misses         1885       1768   -117   
  Partials          0          0          

Powered by Codecov. Last update 5fab741...3bb88cc

codecov-io commented Jan 3, 2017

Current coverage is 73.35% (diff: 100%)

Merging #92 into master will increase coverage by 1.76%

@@             master        #92   diff @@
==========================================
  Files            75         75          
  Lines          6636       6636          
  Methods         892        892          
  Messages          0          0          
  Branches       1034       1034          
==========================================
+ Hits           4751       4868   +117   
+ Misses         1885       1768   -117   
  Partials          0          0          

Powered by Codecov. Last update 5fab741...3bb88cc

@thomasconner

LGTM

@thomasconner thomasconner merged commit 8b5382c into master Jan 4, 2017

4 checks passed

codeclimate no new or fixed issues
Details
codecov/patch 100% of diff hit (target 71.59%)
Details
codecov/project 73.35% (+1.76%) compared to 5fab741
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@thomasconner thomasconner deleted the MLIBZ-1559_test_coverage branch Jan 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment