smarty icon indicating copy to clipboard operation
smarty copied to clipboard

[WIP] Add coverage to travis tests

Open bannanc opened this issue 7 years ago • 6 comments

I think this is all we have to do to generate coverage reports for tests, if it works here I can do it on openforcefield also.

bannanc avatar Feb 08 '18 04:02 bannanc

The README.md shows the code coverage badge, but the status is unknown. Is there a way to tell whether it's working before merging?

(Often I've seen code coverage tests/reports as part of a PR, but turning that on -- likely a toggle on GitHub's settings -- perhaps is a separate test. And maybe the badge only reports on coverage of master?)

Your branch is also out-of-date.

davidlmobley avatar Feb 08 '18 04:02 davidlmobley

Yes, I meant to put a [WIP] on this when I realized there was something up with the tests. The only coverage I've done was with pytest and directly to a master branch so I'm not sure if the report will show up before this is merged into the master.

bannanc avatar Feb 08 '18 18:02 bannanc

For the record this is trying to address #262 and #264

bannanc avatar Apr 26 '18 01:04 bannanc

I've been revisiting my github PRs and Issues and found this. I think it would be good for our (mine and @camizanette 's) portfolios for this to not be failing. I think the current issues here relate to the openeye license being expired and maybe a version problem with networkx. I'm going to try to get things passing and then maybe pin versions so we know it is reproducible within the environment it was developed in.

bannanc avatar Jan 24 '19 02:01 bannanc

Note on that, going to doesn't mean immediately, just means at some point.

bannanc avatar Jan 24 '19 02:01 bannanc

All for pinning versions; having the tests frequently start failing because of a change in somethign we don't control is frustrating.

davidlmobley avatar Jan 24 '19 04:01 davidlmobley