compiletest: Remove empty 'expected' files when blessing by clubby789 · Pull Request #134808 · rust-lang/rust
rustbot
added
A-compiletest
labels
Dec 26, 2024
jieyouxu
added
S-waiting-on-author
and removed S-waiting-on-review
Status: Awaiting review from the assignee but also interested parties.labels
Dec 27, 2024
bors
added
S-waiting-on-bors
and removed S-waiting-on-author
Status: This is awaiting some action (such as code changes or more information) from the author.labels
Dec 27, 2024bors added a commit to rust-lang-ci/rust that referenced this pull request
Dec 27, 2024…iaskrgr Rollup of 6 pull requests Successful merges: - rust-lang#133663 (Add a compiler intrinsic to back `bigint_helper_methods`) - rust-lang#134798 (Make `ty::Error` implement all auto traits) - rust-lang#134808 (compiletest: Remove empty 'expected' files when blessing) - rust-lang#134809 (Add `--no-capture`/`--nocapture` as bootstrap arguments) - rust-lang#134826 (Add spastorino to users_on_vacation) - rust-lang#134828 (Add clubby789 back to bootstrap review rotation) r? `@ghost` `@rustbot` modify labels: rollup
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request
Dec 27, 2024Rollup merge of rust-lang#134808 - clubby789:compiletest-remove-stderr, r=jieyouxu compiletest: Remove empty 'expected' files when blessing Fixes rust-lang#134793 Fixes rust-lang#134196 This also refactors `compare_output` to return an enum; returning a usize was done for convenience but is misleading
poliorcetics pushed a commit to poliorcetics/rust that referenced this pull request
Dec 28, 2024…r, r=jieyouxu compiletest: Remove empty 'expected' files when blessing Fixes rust-lang#134793 Fixes rust-lang#134196 This also refactors `compare_output` to return an enum; returning a usize was done for convenience but is misleading
poliorcetics pushed a commit to poliorcetics/rust that referenced this pull request
Dec 28, 2024…iaskrgr Rollup of 6 pull requests Successful merges: - rust-lang#133663 (Add a compiler intrinsic to back `bigint_helper_methods`) - rust-lang#134798 (Make `ty::Error` implement all auto traits) - rust-lang#134808 (compiletest: Remove empty 'expected' files when blessing) - rust-lang#134809 (Add `--no-capture`/`--nocapture` as bootstrap arguments) - rust-lang#134826 (Add spastorino to users_on_vacation) - rust-lang#134828 (Add clubby789 back to bootstrap review rotation) r? `@ghost` `@rustbot` modify labels: rollup
joshtriplett added a commit to joshtriplett/rust that referenced this pull request
Jan 12, 2025rust-lang#134808 made --bless remove empty expected files. Remove some empty files that were causing noise in unrelated `--bless` invocations.
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request
Jan 12, 2025Remove some empty expected files to fix blessing rust-lang#134808 made --bless remove empty expected files. Remove some empty files that were causing noise in unrelated `--bless` invocations.
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request
Jan 12, 2025Remove some empty expected files to fix blessing rust-lang#134808 made --bless remove empty expected files. Remove some empty files that were causing noise in unrelated `--bless` invocations.
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request
Jan 13, 2025Rollup merge of rust-lang#135401 - joshtriplett:empty-expected, r=lqd Remove some empty expected files to fix blessing rust-lang#134808 made --bless remove empty expected files. Remove some empty files that were causing noise in unrelated `--bless` invocations.
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request
Jan 27, 2025…ui-tests, r=compiler-errors Clean up all dead files inside `tests/ui/` While rebasing rust-lang#135860 I noticed that there are several dead `*.stderr` files inside `tests/ui/`. When I checked thoroughly, I found 69 dead `*.$revision.stderr` files, 3 other dead `*.stderr` files and one dead `*.rs` file. Prior to rust-lang#134808, compiletest's `--bless` didn't remove dead `*.stderr` files when the set of revisions changed in any way (renamings, removals, additions, …) which explains their existence. Regarding the dead `*.rs` file, that one was located inside an `auxiliary/` directory (together with a `*.stderr` file) despite not being meant to be an auxiliary file (it's not referenced by any `//@ aux-*`, it has an accompanying `*.stderr` file and it's obvious from looking at rust-lang#111056 which added it). Ideally compiletest or tidy would forbid `*.std{out,err}` files inside `auxiliary/` dirs, that would've caught it. I moved it, updated it and turned it into a proper UI test. --- How to reproduce: 1. Run `rm tests/ui/**/*.stderr` 2. Run `./x test tests/ui --bless` (or similar) 3. Manually / semi-automatically go through all tests that were ignored (likely due to your OS etc. not matching) and restore any stderr files that were overzealously removed --- r? compiler
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request
Jan 27, 2025Rollup merge of rust-lang#136112 - fmease:clean-up-all-dead-files-in-ui-tests, r=compiler-errors Clean up all dead files inside `tests/ui/` While rebasing rust-lang#135860 I noticed that there are several dead `*.stderr` files inside `tests/ui/`. When I checked thoroughly, I found 69 dead `*.$revision.stderr` files, 3 other dead `*.stderr` files and one dead `*.rs` file. Prior to rust-lang#134808, compiletest's `--bless` didn't remove dead `*.stderr` files when the set of revisions changed in any way (renamings, removals, additions, …) which explains their existence. Regarding the dead `*.rs` file, that one was located inside an `auxiliary/` directory (together with a `*.stderr` file) despite not being meant to be an auxiliary file (it's not referenced by any `//@ aux-*`, it has an accompanying `*.stderr` file and it's obvious from looking at rust-lang#111056 which added it). Ideally compiletest or tidy would forbid `*.std{out,err}` files inside `auxiliary/` dirs, that would've caught it. I moved it, updated it and turned it into a proper UI test. --- How to reproduce: 1. Run `rm tests/ui/**/*.stderr` 2. Run `./x test tests/ui --bless` (or similar) 3. Manually / semi-automatically go through all tests that were ignored (likely due to your OS etc. not matching) and restore any stderr files that were overzealously removed --- r? compiler
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters