search

BurntSushi / cargo-benchcmp

A small utility to compare Rust micro-benchmarks.
The Unlicense
342 stars 21 forks source link

Set up test coverage badge #12

Open Apanatshka opened 8 years ago

Apanatshka commented 8 years ago

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?

BurntSushi commented 8 years ago

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

Apanatshka commented 8 years ago

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..

BurntSushi commented 8 years ago

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.

Apanatshka commented 8 years ago

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.

BurntSushi commented 8 years ago

Yup, will do when I get a chance.