rusty_stack/installers/common/

utils.rs

//! Common utility functions — color-aware output, command existence checks, print helpers.
//!
//! Ports functionality from `scripts/common_utils.sh`:
//! - Color constants that auto-disable for non-TTY and `NO_COLOR`
//! - `command_exists()` utility function
//! - Print helpers (header, section, step, success, warning, error)
//!
//! # Validation Assertions
//!
//! - **VAL-INFRA-016**: Color-aware output auto-disabling for non-TTY, command_exists utility

use std::fmt;
use std::io::IsTerminal;
use std::path::Path;
use std::process::Command;

// ===========================================================================
// Color Support
// ===========================================================================

/// ANSI color constants that auto-disable for non-TTY and `NO_COLOR`.
///
/// When color is disabled, all constants are empty strings.
///
/// # Validation
///
/// - **VAL-INFRA-016**: Color-aware terminal output auto-disabling for non-TTY
#[derive(Debug, Clone, Copy)]
pub struct Colors {
    pub red: &'static str,
    pub green: &'static str,
    pub yellow: &'static str,
    pub blue: &'static str,
    pub magenta: &'static str,
    pub cyan: &'static str,
    pub bold: &'static str,
    pub reset: &'static str,
}

/// Color constants with ANSI escape codes.
const COLOR_ENABLED: Colors = Colors {
    red: "\x1b[0;31m",
    green: "\x1b[0;32m",
    yellow: "\x1b[0;33m",
    blue: "\x1b[0;34m",
    magenta: "\x1b[0;35m",
    cyan: "\x1b[0;36m",
    bold: "\x1b[1m",
    reset: "\x1b[0m",
};

/// Color constants when color is disabled (all empty).
const COLOR_DISABLED: Colors = Colors {
    red: "",
    green: "",
    yellow: "",
    blue: "",
    magenta: "",
    cyan: "",
    bold: "",
    reset: "",
};

/// Determine if color output should be used.
///
/// Returns `false` if:
/// - `NO_COLOR` environment variable is set
/// - stderr is not connected to a TTY
pub fn color_enabled() -> bool {
    if std::env::var("NO_COLOR").is_ok() {
        return false;
    }
    std::io::stderr().is_terminal()
}

/// Get the appropriate color constants based on terminal capabilities.
pub fn get_colors() -> &'static Colors {
    if color_enabled() {
        &COLOR_ENABLED
    } else {
        &COLOR_DISABLED
    }
}

// ===========================================================================
// Command Existence
// ===========================================================================

/// Check if a command exists in the system PATH.
///
/// Uses `which` on Unix systems to check for command availability.
/// Returns `true` if the command is found, `false` otherwise.
///
/// # Validation
///
/// - **VAL-INFRA-016**: command_exists utility function
///
/// # Examples
///
/// ```no_run
/// use rusty_stack::installers::common::utils::command_exists;
///
/// if command_exists("python3") {
///     println!("Python 3 is available");
/// }
/// ```
pub fn command_exists(cmd: &str) -> bool {
    Command::new("which")
        .arg(cmd)
        .output()
        .map(|o| o.status.success())
        .unwrap_or(false)
}

/// Resolve the full path to the `rocminfo` binary.
///
/// When running via `sudo`, the user's PATH is typically stripped, so bare
/// `rocminfo` fails with "command not found" even though `/opt/rocm/bin` is
/// in the user's PATH. This function resolves the full path by checking
/// known ROCm installation directories before falling back to bare `rocminfo`.
///
/// Resolution order:
/// 1. `/opt/rocm/bin/rocminfo` (standard ROCm install)
/// 2. Any `/opt/rocm-*/bin/rocminfo` (versioned installs, newest first)
/// 3. `/usr/lib/rocm/bin/rocminfo` (Arch AUR)
/// 4. `rocminfo` bare (last resort, relies on PATH)
pub fn resolve_rocminfo_path() -> String {
    // Standard ROCm path
    let standard = Path::new("/opt/rocm/bin/rocminfo");
    if standard.exists() {
        return standard.to_string_lossy().to_string();
    }

    // Versioned ROCm installs (newest first)
    if let Ok(entries) = std::fs::read_dir("/opt") {
        let mut rocm_dirs: Vec<_> = entries
            .filter_map(|e| e.ok())
            .filter(|e| e.file_name().to_string_lossy().starts_with("rocm-"))
            .collect();
        rocm_dirs.sort_by_key(|b| std::cmp::Reverse(b.file_name()));
        for dir in rocm_dirs {
            let path = dir.path().join("bin/rocminfo");
            if path.exists() {
                return path.to_string_lossy().to_string();
            }
        }
    }

    // Arch AUR path
    let arch_path = Path::new("/usr/lib/rocm/bin/rocminfo");
    if arch_path.exists() {
        return arch_path.to_string_lossy().to_string();
    }

    // Last resort: bare command (relies on PATH being set)
    "rocminfo".to_string()
}

/// Resolve the path to the ROCm rocprofiler SDK tool library.
///
/// Searches in priority order:
/// 1. `<rocm_home>/lib/rocprofiler-sdk/librocprofiler-sdk-tool.so` (ROCm 7.x layout)
/// 2. `<rocm_home>/lib/librocprofiler-sdk-tool.so` (older layout)
/// 3. `/usr/lib/rocm/lib/rocprofiler-sdk/librocprofiler-sdk-tool.so` (Arch AUR)
///
/// Returns `None` if not found.
pub fn resolve_rocprofiler_tool_path() -> Option<String> {
    let candidates = [
        "/opt/rocm/lib/rocprofiler-sdk/librocprofiler-sdk-tool.so",
        "/opt/rocm/lib/librocprofiler-sdk-tool.so",
        "/usr/lib/rocm/lib/rocprofiler-sdk/librocprofiler-sdk-tool.so",
        "/usr/lib/rocm/lib/librocprofiler-sdk-tool.so",
    ];
    for candidate in &candidates {
        if Path::new(candidate).exists() {
            return Some(candidate.to_string());
        }
    }

    // Check versioned ROCm installs
    if let Ok(entries) = std::fs::read_dir("/opt") {
        let mut rocm_dirs: Vec<_> = entries
            .filter_map(|e| e.ok())
            .filter(|e| e.file_name().to_string_lossy().starts_with("rocm-"))
            .collect();
        rocm_dirs.sort_by_key(|b| std::cmp::Reverse(b.file_name()));
        for dir in rocm_dirs {
            let base = dir.path();
            for subpath in [
                "lib/rocprofiler-sdk/librocprofiler-sdk-tool.so",
                "lib/librocprofiler-sdk-tool.so",
            ] {
                let path = base.join(subpath);
                if path.exists() {
                    return Some(path.to_string_lossy().to_string());
                }
            }
        }
    }

    None
}

// ===========================================================================
// Print Helpers
// ===========================================================================

/// Print a formatted header box to stderr.
///
/// Displays a box-drawing header with the given title, centered.
/// All output goes to stderr to avoid breaking command captures.
pub fn print_header(title: &str) {
    let c = get_colors();
    let width = 57;
    let inner = width - 2; // space for ║ on each side
    let padded = format!("=== {title} ===");
    let centered = center_str(&padded, inner);

    let border_line = "═".repeat(inner);
    let empty_line = " ".repeat(inner);

    eprintln!();
    eprintln!("{}╔{border_line}╗{}", c.cyan, c.reset);
    eprintln!("{}║{empty_line}║{}", c.cyan, c.reset);
    eprintln!("{}║{centered}║{}", c.cyan, c.reset);
    eprintln!("{}║{empty_line}║{}", c.cyan, c.reset);
    eprintln!("{}╚{border_line}╝{}", c.cyan, c.reset);
    eprintln!();
}

/// Print a section separator to stderr.
pub fn print_section(title: &str) {
    let c = get_colors();
    eprintln!();
    eprintln!(
        "{}┌─────────────────────────────────────────────────────────┐{}",
        c.blue, c.reset
    );
    eprintln!("{}│ {}{}{}", c.blue, c.bold, title, c.reset);
    eprintln!(
        "{}└─────────────────────────────────────────────────────────┘{}",
        c.blue, c.reset
    );
}

/// Print a step indicator to stderr.
pub fn print_step(message: &str) {
    let c = get_colors();
    eprintln!("{}➤ {}{}", c.magenta, message, c.reset);
}

/// Print a success message to stderr.
pub fn print_success(message: &str) {
    let c = get_colors();
    eprintln!("{}✓ {}{}", c.green, message, c.reset);
}

/// Print a warning message to stderr.
pub fn print_warning(message: &str) {
    let c = get_colors();
    eprintln!("{}⚠ {}{}", c.yellow, message, c.reset);
}

/// Print an error message to stderr.
pub fn print_error(message: &str) {
    let c = get_colors();
    eprintln!("{}✗ {}{}", c.red, message, c.reset);
}

/// Execute a command with dry-run support.
///
/// When `dry_run` is `true`, logs the command without executing it.
/// Returns `Ok(())` on success or `Err` with the error description.
pub fn execute_command(cmd: &str, description: &str, dry_run: bool) -> Result<(), String> {
    let c = get_colors();
    if dry_run {
        eprintln!("{}[DRY-RUN]{} {}: ", c.yellow, c.reset, description);
        eprintln!("  {}{}{}", c.bold, cmd, c.reset);
        return Ok(());
    }

    print_step(&format!("{description}..."));

    let result = Command::new("bash").arg("-c").arg(cmd).output();

    match result {
        Ok(output) if output.status.success() => {
            print_success(&format!("Done: {description}"));
            Ok(())
        }
        Ok(output) => {
            let stderr = String::from_utf8_lossy(&output.stderr);
            print_error(&format!("Failed: {description}"));
            if !stderr.is_empty() {
                eprintln!("  {stderr}");
            }
            Err(format!("Command failed: {cmd}"))
        }
        Err(e) => {
            print_error(&format!("Failed: {description}: {e}"));
            Err(format!("Command execution error: {e}"))
        }
    }
}

// ===========================================================================
// Python Package Manager Detection (VAL-INFRA-018)
// ===========================================================================

/// Detected Python package manager.
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum PythonPkgManager {
    /// uv package manager (preferred).
    Uv,
    /// pip (fallback).
    Pip,
}

impl fmt::Display for PythonPkgManager {
    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
        match self {
            PythonPkgManager::Uv => write!(f, "uv"),
            PythonPkgManager::Pip => write!(f, "pip"),
        }
    }
}

/// Detect the available Python package manager.
///
/// Prefers `uv` if available, falls back to `pip`.
/// Respects `UV_PYTHON` environment variable.
///
/// # Validation
///
/// - **VAL-INFRA-018**: Python package manager detection (uv/pip)
pub fn detect_python_pkg_manager() -> PythonPkgManager {
    if command_exists("uv") {
        PythonPkgManager::Uv
    } else {
        PythonPkgManager::Pip
    }
}

/// Build a pip install command prefix using the detected package manager.
///
/// Returns the command parts (program + base args) for installing packages.
pub fn pip_install_prefix(python_bin: &str) -> Vec<String> {
    let mgr = detect_python_pkg_manager();
    match mgr {
        PythonPkgManager::Uv => {
            let mut args = vec!["uv".to_string(), "pip".to_string(), "install".to_string()];
            // Respect UV_PYTHON env var
            if std::env::var("UV_PYTHON").is_err() {
                args.push("--python".to_string());
                args.push(python_bin.to_string());
            }
            args
        }
        PythonPkgManager::Pip => {
            vec![
                python_bin.to_string(),
                "-m".to_string(),
                "pip".to_string(),
                "install".to_string(),
            ]
        }
    }
}

// ===========================================================================
// Pip Cache Ownership Fix
// ===========================================================================

/// Result of fixing pip cache directory ownership.
#[derive(Debug, Clone, PartialEq, Eq)]
pub enum PipCacheFixResult {
    /// Cache directory does not exist — nothing to fix.
    NoCacheDir,
    /// Cache directory exists and ownership is already correct.
    OwnershipCorrect,
    /// Ownership was wrong and has been fixed.
    Fixed,
    /// Ownership was wrong but the fix failed.
    FixFailed(String),
}

impl std::fmt::Display for PipCacheFixResult {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        match self {
            PipCacheFixResult::NoCacheDir => write!(f, "No pip cache directory found"),
            PipCacheFixResult::OwnershipCorrect => write!(f, "Pip cache ownership already correct"),
            PipCacheFixResult::Fixed => write!(f, "Fixed pip cache ownership"),
            PipCacheFixResult::FixFailed(e) => write!(f, "Failed to fix pip cache ownership: {e}"),
        }
    }
}

/// Fix ownership of the pip cache directory (`~/.cache/pip`).
///
/// When installation steps use `sudo pip install`, the pip cache directory
/// (`~/.cache/pip`) can end up owned by root. This prevents the unprivileged
/// user from writing to the cache, causing permission errors on subsequent
/// `pip install` runs.
///
/// This function:
/// 1. Checks if `~/.cache/pip` exists
/// 2. Compares the directory's owner UID with the current effective UID
/// 3. If the owner is root (UID 0) and we know the target user (via
///    `SUDO_USER` or `HOME` metadata), runs `chown -R` to fix ownership
///
/// # Arguments
///
/// * `user_home` - The user's home directory path (e.g., `/home/user`)
///
/// # Returns
///
/// A `PipCacheFixResult` indicating what action was taken.
///
/// # Safety
///
/// Uses `libc::geteuid()` to check current effective UID. The chown command
/// is only executed when running as root with `SUDO_USER` set, which is the
/// expected scenario when sudo pip operations have corrupted cache ownership.
#[cfg(unix)]
pub fn fix_pip_cache_ownership(user_home: &str) -> PipCacheFixResult {
    use std::os::unix::fs::MetadataExt;

    let cache_dir = Path::new(user_home).join(".cache/pip");

    // Step 1: Check if cache directory exists
    if !cache_dir.exists() {
        return PipCacheFixResult::NoCacheDir;
    }

    // Step 2: Get the expected target UID from the home directory metadata
    // This is more reliable than checking the current euid because when
    // running under sudo, euid is 0 (root) but we want the actual user's UID.
    let home_meta = match std::fs::metadata(user_home) {
        Ok(m) => m,
        Err(_) => return PipCacheFixResult::FixFailed("Cannot stat home directory".into()),
    };
    let target_uid = home_meta.uid();
    let target_gid = home_meta.gid();

    // Step 3: Check the cache directory's current ownership
    let cache_meta = match std::fs::metadata(&cache_dir) {
        Ok(m) => m,
        Err(_) => return PipCacheFixResult::FixFailed("Cannot stat cache directory".into()),
    };

    // If ownership already matches, nothing to do
    if cache_meta.uid() == target_uid && cache_meta.gid() == target_gid {
        return PipCacheFixResult::OwnershipCorrect;
    }

    // Step 4: Fix ownership via chown -R
    // We need to run chown as root (or with appropriate privileges).
    // When running under sudo, we have root privileges.
    let chown_cmd = format!(
        "chown -R {}:{} \"{}\"",
        target_uid,
        target_gid,
        cache_dir.display()
    );

    match Command::new("bash").arg("-c").arg(&chown_cmd).output() {
        Ok(output) if output.status.success() => PipCacheFixResult::Fixed,
        Ok(output) => {
            let stderr = String::from_utf8_lossy(&output.stderr);
            PipCacheFixResult::FixFailed(format!("chown failed: {stderr}"))
        }
        Err(e) => PipCacheFixResult::FixFailed(format!("chown execution error: {e}")),
    }
}

/// Fix ownership of the pip cache directory (non-Unix stub).
///
/// On non-Unix platforms, this is a no-op that always returns `OwnershipCorrect`.
#[cfg(not(unix))]
pub fn fix_pip_cache_ownership(_user_home: &str) -> PipCacheFixResult {
    PipCacheFixResult::OwnershipCorrect
}

// ===========================================================================
// Helpers
// ===========================================================================

/// Center a string within a given width, padding with spaces.
fn center_str(s: &str, width: usize) -> String {
    if s.len() >= width {
        return s.to_string();
    }
    let left = (width - s.len()) / 2;
    let right = width - s.len() - left;
    format!("{}{}{}", " ".repeat(left), s, " ".repeat(right))
}

// ===========================================================================
// Tests
// ===========================================================================

#[cfg(test)]
mod tests {
    use super::*;

    // --- Color tests (VAL-INFRA-016) ---

    #[test]
    fn test_colors_enabled_has_escapes() {
        let c = &COLOR_ENABLED;
        assert!(!c.red.is_empty());
        assert!(!c.green.is_empty());
        assert!(!c.reset.is_empty());
        assert!(c.red.starts_with('\x1b'));
    }

    #[test]
    fn test_colors_disabled_is_empty() {
        let c = &COLOR_DISABLED;
        assert!(c.red.is_empty());
        assert!(c.green.is_empty());
        assert!(c.yellow.is_empty());
        assert!(c.blue.is_empty());
        assert!(c.magenta.is_empty());
        assert!(c.cyan.is_empty());
        assert!(c.bold.is_empty());
        assert!(c.reset.is_empty());
    }

    #[test]
    fn test_get_colors_returns_valid_ref() {
        let _colors = get_colors();
        // Just verify it doesn't panic
    }

    // --- command_exists tests (VAL-INFRA-016) ---

    #[test]
    fn test_command_exists_true() {
        // bash should exist on any Linux system
        assert!(command_exists("bash"));
    }

    #[test]
    fn test_command_exists_false() {
        assert!(!command_exists("nonexistent_command_xyz_12345"));
    }

    #[test]
    fn test_command_exists_which() {
        // which is used by the implementation itself
        assert!(command_exists("which"));
    }

    // --- Print helper tests ---

    #[test]
    fn test_print_header_no_panic() {
        // Just verify no panic
        print_header("Test Title");
    }

    #[test]
    fn test_print_section_no_panic() {
        print_section("Test Section");
    }

    #[test]
    fn test_print_step_no_panic() {
        print_step("Test step");
    }

    #[test]
    fn test_print_success_no_panic() {
        print_success("Test success");
    }

    #[test]
    fn test_print_warning_no_panic() {
        print_warning("Test warning");
    }

    #[test]
    fn test_print_error_no_panic() {
        print_error("Test error");
    }

    // --- execute_command tests ---

    #[test]
    fn test_execute_command_dry_run() {
        let result = execute_command("echo hello", "test dry run", true);
        assert!(result.is_ok());
    }

    #[test]
    fn test_execute_command_success() {
        let result = execute_command("true", "test success", false);
        assert!(result.is_ok());
    }

    #[test]
    fn test_execute_command_failure() {
        let result = execute_command("false", "test failure", false);
        assert!(result.is_err());
    }

    // --- PythonPkgManager tests (VAL-INFRA-018) ---

    #[test]
    fn test_detect_python_pkg_manager() {
        // Should return either Uv or Pip without panicking
        let mgr = detect_python_pkg_manager();
        assert!(mgr == PythonPkgManager::Uv || mgr == PythonPkgManager::Pip);
    }

    #[test]
    fn test_python_pkg_manager_display() {
        assert_eq!(format!("{}", PythonPkgManager::Uv), "uv");
        assert_eq!(format!("{}", PythonPkgManager::Pip), "pip");
    }

    #[test]
    fn test_pip_install_prefix_returns_nonempty() {
        let prefix = pip_install_prefix("python3");
        assert!(!prefix.is_empty());
        // First element should be either "uv" or "python3"
        assert!(prefix[0] == "uv" || prefix[0] == "python3");
    }

    // --- Center string helper ---

    #[test]
    fn test_center_str_short() {
        let result = center_str("hi", 10);
        assert_eq!(result.len(), 10);
        assert_eq!(result.trim(), "hi");
    }

    #[test]
    fn test_center_str_exact() {
        let result = center_str("hello", 5);
        assert_eq!(result, "hello");
    }

    #[test]
    fn test_center_str_longer() {
        let result = center_str("hello world", 5);
        assert_eq!(result, "hello world");
    }

    // --- resolve_rocminfo_path tests ---

    #[test]
    fn test_resolve_rocminfo_path_returns_string() {
        // Should return a string (either a full path or bare "rocminfo")
        let path = resolve_rocminfo_path();
        assert!(
            !path.is_empty(),
            "resolve_rocminfo_path must return non-empty string"
        );
    }

    #[test]
    fn test_resolve_rocminfo_path_prefers_standard_path() {
        // On a system with ROCm at /opt/rocm, should return full path
        let path = resolve_rocminfo_path();
        if Path::new("/opt/rocm/bin/rocminfo").exists() {
            assert_eq!(path, "/opt/rocm/bin/rocminfo");
        }
        // Otherwise just verify it returns something valid
        assert!(!path.is_empty());
    }

    #[test]
    fn test_resolve_rocminfo_path_fallback_is_rocminfo() {
        // When no known ROCm path exists, should fall back to bare "rocminfo"
        // We can't easily test this on a system WITH ROCm, but we can verify
        // the function doesn't panic and returns a valid string
        let path = resolve_rocminfo_path();
        assert!(
            path == "rocminfo" || path.contains("rocminfo"),
            "Path should contain 'rocminfo', got: {path}"
        );
    }

    // --- PipCacheFixResult tests ---

    #[test]
    fn test_pip_cache_fix_result_display_no_cache_dir() {
        assert_eq!(
            format!("{}", PipCacheFixResult::NoCacheDir),
            "No pip cache directory found"
        );
    }

    #[test]
    fn test_pip_cache_fix_result_display_ownership_correct() {
        assert_eq!(
            format!("{}", PipCacheFixResult::OwnershipCorrect),
            "Pip cache ownership already correct"
        );
    }

    #[test]
    fn test_pip_cache_fix_result_display_fixed() {
        assert_eq!(
            format!("{}", PipCacheFixResult::Fixed),
            "Fixed pip cache ownership"
        );
    }

    #[test]
    fn test_pip_cache_fix_result_display_fix_failed() {
        let result = PipCacheFixResult::FixFailed("some error".into());
        assert!(format!("{}", result).contains("some error"));
    }

    #[test]
    fn test_pip_cache_fix_result_equality() {
        assert_eq!(PipCacheFixResult::NoCacheDir, PipCacheFixResult::NoCacheDir);
        assert_eq!(
            PipCacheFixResult::OwnershipCorrect,
            PipCacheFixResult::OwnershipCorrect
        );
        assert_eq!(PipCacheFixResult::Fixed, PipCacheFixResult::Fixed);
        assert_ne!(PipCacheFixResult::NoCacheDir, PipCacheFixResult::Fixed);
    }

    #[test]
    fn test_fix_pip_cache_no_cache_dir() {
        // Use /tmp/nonexistent as home — no .cache/pip there
        let result = fix_pip_cache_ownership("/tmp/nonexistent_test_dir_xyz");
        assert_eq!(result, PipCacheFixResult::NoCacheDir);
    }

    #[test]
    fn test_fix_pip_cache_ownership_correct() {
        // Use the real user home — cache dir may or may not exist.
        // If it exists, it should be owned by the current user already.
        // If it doesn't, we get NoCacheDir.
        let home = std::env::var("HOME").unwrap_or_else(|_| "/tmp".into());
        let result = fix_pip_cache_ownership(&home);
        // Either the dir doesn't exist, or ownership is already correct,
        // or (unlikely in test) it was fixed.
        assert!(
            result == PipCacheFixResult::NoCacheDir
                || result == PipCacheFixResult::OwnershipCorrect
                || result == PipCacheFixResult::Fixed,
            "Expected NoCacheDir, OwnershipCorrect, or Fixed, got: {result}"
        );
    }

    #[test]
    fn test_fix_pip_cache_with_temp_dir_correct_ownership() {
        // Create a temp dir with .cache/pip owned by current user
        let tmp_dir = std::env::temp_dir().join("rusty_stack_test_pip_cache_correct");
        let cache_dir = tmp_dir.join(".cache/pip");
        let _ = std::fs::create_dir_all(&cache_dir);

        let result = fix_pip_cache_ownership(tmp_dir.to_string_lossy().as_ref());
        // Should report ownership correct since we created it as current user
        assert_eq!(result, PipCacheFixResult::OwnershipCorrect);

        // Cleanup
        let _ = std::fs::remove_dir_all(&tmp_dir);
    }

    #[test]
    fn test_fix_pip_cache_with_temp_dir_no_cache() {
        // Create a temp dir without .cache/pip
        let tmp_dir = std::env::temp_dir().join("rusty_stack_test_pip_cache_none");
        let _ = std::fs::create_dir_all(&tmp_dir);

        let result = fix_pip_cache_ownership(tmp_dir.to_string_lossy().as_ref());
        assert_eq!(result, PipCacheFixResult::NoCacheDir);

        // Cleanup
        let _ = std::fs::remove_dir_all(&tmp_dir);
    }

    #[test]
    fn test_fix_pip_cache_result_debug() {
        // Verify Debug trait works
        let result = PipCacheFixResult::Fixed;
        let debug_str = format!("{:?}", result);
        assert!(debug_str.contains("Fixed"));
    }
}