Introduce cli module to avoid dead code warning

Author: MichaReiser

crates/ruff/tests/lint.rs renamed to crates/ruff/tests/cli/lint.rs

@@ -1,7 +1,5 @@
 //! Tests the interaction of the `lint` configuration section
 
-#![cfg(not(target_family = "wasm"))]
-
 use regex::escape;
 use std::process::Command;
 use std::str;
@@ -12,9 +10,7 @@ use anyhow::Result;
 use insta_cmd::{assert_cmd_snapshot, get_cargo_bin};
 use tempfile::TempDir;
 
-mod test_fixture;
-
-use test_fixture::RuffTestFixture;
+use crate::RuffTestFixture;
 
 const BIN_NAME: &str = "ruff";
 const STDIN_BASE_OPTIONS: &[&str] = &["check", "--no-cache", "--output-format", "concise"];

crates/ruff/tests/test_fixture.rs renamed to crates/ruff/tests/cli/main.rs

@@ -3,6 +3,8 @@
 //! The core concept is borrowed from ty/tests/cli/main.rs and can be extended
 //! with more functionality from there in the future if needed.
 
+#![cfg(not(target_family = "wasm"))]
+
 use anyhow::{Context as _, Result};
 use insta::internals::SettingsBindDropGuard;
 use insta_cmd::get_cargo_bin;
@@ -14,6 +16,8 @@ use std::{
 };
 use tempfile::TempDir;
 
+mod lint;
+
 const BIN_NAME: &str = "ruff";
 
 /// Creates a regex filter for replacing temporary directory paths in snapshots
@@ -195,79 +199,3 @@ impl RuffTestFixture {
         command
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::fs;
-
-    #[test]
-    fn test_tempdir_filter() {
-        let path = "/tmp/test/path";
-        let filter = tempdir_filter(path);
-        assert!(filter.contains("test"));
-        assert!(filter.contains("path"));
-    }
-
-    #[test]
-    fn test_fixture_new() -> Result<()> {
-        let fixture = RuffTestFixture::new()?;
-        assert!(fixture.root().exists());
-        Ok(())
-    }
-
-    #[test]
-    fn test_fixture_with_file() -> Result<()> {
-        let fixture = RuffTestFixture::with_file("test.py", "print('hello')")?;
-        let file_path = fixture.root().join("test.py");
-        assert!(file_path.exists());
-        let content = fs::read_to_string(file_path)?;
-        assert_eq!(content, "print('hello')");
-        Ok(())
-    }
-
-    #[test]
-    fn test_write_file() -> Result<()> {
-        let fixture = RuffTestFixture::new()?;
-        fixture.write_file("nested/dir/file.py", "test content")?;
-        let file_path = fixture.root().join("nested/dir/file.py");
-        assert!(file_path.exists());
-        let content = fs::read_to_string(file_path)?;
-        assert_eq!(content, "test content");
-        Ok(())
-    }
-
-    #[test]
-    fn test_tempdir_filter_method() -> Result<()> {
-        let fixture = RuffTestFixture::new()?;
-        let filter = fixture.tempdir_filter();
-        assert!(!filter.is_empty());
-        Ok(())
-    }
-
-    #[test]
-    fn test_command_creation() -> Result<()> {
-        let fixture = RuffTestFixture::new()?;
-        let command = fixture.command();
-        assert_eq!(command.get_current_dir(), Some(fixture.root()));
-        Ok(())
-    }
-
-    #[test]
-    fn test_dedent_functionality() -> Result<()> {
-        let fixture = RuffTestFixture::new()?;
-        let indented_content = "
-            def hello():
-                print('hello')
-                return True
-        ";
-        fixture.write_file("test_dedent.py", indented_content)?;
-        let file_path = fixture.root().join("test_dedent.py");
-        let content = fs::read_to_string(file_path)?;
-        // Verify that the indentation was removed
-        assert!(content.trim().starts_with("def hello():"));
-        assert!(content.contains("    print('hello')")); // Still has relative indentation
-        assert!(content.contains("    return True"));
-        Ok(())
-    }
-}