test: 💍 add unit testing && coverage check using codecov#7
Conversation
|
Modify the & Do & Finally squash all commits into one 😄 |
|
@unsuitable001 Thank you for the review TODO
notes
|
|
Thanks a lot for your valuable contribution @theashraf . I'm glad to have you here. Please put the coverage report under And, the environment variables are not available for pull request See : https://docs.travis-ci.com/user/pull-requests/#pull-requests-and-security-restrictions So, check that too. There's a env variable in Travis But, I'll be glad if you come up with a solution for that PR thing. Because, it's important to check a coverage of a PR. I'm also finding a solution. Everything else is great! & You are right about that |
TODO
Notes
|
|
@theashraf It's all set actually. But, the issue is, Can you suggest a way to report coverage for PRs as well? |
|
I think codecov is a nice tool to integrate, check it out |
|
You mean, moving from codebeat to codecov? |
|
yes, it would be easier for me to integrate since I've used it before |
|
Ok. I'm setting it up ASAP. You can start working on it in this PR. |
|
Done @theashraf from my side! |
Codecov Report
@@ Coverage Diff @@
## master #7 +/- ##
=========================================
Coverage ? 81.81%
=========================================
Files ? 1
Lines ? 44
Branches ? 9
=========================================
Hits ? 36
Misses ? 7
Partials ? 1
Continue to review full report at Codecov.
|
|
Well done!!!! I'll merge it soon. |
|
Great, I think having a simple example like the one that you already created under the |
|
@theashraf I am also thinking the same. Great work. |


test:coveragenpm scriptcodeclimate-test-reporterpackagevalidatenpm script , onpre-commithook usinghuskypackageCloses: [unit testing] Add coverage check using codecov #4