cargo-benchcmp icon indicating copy to clipboard operation
cargo-benchcmp copied to clipboard

Published 20 hours ago • verified publisher icon BurntSushi

Set up test coverage badge

Open Apanatshka opened this issue 8 years ago • 5 comments

I saw another crate use a badge for test coverage using a service called Coveralls. I have no experience with them, but it looks nice. Might be a good idea when #3 is resolved?

Apanatshka avatar Sep 22 '16 19:09 Apanatshka

Yes please! I think lifting this straight from their .travis.yml will do the trick: https://github.com/phsym/prettytable-rs/blob/master/.travis.yml#L15

BurntSushi avatar Sep 23 '16 10:09 BurntSushi

I just ran kcov locally, because I was curious and the unit (quickcheck) tests give a 93.3% coverage :smile: I also tried to get coverage from the integration tests but that doesn't seem to be working..

Apanatshka avatar Sep 23 '16 19:09 Apanatshka

Yeah, I don't expect the integration tests to contribute to it automatically. I think you'd have to modify the integration test suite to run kcov itself, and then merge the results for every test. It's not worth it.

BurntSushi avatar Sep 26 '16 11:09 BurntSushi

Yeah, I'm not going to look into it. Can you set up the badge? I would do it for you but it's inconvenient to get this working of my fork first then get the values right in a PR.

Apanatshka avatar Sep 26 '16 11:09 Apanatshka

Yup, will do when I get a chance.

BurntSushi avatar Sep 26 '16 11:09 BurntSushi