-
Notifications
You must be signed in to change notification settings - Fork 492
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
code coverage instrumentation for integration tests #4960
Comments
Any chance we can get coveralls to use this? |
Not sure. I think that coveralls uses the result from travis, and I don't think that we've got travis setup to run integration tests. |
@oscardssmith coveralls the GitHub integration (not without a way to run integration tests from something that integrates with GitHub), or coveralls the |
@pameyer I see you made pull request #5016 (thanks!) but this issue is still in the "inbox" according to https://waffle.io/IQSS/dataverse and I'm wondering if you're ready for code review. |
@pdurbin It's ready for review, but also didn't seem worth interrupting the flow of what's already going on - and doesn't seem like it's likely to generate merge conflicts. |
@pameyer ok, thanks, I went ahead and dragged it over to code review. I'm curious if the percentage is higher or lower than than the 16% we show currently for unit tests on our README: |
I'm moving this to QA because the directions are clear enough but I didn't try them myself. Note that there's a tip that this technique could be used in production to figure out how much code is actually exercised after a week or a month or whatever of usage. Pretty neat! |
unit tests are currently instrumented to measure code coverage, integration tests are not.
The text was updated successfully, but these errors were encountered: