diff options
| author | bjorn3 <17426603+bjorn3@users.noreply.github.com> | 2025-07-24 13:01:08 +0000 |
|---|---|---|
| committer | bjorn3 <17426603+bjorn3@users.noreply.github.com> | 2025-07-24 13:01:08 +0000 |
| commit | b481c5b8ba322612fb3bc167e323f7a548a2390c (patch) | |
| tree | 834f0cad95c61f1e0defb9b19683c68b35f8e42d /src | |
| parent | c5d7021cddd97a4ec8dd64f878711ec983986fcd (diff) | |
| download | rust-b481c5b8ba322612fb3bc167e323f7a548a2390c.tar.gz rust-b481c5b8ba322612fb3bc167e323f7a548a2390c.zip | |
std_detect testing improvements
* Fix riscv testing. Previously the mod tests; would be looking for src/detect/os/tests.rs. * Replace a test with an unnamed const item. It is testing that no warnings are emitted. It doesn't contain any checks that need to run at runtime. Replacing the test allows removing the tidy:skip directive for test locations.
Diffstat (limited to 'src')
| -rw-r--r-- | src/tools/tidy/src/unit_tests.rs | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/tools/tidy/src/unit_tests.rs b/src/tools/tidy/src/unit_tests.rs index 90da1bc765a..3d14a467319 100644 --- a/src/tools/tidy/src/unit_tests.rs +++ b/src/tools/tidy/src/unit_tests.rs @@ -79,8 +79,7 @@ pub fn check(root_path: &Path, stdlib: bool, bad: &mut bool) { let line = line.trim(); let is_test = || line.contains("#[test]") && !line.contains("`#[test]"); let is_bench = || line.contains("#[bench]") && !line.contains("`#[bench]"); - let manual_skip = line.contains("//tidy:skip"); - if !line.starts_with("//") && (is_test() || is_bench()) && !manual_skip { + if !line.starts_with("//") && (is_test() || is_bench()) { let explanation = if stdlib { format!( "`{package}` unit tests and benchmarks must be placed into `{package}tests`" |