Skip to content
This repository has been archived by the owner on Jul 6, 2019. It is now read-only.

Set up test code coverage reports #69

Closed
farcaller opened this issue Jun 15, 2014 · 8 comments
Closed

Set up test code coverage reports #69

farcaller opened this issue Jun 15, 2014 · 8 comments
Assignees
Milestone

Comments

@farcaller
Copy link
Member

It seems that rust binaries can provide coverage data via kcov now that -g is fixed (rust-lang/rust#690). The problem is that kcov doesn't run on either OS X or Linux/arm hosts (the latter is similar to SimonKagstrom/kcov#18), but works perfectly well on Linux/x86_64.

One option is to build coverage data on travis and then upload it to s3. it doesn't deal with historic coverage results though, but will provide overall coverage state for master/HEAD.

@farcaller
Copy link
Member Author

Statically compiled kcov for linux/x86_64: http://vfp.in/kcov.gz

@thehydroimpulse
Copy link

I've always wanted to have this in Rust. Have you looked at https://coveralls.io/ ? It has support for Ruby, JavaScript (super hacky though), Go and a few others.

@thehydroimpulse
Copy link

It can also be triggered upon a new Travis CI build, and keeps the history behind it.

@farcaller
Copy link
Member Author

Thanks for a hint, I've pinged @lemurheavy on adding rust support in coveralls.

@farcaller
Copy link
Member Author

Coverage is being generated now.
PR: #70
Web: http://cov.zinc.rs/platformtree_test

farcaller added a commit to farcaller/zinc that referenced this issue Jun 15, 2014
@farcaller farcaller added this to the 0.2 milestone Oct 8, 2014
@SimonKagstrom
Copy link

kcov should work better with rust since a few commits back (by Kang Seonghoon), and while I very irregularly test ARM support, there was a few fixes to get it working a few months ago (albeit with very limited testing).

And kcov actually supports coveralls.io as well these days, basically just run it with '''--coveralls-id=$TRAVIS_JOB_ID''' from the travis build script.

@farcaller
Copy link
Member Author

Yup, I tested the functionality today and it works just fine for us. Thanks for pointing out coveralls integration, that sounds very useful.

@farcaller
Copy link
Member Author

This is now done.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants