Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Rollup of 15 pull requests #33589

Closed
wants to merge 3 commits into from
Closed

Rollup of 15 pull requests #33589

wants to merge 3 commits into from

Conversation

Ensure that `rerun-if-changed` is printed for all build scripts to ensure that
they've all got the right list of dependencies.
@rust-highfive
Copy link
Contributor

r? @pnkfelix

(rust_highfive has picked a reviewer for you, use r? to override)

@eddyb
Copy link
Member Author

eddyb commented May 12, 2016

@bors r+ p=10

@bors
Copy link
Collaborator

bors commented May 12, 2016

📌 Commit fa11bd5 has been approved by eddyb

@eddyb
Copy link
Member Author

eddyb commented May 12, 2016

@bors force

@bors
Copy link
Collaborator

bors commented May 12, 2016

⌛ Testing commit fa11bd5 with merge a27ed51...

@bors
Copy link
Collaborator

bors commented May 12, 2016

💔 Test failed - auto-win-gnu-64-nopt-t

This commit adds support to rustbuild to run crate unit tests (those defined by
`#[test]`) as well as documentation tests. All tests are powered by `cargo test`
under the hood.

Each step requires the `libtest` library is built for that corresponding stage.
Ideally the `test` crate would be a dev-dependency, but for now it's just easier
to ensure that we sequence everything in the right order.

Currently no filtering is implemented, so there's not actually a method of
testing *only* libstd or *only* libcore, but rather entire swaths of crates are
tested all at once.

A few points of note here are:

* The `coretest` and `collectionstest` crates are just listed as `[[test]]`
  entires for `cargo test` to naturally pick up. This mean that `cargo test -p
  core` actually runs all the tests for libcore.
* Libraries that aren't tested all mention `test = false` in their `Cargo.toml`
* Crates aren't currently allowed to have dev-dependencies due to
  rust-lang/cargo#860, but we can likely alleviate this restriction once
  workspaces are implemented.

cc rust-lang#31590
@eddyb
Copy link
Member Author

eddyb commented May 12, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented May 12, 2016

📌 Commit 6c5b862 has been approved by eddyb

@eddyb
Copy link
Member Author

eddyb commented May 12, 2016

@bors force

@bors
Copy link
Collaborator

bors commented May 12, 2016

⌛ Testing commit 6c5b862 with merge 302c9ac...

@bors
Copy link
Collaborator

bors commented May 12, 2016

💔 Test failed - auto-linux-64-opt-rustbuild

… r=brson

rustbuild: Add support for crate tests + doctests

This commit adds support to rustbuild to run crate unit tests (those defined by
`#[test]`) as well as documentation tests. All tests are powered by `cargo test`
under the hood.

Each step requires the `libtest` library is built for that corresponding stage.
Ideally the `test` crate would be a dev-dependency, but for now it's just easier
to ensure that we sequence everything in the right order.

Currently no filtering is implemented, so there's not actually a method of
testing *only* libstd or *only* libcore, but rather entire swaths of crates are
tested all at once.

A few points of note here are:

* The `coretest` and `collectionstest` crates are just listed as `[[test]]`
  entires for `cargo test` to naturally pick up. This mean that `cargo test -p
  core` actually runs all the tests for libcore.
* Libraries that aren't tested all mention `test = false` in their `Cargo.toml`
* Crates aren't currently allowed to have dev-dependencies due to
  rust-lang/cargo#860, but we can likely alleviate this restriction once
  workspaces are implemented.

cc rust-lang#31590
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants