Skip to content

Test rustdoc. - #44274

Merged
bors merged 1 commit into
rust-lang:masterfrom
Mark-Simulacrum:rustdoc-tests
Sep 10, 2017
Merged

Test rustdoc.#44274
bors merged 1 commit into
rust-lang:masterfrom
Mark-Simulacrum:rustdoc-tests

Conversation

@Mark-Simulacrum

@Mark-Simulacrum Mark-Simulacrum commented Sep 2, 2017

Copy link
Copy Markdown
Member

Also fixes the broken tests.

r? @alexcrichton

@Mark-Simulacrum

Mark-Simulacrum commented Sep 2, 2017

Copy link
Copy Markdown
Member Author

I suspect that this will fail until #44259 is backported to beta.

@alexcrichton

alexcrichton commented Sep 2, 2017

Copy link
Copy Markdown
Member

Hm so this is running tests for all of rustdoc's dependencies? Don't we want to only run tests for librustdoc?

@Mark-Simulacrum

Mark-Simulacrum commented Sep 2, 2017

Copy link
Copy Markdown
Member Author

Yeah, that's true. I've reworked it to only run for rustdoc now.

@alexcrichton

alexcrichton commented Sep 3, 2017

Copy link
Copy Markdown
Member

To land this now, can we disable doctests for rustdoc?

@Mark-Simulacrum

Mark-Simulacrum commented Sep 5, 2017

Copy link
Copy Markdown
Member Author

Sorry, I missed your comment! Yes, we can disable tests for now. I've passed --lib to cargo test for rustdoc, which should solve the problem here I believe. cc @rust-lang/dev-tools: Rustdoc's doc tests will not be enabled until next release cycle most likely, but unit tests will work after this lands.

@Mark-Simulacrum Mark-Simulacrum added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 5, 2017
@oli-obk

oli-obk commented Sep 5, 2017

Copy link
Copy Markdown
Contributor

#43628 adds support for disabling tests or building of tools conveniently via src/tools/toolstate.toml. We can trivially insert rustdoc into that file once the PR is merged

@Mark-Simulacrum

Mark-Simulacrum commented Sep 5, 2017

Copy link
Copy Markdown
Member Author

I don't think we want to disable tests or rustdoc; this PR disable the doc tests and as such should be merge able.

alexcrichton
alexcrichton reviewed Sep 5, 2017
View reviewed changes
Comment thread src/bootstrap/check.rs Outdated

@alexcrichton alexcrichton Sep 5, 2017

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh instead of passing --lib, could this instead specify:

[lib]
doctest = false

in the rustdoc Cargo.toml?

(that way this could just be cargo test)

alexcrichton
alexcrichton reviewed Sep 5, 2017
View reviewed changes
Comment thread src/bootstrap/check.rs Outdated

@alexcrichton alexcrichton Sep 5, 2017

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can probably be omitted, we'll never test rustdoc for these architectures

@Mark-Simulacrum
Test rustdoc unit tests.
f87696b 
Doc tests are temporarily disabled until next release cycle, since
current beta Cargo errors on them. Upgrade should be smooth as the
relevant tests are already fixed in this commit.
@Mark-Simulacrum

Mark-Simulacrum commented Sep 5, 2017

Copy link
Copy Markdown
Member Author

Fixed.

@alexcrichton

alexcrichton commented Sep 5, 2017

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Sep 5, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit f87696b has been approved by alexcrichton

@arielb1 arielb1 added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 5, 2017
@frewsxcv frewsxcv mentioned this pull request Sep 9, 2017
Closed
@bors

bors commented Sep 10, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f87696b with merge f982ff0...

bors added a commit that referenced this pull request Sep 10, 2017
@bors
Auto merge of #44274 - Mark-Simulacrum:rustdoc-tests, r=alexcrichton
f982ff0 
Test rustdoc.

Also fixes the broken tests.

r? @alexcrichton
@bors

bors commented Sep 10, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing f982ff0 to master...

@bors
bors merged commit f87696b into rust-lang:master Sep 10, 2017
@bors bors mentioned this pull request Sep 10, 2017
Merged
@QuietMisdreavus QuietMisdreavus mentioned this pull request Mar 16, 2018
Merged
bors added a commit that referenced this pull request Mar 17, 2018
@bors
Auto merge of #49090 - QuietMisdreavus:test-rustdoc-again, r=Mark-Sim…
8cabda4 
…ulacrum

re-enable testing librustdoc

This was originally put in in #44274, but #48105 accidentally hid it. This change puts librustdoc unit/doc tests back in the main test listing.

fixes #44237 (again)
@Mark-Simulacrum
Mark-Simulacrum deleted the rustdoc-tests branch June 8, 2019 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexcrichton alexcrichton alexcrichton left review comments

Assignees

@alexcrichton alexcrichton

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Mark-Simulacrum @alexcrichton @oli-obk @bors @arielb1