Assert that test names cannot contain dots

This is so that we can catch stray test output files, since test names without dots can form predictable patterns we can match on.
matthiaskrgr · Mar 5, 2024 · 53f48dd · 53f48dd
1 parent 9bebbf1
commit 53f48dd
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/tools/tidy/src/tests_revision_unpaired_stdout_stderr.rs b/src/tools/tidy/src/tests_revision_unpaired_stdout_stderr.rs
@@ -88,6 +88,12 @@ pub fn check(tests_path: impl AsRef<Path>, bad: &mut bool) {
                 continue;
             };
 
+            assert!(
+                !test_name.contains('.'),
+                "test name cannot contain dots '.': `{}`",
+                test.display()
+            );
+
             test_info.insert(test_name.to_string(), (test, expected_revisions));
         }