Skip to content

Commit 2ceb7bc

Browse files
committed
Move multiple UI issue tests to subdirectories
Issue tests numbered 1920, 3668, 5997, 23302, 32122, 40510, 57741, 71676, and 76077 were moved to relevant better-named subdirectories. ISSUES_ENTRY_LIMIT was adjusted to match new number of files and FIXME note was expanded.
1 parent e5121c7 commit 2ceb7bc

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

+5-2
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] = &[

0 commit comments

Comments
 (0)