Skip to content
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

travis-cargo currently cannot upload / generate code coverage information for documentation tests #36956

Open
gnzlbg opened this issue Oct 4, 2016 · 2 comments
Labels
C-feature-request Category: A feature request, i.e: not implemented / a PR. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@gnzlbg
Copy link
Contributor

gnzlbg commented Oct 4, 2016

@alexcrichton mentioned in this issue that this might be a bug in rustdoc --test. Currently it is not possible to generate code coverage information from documentation tests. This results in either wrong coverage information being generated (when some functionality is tested exclusively in documentation tests). The current workarounds are unfortunately to either duplicate documentation tests into module tests to generate correct code coverage information, or to "accept" wrong code coverage information in e.g. coveralls.

@steveklabnik steveklabnik added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Oct 7, 2016
@steveklabnik steveklabnik added T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue. and removed T-tools labels May 18, 2017
@Mark-Simulacrum Mark-Simulacrum added the C-feature-request Category: A feature request, i.e: not implemented / a PR. label Jul 26, 2017
@steveklabnik
Copy link
Member

Triage: I don't personally use these features; @gnzlbg are you still seeing this?

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jan 10, 2020

I can try, but I don't think anything has happened upstream to support this? cc @QuietMisdreavus I've seen rustdoc can now output the percentage of documented APIs, but that is different from actually being able to run the doc-tests with code-coverage instrumentation, and output the instrumentation results to a meaningful place, or to provide some way that cargo-travis could use to run the doc tests with external code coverage sampling (maybe this would need a custom test runner?).

@ehuss ehuss removed the T-dev-tools Relevant to the dev-tools subteam, which will review and decide on the PR/issue. label Jan 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature-request Category: A feature request, i.e: not implemented / a PR. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

4 participants