-
Notifications
You must be signed in to change notification settings - Fork 108
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
Include all tests in coverage #92
Conversation
1 similar comment
Also include dead code in the coverage information.
2 similar comments
Cargo only creates temporary executables for doc tests, so they can't be included. I haven't added benches/examples yet. They shouldn't be hard to add, but they're not actually tests, so not sure they should be included. I can't browse the files on coveralls.io anymore with this change, but I can't see what might have caused that. We're currently doing coverage for stable, beta and nightly. Is that useful or should we change it to stable only? |
In order to make them browse-able initially, I had to set "src" as the root path somewhere in the coveralls settings. I guess this broke that because now there are also files whose path starts with "tests". |
Thanks for tackling this @philipc :) |
Ok should be fixed now, eg: https://coveralls.io/builds/7752904/source?filename=src%2Fline.rs |
No description provided.