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

0.5 test coverage for utilities #49

Merged

Conversation

eupakhomov
Copy link
Contributor

Added tests for the new utilities.

Eugene Pakhomov added 2 commits April 16, 2018 11:45
Stopwatch updated to provide better interface to tests
Tests for new utilities MoreObjects and Stopwatch created
@coveralls
Copy link

coveralls commented Apr 16, 2018

Coverage Status

Coverage increased (+5.5%) to 55.651% when pulling 943f3d2 on eupakhomov:0.5_test_coverage_for_utilities into 91432d5 on ChargeTimeEU:master.

@TVolden
Copy link
Member

TVolden commented Apr 17, 2018

It looks like the API changes has made some merge conflicts. Can you resolve them?

@codecov-io
Copy link

codecov-io commented Apr 17, 2018

Codecov Report

Merging #49 into master will increase coverage by 5.08%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #49      +/-   ##
============================================
+ Coverage     47.63%   52.71%   +5.08%     
- Complexity      682      710      +28     
============================================
  Files           164      164              
  Lines          2792     2796       +4     
  Branches        201      201              
============================================
+ Hits           1330     1474     +144     
+ Misses         1392     1241     -151     
- Partials         70       81      +11

@eupakhomov
Copy link
Contributor Author

Merged from upstream. Should be ok.

@eupakhomov
Copy link
Contributor Author

I also checked the ones for @sumlin. Seems nothing to do with API - rather formatting issue (something with line endings or spaces).

@sumlin
Copy link
Contributor

sumlin commented Apr 18, 2018

Thank you, @eupakhomov :)

@TVolden TVolden merged commit 68fcba1 into ChargeTimeEU:master Apr 18, 2018
@eupakhomov eupakhomov deleted the 0.5_test_coverage_for_utilities branch April 18, 2018 13:52
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.

5 participants