Skip to content

Move rustdoc run-make tests into new tests/run-make/rustdoc/#157860

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
fmease:mkdir-tests-run-make-rustdoc
Jul 18, 2026
Merged

Move rustdoc run-make tests into new tests/run-make/rustdoc/#157860
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
fmease:mkdir-tests-run-make-rustdoc

Conversation

@fmease

@fmease fmease commented Jun 13, 2026

Copy link
Copy Markdown
Member

Furthermore, populate tests/run-make/rustdoc/doctest/ with doctest run-make tests and tests/run-make/rustdoc/scrape-example/ with scrape-example run-make tests.

For context we currently have 11 doctest run-make tests + 8 scrape-example run-make tests + 24 other rustdoc run-make tests = 43 rustdoc run-make tests in total.

Motivation: Sorta obvious but

  1. I want to be able to just call e.g., ./x t tests/run-make/rustdoc/doctest tests/rustdoc-ui/doctest when working on doctest internals to quickly see if anything broke instead of having to execute all run-make tests "every time I make a small change"
  2. having them in one official place makes it a lot easier to spot which areas might need more coverage, to take inspiration from existing rustdoc run-make tests, to find existing test files that one can extend instead of creating a whole new run-make test, etc.

r? rustdoc

@rustbot

rustbot commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

rustc-dev-guide is developed in its own repository. If possible, consider making this change to rust-lang/rustc-dev-guide instead.

cc @BoxyUwU, @tshepang

@rustbot rustbot added A-run-make Area: port run-make Makefiles to rmake.rs A-rustc-dev-guide Area: rustc-dev-guide S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 13, 2026
@rust-bors

This comment has been minimized.

@notriddle

Copy link
Copy Markdown
Contributor

this looks fine, but needs rebased

Furthermore, populate `tests/run-make/rustdoc/doctest/` with doctest
run-make tests and `tests/run-make/rustdoc/scrape-example/` with scrape
example run-make tests.
@fmease
fmease force-pushed the mkdir-tests-run-make-rustdoc branch from ecf2b75 to 7cc650f Compare July 18, 2026 13:21
@rustbot

rustbot commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@fmease

fmease commented Jul 18, 2026

Copy link
Copy Markdown
Member Author

@bors r=notriddle rollup

@rust-bors

rust-bors Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 7cc650f has been approved by notriddle

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 Jul 18, 2026
rust-bors Bot pushed a commit that referenced this pull request Jul 18, 2026
…uwer

Rollup of 8 pull requests

Successful merges:

 - #159425 (Windows: add context when opening NUL for child stdio fails)
 - #159467 (Add explicit `Iterator::count` impl for `str::EncodeUtf16`)
 - #157860 (Move rustdoc run-make tests into new `tests/run-make/rustdoc/`)
 - #158545 (Move `std::io::read_to_string` to `alloc::io`)
 - #159328 (tests/assembly-llvm: pin frame pointer in issue-141649 aarch64 test)
 - #159459 (Detect when trait bound requires closure to return itself)
 - #159470 (Fix string indexing in diagnostic format strings)
 - #159500 (Move compiletest CLI parsing to `cli.rs`)
@rust-bors
rust-bors Bot merged commit 0bf6c18 into rust-lang:main Jul 18, 2026
13 checks passed
@rustbot rustbot added this to the 1.99.0 milestone Jul 18, 2026
rust-timer added a commit that referenced this pull request Jul 18, 2026
Rollup merge of #157860 - fmease:mkdir-tests-run-make-rustdoc, r=notriddle

Move rustdoc run-make tests into new `tests/run-make/rustdoc/`

Furthermore, populate `tests/run-make/rustdoc/doctest/` with doctest run-make tests and `tests/run-make/rustdoc/scrape-example/` with scrape-example run-make tests.

For context we currently have 11 doctest run-make tests + 8 scrape-example run-make tests + 24 other rustdoc run-make tests = 43 rustdoc run-make tests in total.

**Motivation**: Sorta obvious but

1. I want to be able to just call e.g., `./x t tests/run-make/rustdoc/doctest tests/rustdoc-ui/doctest` when working on doctest internals to quickly see if anything broke instead of having to execute all run-make tests "every time I make a small change"
2. having them in one official place makes it a lot easier to spot which areas might need more coverage, to take inspiration from existing rustdoc run-make tests, to find existing test files that one can extend instead of creating a whole new run-make test, etc.

r? rustdoc
@fmease
fmease deleted the mkdir-tests-run-make-rustdoc branch July 18, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-run-make Area: port run-make Makefiles to rmake.rs A-rustc-dev-guide Area: rustc-dev-guide S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants