Skip to content

Commit 3524d8a

Browse files
Rollup merge of rust-lang#120439 - seqre-contrib:moving-ui-tests, r=nnethercote
Move UI issue tests to subdirectories I've moved issue tests numbered 1920, 3668, 5997, 23302, 32122, 40510, 57741, 71676, and 76077 to relevant better-named subdirectories (tracking issue rust-lang#73494). The issues were chosen by having the highest number of files per issue. I adjusted the `ISSUES_ENTRY_LIMIT` because `tidy` was shouting at me.
2 parents 8c2571c + ed55629 commit 3524d8a

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

53 files changed

+5
-2
lines changed

src/tools/tidy/src/ui_tests.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,12 @@ use std::ffi::OsStr;
88
use std::fs;
99
use std::path::{Path, PathBuf};
1010

11+
// FIXME: GitHub's UI truncates file lists that exceed 1000 entries, so these
12+
// should all be 1000 or lower. Limits significantly smaller than 1000 are also
13+
// desirable, because large numbers of files are unwieldy in general. See issue
14+
// #73494.
1115
const ENTRY_LIMIT: usize = 900;
12-
// FIXME: The following limits should be reduced eventually.
13-
const ISSUES_ENTRY_LIMIT: usize = 1849;
16+
const ISSUES_ENTRY_LIMIT: usize = 1807;
1417
const ROOT_ENTRY_LIMIT: usize = 870;
1518

1619
const EXPECTED_TEST_FILE_EXTENSIONS: &[&str] = &[
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)