enable tracking of test suite coverage using coveralls#3204
enable tracking of test suite coverage using coveralls#3204boegel merged 6 commits intoeasybuilders:developfrom
Conversation
coveralls from marketplace to get report in coveralls
|
@boegel can you please enable coveralls from marketplace https://github.com/marketplace/coveralls |
|
@shahzebsiddiqui I think that We'll need to fix the issue where I guess coverage is picking up the use of that files, and expects it to still be there when collecting the report. Can we make it ignore missing files? |
|
@boegel I cant seem to find where See https://coverage.readthedocs.io/en/coverage-5.0.3/source.html I think something like coverage can even exclude by lines. Most useful when ignoring methods that you know are not to be tested or part of development that you can blacklist for time being. See https://coverage.readthedocs.io/en/coverage-5.0.3/excluding.html I can fix it but dont know the full path to easyblock relative to Travis build directory. |
|
@shahzebsiddiqui The issue is that this file is only created when a specific test is running, and then cleaned up afterwards (it's only stored for a short while in a temporary location). So, the path changes on every run (to ensure a unique temporary location). It may be possible to tweak the test a bit to avoid that coveralls thinks the file should still be there though... |
|
@boegel well i am going to try one more thing clearly from the log it looks like a coverall issue. I found a similar issue reported so i will try to remove |
Testing Travis Build due to similar error reported with where it fails to import Reporter z4r/python-coveralls#73
|
@boegel all Travis builds were complete, only one build failed and it is okay since it was Python 2.6 unable to install |
Only run coverage report and push to coveralls for all builds except for Python 2.6
| # run test suite | ||
| - python -O -m test.framework.suite 2>&1 | tee test_framework_suite.log | ||
| # run coverage on all travis builds except for Python 2.6 | ||
| - if [ "x$TRAVIS_PYTHON_VERSION" != 'x2.6' ]; then coverage run -m test.framework.suite; coverage report -m --ignore-errors; fi |
There was a problem hiding this comment.
@shahzebsiddiqui You should include the 2>&1 | tee test_framework_suite.log part after coverage run -m test.framework.suite, that's important w.r.t. a check below where we make sure there's no unexpected output from the test suite.
There was a problem hiding this comment.
@boegel Looks good to me, everything is green
|
See coverage report @ https://coveralls.io/github/easybuilders/easybuild-framework Thanks a lot for looking into this @shahzebsiddiqui! |
coveralls from marketplace to get report in coveralls