Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit ed55629

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 8a0b5ae commit ed55629

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

0 commit comments

Comments
 (0)