netspeed-cli 0.10.3

//! Output formatting for speed test results.
//!
//! This module is organized into submodules:
//! - [`ratings`] — Rating helper functions (ping, speed, connection, bufferbloat)
//! - [`sections`] — Output section formatters (latency, download, upload, etc.)
//! - [`stability`] — Speed stability analysis and latency percentiles
//! - [`estimates`] — Usage check targets and download time estimates

use crate::common;
use crate::error::Error;
use crate::grades;
use crate::profiles::UserProfile;
use crate::terminal;
use crate::theme::{Colors, Theme};
use crate::types::{CsvOutput, TestResult};
use owo_colors::OwoColorize;

/// Which test phases were skipped by the user (e.g. `--no-download`).
#[derive(Debug, Clone, Copy, Default)]
pub struct SkipState {
    /// Download test was skipped.
    pub download: bool,
    /// Upload test was skipped.
    pub upload: bool,
}

/// Build a section header with consistent formatting.
fn section_header(title: &str, nc: bool, theme: Theme) -> String {
    if nc {
        format!("  ◈ {title}")
    } else {
        format!(
            "  {} {}",
            Colors::muted("◈", theme),
            Colors::header(title, theme)
        )
    }
}

/// Output format selection — Strategy pattern.
/// Add new variants here to extend output formats (OCP).
#[derive(Debug)]
pub enum OutputFormat {
    Json,
    Csv {
        delimiter: char,
        header: bool,
    },
    Simple {
        theme: Theme,
    },
    Minimal {
        theme: Theme,
    },
    Jsonl,
    Compact {
        dl_bytes: u64,
        ul_bytes: u64,
        dl_duration: f64,
        ul_duration: f64,
        elapsed: std::time::Duration,
        profile: UserProfile,
        theme: Theme,
    },
    Detailed {
        dl_bytes: u64,
        ul_bytes: u64,
        dl_duration: f64,
        ul_duration: f64,
        skipped: SkipState,
        elapsed: std::time::Duration,
        profile: UserProfile,
        minimal: bool,
        theme: Theme,
    },
    Dashboard {
        dl_mbps: f64,
        dl_peak_mbps: f64,
        dl_bytes: u64,
        dl_duration: f64,
        ul_mbps: f64,
        ul_peak_mbps: f64,
        ul_bytes: u64,
        ul_duration: f64,
        elapsed: std::time::Duration,
        profile: UserProfile,
        theme: Theme,
    },
}

impl OutputFormat {
    /// Execute the formatting strategy.
    ///
    /// # Errors
    ///
    /// Returns an error if output serialization or writing fails.
    pub fn format(&self, result: &TestResult, bytes: bool) -> Result<(), Error> {
        match self {
            OutputFormat::Json => format_json(result),
            OutputFormat::Jsonl => format_jsonl(result),
            OutputFormat::Csv { delimiter, header } => format_csv(result, *delimiter, *header),
            OutputFormat::Simple { theme } => format_simple(result, bytes, *theme),
            OutputFormat::Minimal { theme } => format_minimal(result, bytes, *theme),
            OutputFormat::Compact {
                dl_bytes,
                ul_bytes,
                dl_duration,
                ul_duration,
                elapsed,
                profile,
                theme,
            } => {
                format_compact(
                    result,
                    bytes,
                    *dl_bytes,
                    *ul_bytes,
                    *dl_duration,
                    *ul_duration,
                    *elapsed,
                    *profile,
                    *theme,
                );
                Ok(())
            }
            OutputFormat::Detailed {
                dl_bytes,
                ul_bytes,
                dl_duration,
                ul_duration,
                skipped,
                elapsed,
                profile,
                minimal,
                theme,
            } => {
                format_detailed(
                    result,
                    bytes,
                    *dl_bytes,
                    *ul_bytes,
                    *dl_duration,
                    *ul_duration,
                    *skipped,
                    *elapsed,
                    *profile,
                    *minimal,
                    *theme,
                )?;
                format_verbose_sections(result, *profile, *minimal, *theme);
                Ok(())
            }
            OutputFormat::Dashboard {
                dl_mbps,
                dl_peak_mbps,
                dl_bytes,
                dl_duration,
                ul_mbps,
                ul_peak_mbps,
                ul_bytes,
                ul_duration,
                elapsed,
                profile,
                theme,
            } => {
                dashboard::show(
                    result,
                    &dashboard::Summary {
                        dl_mbps: *dl_mbps,
                        dl_peak_mbps: *dl_peak_mbps,
                        dl_bytes: *dl_bytes,
                        dl_duration: *dl_duration,
                        ul_mbps: *ul_mbps,
                        ul_peak_mbps: *ul_peak_mbps,
                        ul_bytes: *ul_bytes,
                        ul_duration: *ul_duration,
                        elapsed: *elapsed,
                        profile: *profile,
                        theme: *theme,
                    },
                )?;
                Ok(())
            }
        }
    }
}

/// Trait for output formatting strategies.
///
/// Implement this trait to provide custom output formatters.
/// This enables the Open-Closed Principle: new formatters can be added
/// without modifying existing code that uses formatters.
///
/// # Example
///
/// ```
/// use netspeed_cli::formatter::{Formatter, OutputFormat};
/// use netspeed_cli::types::{Server, TestResult};
/// use netspeed_cli::error::Error;
///
/// struct MyFormatter;
///
/// impl Formatter for MyFormatter {
///     fn format(&self, result: &TestResult, use_bytes: bool) -> Result<(), Error> {
///         println!("Custom: {:?}", result.ping);
///         Ok(())
///     }
///
///     fn format_list(&self, servers: &[Server]) -> Result<(), Error> {
///         println!("Servers: {}", servers.len());
///         Ok(())
///     }
/// }
/// ```
pub trait Formatter: Send + Sync {
    /// Format a test result for output.
    ///
    /// # Errors
    ///
    /// Returns an error if output fails.
    fn format(
        &self,
        result: &crate::types::TestResult,
        use_bytes: bool,
    ) -> Result<(), crate::error::Error>;

    /// Format a list of servers for output.
    ///
    /// # Errors
    ///
    /// Returns an error if output fails.
    fn format_list(&self, servers: &[crate::types::Server]) -> Result<(), crate::error::Error>;
}

/// Allows using `OutputFormat` polymorphically through the trait.
impl Formatter for OutputFormat {
    fn format(
        &self,
        result: &crate::types::TestResult,
        use_bytes: bool,
    ) -> Result<(), crate::error::Error> {
        self.format(result, use_bytes)
    }

    fn format_list(&self, servers: &[crate::types::Server]) -> Result<(), crate::error::Error> {
        sections::format_list(servers, Theme::Dark).map_err(crate::error::Error::IoError)
    }
}

pub mod dashboard;
pub mod estimates;
pub mod ratings;
pub mod scenarios;
pub mod sections;
pub mod stability;

// Re-export commonly used functions for backward compatibility
pub use estimates::{format_targets, show};
pub use ratings::{
    BufferbloatGrade, bufferbloat_colorized, bufferbloat_grade, colorize_rating, connection_rating,
    degradation_str, format_duration, format_overall_rating, format_speed_colored,
    format_speed_plain, ping_rating, speed_rating_mbps,
};
pub use sections::{
    build_elapsed_time, format_connection_info, format_download_section, format_elapsed_time,
    format_footer, format_latency_section, format_list, format_test_summary, format_upload_section,
};
pub use stability::{compute_cv, compute_percentiles, format_stability_line};

/// Simple mode — single line.
///
/// # Errors
///
/// This function does not currently return errors, but the signature is
/// `Result` for future extensibility.
pub fn format_simple(result: &TestResult, bytes: bool, theme: Theme) -> Result<(), Error> {
    let nc = terminal::no_color();
    let mut parts = Vec::new();

    if let Some(ping) = result.ping {
        parts.push(if nc {
            format!("Latency: {ping:.1} ms")
        } else {
            format!("Latency: {} ms", Colors::info(&format!("{ping:.1}"), theme))
        });
    }

    if let Some(dl) = result.download {
        let speed = if nc {
            ratings::format_speed_plain(dl, bytes)
        } else {
            ratings::format_speed_colored(dl, bytes, theme)
        };
        parts.push(format!("Download: {speed}"));
    }

    if let Some(ul) = result.upload {
        let speed = if nc {
            ratings::format_speed_plain(ul, bytes)
        } else {
            ratings::format_speed_colored(ul, bytes, theme)
        };
        parts.push(format!("Upload: {speed}"));
    }

    eprintln!("{}", parts.join(" | "));
    Ok(())
}

/// Minimal mode — ultra-compact: just "B+ 150.5↓ 25.3↑ 12ms"
///
/// # Errors
///
/// This function does not currently return errors, but the signature is
/// `Result` for future extensibility.
pub fn format_minimal(result: &TestResult, _bytes: bool, theme: Theme) -> Result<(), Error> {
    let nc = terminal::no_color();
    let profile = UserProfile::default();

    let overall_grade = grades::grade_overall(
        result.ping,
        result.jitter,
        result.download,
        result.upload,
        profile,
    );
    let grade_str = if nc {
        format!("[{}]", overall_grade.as_str())
    } else {
        overall_grade.color_str(nc, theme)
    };

    let dl_str = result.download.map_or_else(
        || "—↓".to_string(),
        |d| {
            let mbps = d / 1_000_000.0;
            format!("{mbps:.1}↓")
        },
    );

    let ul_str = result.upload.map_or_else(
        || "—↑".to_string(),
        |u| {
            let mbps = u / 1_000_000.0;
            format!("{mbps:.1}↑")
        },
    );

    let lat_str = result
        .ping
        .map_or_else(|| "—ms".to_string(), |p| format!("{p:.0}ms"));

    eprintln!("{grade_str}  {dl_str}  {ul_str}  {lat_str}");
    Ok(())
}

/// JSONL mode — one JSON object per line, ideal for logging/parsing.
///
/// # Errors
///
/// Returns [`Error::ParseJson`] if serialization fails.
pub fn format_jsonl(result: &TestResult) -> Result<(), Error> {
    println!("{}", serde_json::to_string(result)?);
    Ok(())
}

/// Compact mode — key metrics with ratings and brief summary.
/// Middle ground between simple (too minimal) and detailed (too verbose).
pub fn format_compact(
    result: &TestResult,
    bytes: bool,
    dl_bytes: u64,
    ul_bytes: u64,
    dl_duration: f64,
    ul_duration: f64,
    elapsed: std::time::Duration,
    profile: UserProfile,
    theme: Theme,
) {
    let nc = terminal::no_color();
    let overall_grade = grades::grade_overall(
        result.ping,
        result.jitter,
        result.download,
        result.upload,
        profile,
    );
    let term_w = common::get_terminal_width().unwrap_or(90) as usize;

    eprintln!();
    eprintln!(
        "{}",
        dashboard::boxed_header(&overall_grade, nc, theme, term_w)
    );
    eprintln!();

    sections::format_latency_section(result, nc, theme);
    eprintln!();

    sections::format_download_section(result, bytes, nc, false, theme);
    eprintln!();

    sections::format_upload_section(result, bytes, nc, false, theme);
    eprintln!();

    if let Some(ip) = &result.client_ip {
        if nc {
            eprintln!("  Server: {} · {}", result.server.sponsor, ip);
        } else {
            eprintln!(
                "  {} {} · {}",
                "Server:".dimmed(),
                Colors::bold(&result.server.sponsor, theme),
                Colors::muted(ip, theme),
            );
        }
        eprintln!();
    }

    if nc {
        eprintln!("  SUMMARY");
    } else {
        eprintln!("  {}", Colors::header("SUMMARY", theme));
    }
    if dl_bytes > 0 {
        eprintln!(
            "  {:>14}:   {} in {:.1}s",
            Colors::dimmed("Download", theme),
            common::format_data_size(dl_bytes),
            dl_duration
        );
    }
    if ul_bytes > 0 {
        eprintln!(
            "  {:>14}:   {} in {:.1}s",
            Colors::dimmed("Upload", theme),
            common::format_data_size(ul_bytes),
            ul_duration
        );
    }

    eprintln!();
    if nc {
        eprintln!("  Total time: {:.1}s", elapsed.as_secs_f64());
    } else {
        eprintln!(
            "  {}: {}",
            "Total time".dimmed(),
            Colors::info(&format!("{:.1}s", elapsed.as_secs_f64()), theme),
        );
    }
    sections::format_footer(&result.timestamp, nc, theme);
}

/// Detailed mode — clean key/value pairs.
///
/// # Errors
///
/// This function does not currently return errors, but the signature is
/// `Result` for future extensibility.
#[allow(clippy::too_many_arguments)]
pub fn format_detailed(
    result: &TestResult,
    bytes: bool,
    dl_bytes: u64,
    ul_bytes: u64,
    dl_duration: f64,
    ul_duration: f64,
    skipped: SkipState,
    elapsed: std::time::Duration,
    profile: UserProfile,
    minimal: bool,
    theme: Theme,
) -> Result<(), Error> {
    let nc = terminal::no_color() || minimal;
    let overall_grade = grades::grade_overall(
        result.ping,
        result.jitter,
        result.download,
        result.upload,
        profile,
    );
    let term_w = common::get_terminal_width().unwrap_or(90) as usize;

    eprintln!();
    eprintln!(
        "{}",
        dashboard::boxed_header(&overall_grade, nc, theme, term_w)
    );
    eprintln!();

    sections::format_latency_section(result, nc, theme);
    eprintln!();
    sections::format_download_section(result, bytes, nc, skipped.download, theme);
    eprintln!();
    sections::format_upload_section(result, bytes, nc, skipped.upload, theme);
    sections::format_connection_info(result, nc, theme);
    sections::format_test_summary(dl_bytes, ul_bytes, dl_duration, ul_duration, nc, theme);

    eprintln!();
    if nc {
        eprintln!("  Total time: {:.1}s", elapsed.as_secs_f64());
    } else {
        eprintln!(
            "  {}: {}",
            "Total time".dimmed(),
            Colors::info(&format!("{:.1}s", elapsed.as_secs_f64()), theme),
        );
    }

    sections::format_footer(&result.timestamp, nc, theme);

    Ok(())
}

/// Output test results as JSON to stdout.
///
/// # Errors
///
/// Returns [`Error::ParseJson`] if serialization fails.
pub fn format_json(result: &TestResult) -> Result<(), Error> {
    let is_tty = {
        use std::io::IsTerminal;
        std::io::stdout().is_terminal()
    };
    let output = if is_tty {
        serde_json::to_string_pretty(result)?
    } else {
        serde_json::to_string(result)?
    };
    println!("{output}");
    Ok(())
}

/// Output test results as CSV to stdout.
///
/// # Errors
///
/// Returns [`Error::Csv`] if CSV serialization fails.
pub fn format_csv(result: &TestResult, delimiter: char, print_header: bool) -> Result<(), Error> {
    let stdout = std::io::stdout();
    let mut wtr = csv::WriterBuilder::new()
        .delimiter(delimiter as u8)
        .from_writer(stdout);
    if print_header {
        wtr.write_record([
            "Server ID",
            "Sponsor",
            "Server Name",
            "Timestamp",
            "Distance",
            "Ping",
            "Jitter",
            "Packet Loss",
            "Download",
            "Download Peak",
            "Upload",
            "Upload Peak",
            "IP Address",
        ])?;
    }
    let csv_output = CsvOutput {
        server_id: result.server.id.clone(),
        sponsor: result.server.sponsor.clone(),
        server_name: result.server.name.clone(),
        timestamp: result.timestamp.clone(),
        distance: result.server.distance,
        ping: result.ping.unwrap_or(0.0),
        jitter: result.jitter.unwrap_or(0.0),
        packet_loss: result.packet_loss.unwrap_or(0.0),
        download: result.download.unwrap_or(0.0),
        download_peak: result.download_peak.unwrap_or(0.0),
        upload: result.upload.unwrap_or(0.0),
        upload_peak: result.upload_peak.unwrap_or(0.0),
        ip_address: result.client_ip.clone().unwrap_or_default(),
    };
    wtr.serialize(csv_output)?;
    wtr.flush()?;
    Ok(())
}

/// Format additional verbose output sections: stability, latency percentiles, and historical comparison.
/// Only used in detailed (verbose) mode.
pub fn format_verbose_sections(
    result: &TestResult,
    profile: UserProfile,
    minimal: bool,
    theme: Theme,
) {
    let nc = terminal::no_color() || minimal;

    if profile.show_estimates() {
        let estimates = estimates::build(result.download, nc, theme);
        if !estimates.is_empty() {
            eprintln!("{estimates}");
        }
    }

    if profile.show_stability() {
        if let (Some(dl_s), Some(ul_s)) = (&result.download_samples, &result.upload_samples) {
            let dl_cv = compute_cv(dl_s);
            let ul_cv = compute_cv(ul_s);
            let dl_grade = grades::grade_stability(dl_cv);
            let ul_grade = grades::grade_stability(ul_cv);
            let dl_stability = format_stability_line(dl_cv, nc, theme);
            let ul_stability = format_stability_line(ul_cv, nc, theme);
            eprintln!();
            eprintln!("{}", section_header("STABILITY", nc, theme));
            if nc {
                eprintln!("  {:>14}:   [{dl_stability}]", "Download");
                eprintln!("  {:>14}:   [{ul_stability}]", "Upload");
            } else {
                eprintln!(
                    "  {:>14}:   {} {dl_stability}",
                    "Download".dimmed(),
                    dl_grade.color_str(nc, theme)
                );
                eprintln!(
                    "  {:>14}:   {} {ul_stability}",
                    "Upload".dimmed(),
                    ul_grade.color_str(nc, theme)
                );
            }
        }
    }

    if profile.show_percentiles() {
        if let Some(ref samples) = result.ping_samples {
            if let Some((p50, p95, p99)) = compute_percentiles(samples) {
                eprintln!();
                eprintln!("{}", section_header("LATENCY PERCENTILES", nc, theme));
                let p50_str = format!("{p50:.1} ms");
                let p95_str = format!("{p95:.1} ms");
                let p99_str = format!("{p99:.1} ms");
                if nc {
                    eprintln!("  P50: {p50_str}  P95: {p95_str}  P99: {p99_str}");
                } else {
                    eprintln!(
                        "  {}: {}  {}: {}  {}: {}",
                        "P50".dimmed(),
                        Colors::info(&p50_str, theme),
                        "P95".dimmed(),
                        Colors::warn(&p95_str, theme),
                        "P99".dimmed(),
                        Colors::bad(&p99_str, theme),
                    );
                }
            }
        }
    }

    if profile.show_history() {
        let dl_mbps = result.download.map_or(0.0, |d| d / 1_000_000.0);
        let ul_mbps = result.upload.map_or(0.0, |u| u / 1_000_000.0);
        if let Some(comparison) = crate::history::format_comparison(dl_mbps, ul_mbps, nc, theme) {
            eprintln!();
            eprintln!("  {comparison}");
        }
    }
}

// =============================================================================
// Formatter Factory - SOLID: Factory pattern for flexible creation
// =============================================================================

/// Factory for creating formatter instances.
///
/// Enables runtime formatter selection and dependency injection.
pub struct FormatterFactory;

impl FormatterFactory {
    /// Create a formatter from config format option and theme.
    pub fn create(format: Option<crate::config::Format>, theme: Theme) -> Box<dyn Formatter> {
        match format {
            Some(crate::config::Format::Json) => Box::new(OutputFormat::Json),
            Some(crate::config::Format::Jsonl) => Box::new(OutputFormat::Jsonl),
            Some(crate::config::Format::Csv) => Box::new(OutputFormat::Csv {
                delimiter: ',',
                header: true,
            }),
            Some(crate::config::Format::Simple) => Box::new(OutputFormat::Simple { theme }),
            Some(crate::config::Format::Minimal) => Box::new(OutputFormat::Minimal { theme }),
            Some(crate::config::Format::Compact) => Box::new(OutputFormat::Compact {
                dl_bytes: 0,
                ul_bytes: 0,
                dl_duration: 0.0,
                ul_duration: 0.0,
                elapsed: std::time::Duration::ZERO,
                profile: crate::profiles::UserProfile::default(),
                theme,
            }),
            Some(crate::config::Format::Detailed) => Box::new(OutputFormat::Detailed {
                dl_bytes: 0,
                ul_bytes: 0,
                dl_duration: 0.0,
                ul_duration: 0.0,
                skipped: SkipState::default(),
                elapsed: std::time::Duration::ZERO,
                profile: crate::profiles::UserProfile::default(),
                minimal: false,
                theme,
            }),
            Some(crate::config::Format::Dashboard) => Box::new(OutputFormat::Dashboard {
                dl_mbps: 0.0,
                dl_peak_mbps: 0.0,
                dl_bytes: 0,
                dl_duration: 0.0,
                ul_mbps: 0.0,
                ul_peak_mbps: 0.0,
                ul_bytes: 0,
                ul_duration: 0.0,
                elapsed: std::time::Duration::ZERO,
                profile: crate::profiles::UserProfile::default(),
                theme,
            }),
            None => Box::new(OutputFormat::Simple { theme }),
        }
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use crate::types::{PhaseResult, ServerInfo, TestPhases, TestResult};

    fn make_test_result() -> TestResult {
        TestResult {
            status: "ok".to_string(),
            version: env!("CARGO_PKG_VERSION").to_string(),
            test_id: None,
            server: ServerInfo {
                id: "1".to_string(),
                name: "Test".to_string(),
                sponsor: "Test ISP".to_string(),
                country: "US".to_string(),
                distance: 10.0,
            },
            ping: Some(15.0),
            jitter: Some(1.5),
            packet_loss: Some(0.0),
            download: Some(100_000_000.0),
            download_peak: Some(120_000_000.0),
            upload: Some(50_000_000.0),
            upload_peak: Some(60_000_000.0),
            latency_download: Some(20.0),
            latency_upload: Some(18.0),
            download_samples: Some(vec![95_000_000.0, 100_000_000.0, 105_000_000.0]),
            upload_samples: Some(vec![48_000_000.0, 50_000_000.0, 52_000_000.0]),
            ping_samples: Some(vec![12.0, 15.0, 18.0]),
            timestamp: "2026-01-01T00:00:00Z".to_string(),
            client_ip: Some("192.168.1.100".to_string()),
            client_location: None,
            download_cv: Some(0.05),
            upload_cv: Some(0.04),
            download_ci_95: Some((140.0, 160.0)),
            upload_ci_95: Some((45.0, 55.0)),
            overall_grade: None,
            download_grade: None,
            upload_grade: None,
            connection_rating: None,
            phases: TestPhases {
                ping: PhaseResult::completed(),
                download: PhaseResult::completed(),
                upload: PhaseResult::completed(),
            },
        }
    }

    #[test]
    fn test_format_simple_with_data() {
        let result = make_test_result();
        // Just verify it doesn't panic
        let _ = format_simple(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_simple_no_ping() {
        let mut result = make_test_result();
        result.ping = None;
        let _ = format_simple(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_simple_no_download() {
        let mut result = make_test_result();
        result.download = None;
        let _ = format_simple(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_simple_no_upload() {
        let mut result = make_test_result();
        result.upload = None;
        let _ = format_simple(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_simple_bytes_mode() {
        let result = make_test_result();
        let _ = format_simple(&result, true, Theme::Dark);
    }

    #[test]
    fn test_format_simple_light_theme() {
        let result = make_test_result();
        let _ = format_simple(&result, false, Theme::Light);
    }

    #[test]
    fn test_format_minimal_basic() {
        let result = make_test_result();
        let _ = format_minimal(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_minimal_no_download() {
        let mut result = make_test_result();
        result.download = None;
        let _ = format_minimal(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_minimal_no_upload() {
        let mut result = make_test_result();
        result.upload = None;
        let _ = format_minimal(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_minimal_no_ping() {
        let mut result = make_test_result();
        result.ping = None;
        let _ = format_minimal(&result, false, Theme::Dark);
    }

    #[test]
    fn test_format_jsonl_basic() {
        let result = make_test_result();
        let _ = format_jsonl(&result);
    }

    #[test]
    fn test_format_compact_basic() {
        let result = make_test_result();
        format_compact(
            &result,
            false,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            std::time::Duration::from_secs(5),
            UserProfile::default(),
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_compact_with_client_ip() {
        let result = make_test_result();
        format_compact(
            &result,
            false,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            std::time::Duration::from_secs(5),
            UserProfile::default(),
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_compact_bytes_mode() {
        let result = make_test_result();
        format_compact(
            &result,
            true,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            std::time::Duration::from_secs(5),
            UserProfile::default(),
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_compact_nc_mode() {
        let result = make_test_result();
        // NC mode should not use colors
        format_compact(
            &result,
            false,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            std::time::Duration::from_secs(5),
            UserProfile::default(),
            Theme::Monochrome,
        );
    }

    #[test]
    fn test_format_compact_gamer_profile() {
        let result = make_test_result();
        format_compact(
            &result,
            false,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            std::time::Duration::from_secs(5),
            UserProfile::Gamer,
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_detailed_basic() {
        let result = make_test_result();
        let _ = format_detailed(
            &result,
            false,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            SkipState {
                download: false,
                upload: false,
            },
            std::time::Duration::from_secs(5),
            UserProfile::default(),
            false,
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_detailed_with_skipped() {
        let result = make_test_result();
        let _ = format_detailed(
            &result,
            false,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            SkipState {
                download: true,
                upload: true,
            },
            std::time::Duration::from_secs(3),
            UserProfile::default(),
            false,
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_detailed_minimal_mode() {
        let result = make_test_result();
        let _ = format_detailed(
            &result,
            false,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            SkipState {
                download: false,
                upload: false,
            },
            std::time::Duration::from_secs(5),
            UserProfile::default(),
            true,
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_detailed_bytes_mode() {
        let result = make_test_result();
        let _ = format_detailed(
            &result,
            true,
            10_000_000,
            5_000_000,
            2.0,
            1.0,
            SkipState {
                download: false,
                upload: false,
            },
            std::time::Duration::from_secs(5),
            UserProfile::default(),
            false,
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_json_basic() {
        let result = make_test_result();
        let _ = format_json(&result);
    }

    #[test]
    fn test_format_csv_basic() {
        let result = make_test_result();
        let _ = format_csv(&result, ',', true);
    }

    #[test]
    fn test_format_csv_no_header() {
        let result = make_test_result();
        let _ = format_csv(&result, ';', false);
    }

    #[test]
    fn test_format_csv_tab_delimiter() {
        let result = make_test_result();
        let _ = format_csv(&result, '\t', true);
    }

    #[test]
    fn test_format_csv_with_missing_values() {
        let mut result = make_test_result();
        result.ping = None;
        result.jitter = None;
        result.packet_loss = None;
        let _ = format_csv(&result, ',', true);
    }

    #[test]
    fn test_skip_state_default() {
        let skip = SkipState::default();
        assert!(!skip.download);
        assert!(!skip.upload);
    }

    #[test]
    fn test_skip_state_custom() {
        let skip = SkipState {
            download: true,
            upload: false,
        };
        assert!(skip.download);
        assert!(!skip.upload);
    }

    #[test]
    fn test_output_format_json() {
        let fmt = OutputFormat::Json;
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_output_format_jsonl() {
        let fmt = OutputFormat::Jsonl;
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_output_format_csv() {
        let fmt = OutputFormat::Csv {
            delimiter: ',',
            header: true,
        };
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_output_format_simple() {
        let fmt = OutputFormat::Simple { theme: Theme::Dark };
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_output_format_minimal() {
        let fmt = OutputFormat::Minimal { theme: Theme::Dark };
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_output_format_detailed() {
        let fmt = OutputFormat::Detailed {
            dl_bytes: 10_000_000,
            ul_bytes: 5_000_000,
            dl_duration: 2.0,
            ul_duration: 1.0,
            skipped: SkipState::default(),
            elapsed: std::time::Duration::from_secs(5),
            profile: UserProfile::default(),
            minimal: false,
            theme: Theme::Dark,
        };
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_output_format_compact() {
        let fmt = OutputFormat::Compact {
            dl_bytes: 10_000_000,
            ul_bytes: 5_000_000,
            dl_duration: 2.0,
            ul_duration: 1.0,
            elapsed: std::time::Duration::from_secs(5),
            profile: UserProfile::default(),
            theme: Theme::Dark,
        };
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_output_format_dashboard() {
        let fmt = OutputFormat::Dashboard {
            dl_mbps: 100.0,
            dl_peak_mbps: 120.0,
            dl_bytes: 10_000_000,
            dl_duration: 2.0,
            ul_mbps: 50.0,
            ul_peak_mbps: 60.0,
            ul_bytes: 5_000_000,
            ul_duration: 1.0,
            elapsed: std::time::Duration::from_secs(5),
            profile: UserProfile::default(),
            theme: Theme::Dark,
        };
        let result = make_test_result();
        assert!(fmt.format(&result, false).is_ok());
    }

    #[test]
    fn test_section_header_nc_mode() {
        let header = section_header("Test Header", true, Theme::Dark);
        assert!(header.contains("Test Header"));
        assert!(!header.contains("\x1b")); // No ANSI codes
    }

    #[test]
    fn test_section_header_colored() {
        let header = section_header("Test Header", false, Theme::Dark);
        assert!(header.contains("Test Header"));
    }

    #[test]
    fn test_format_verbose_sections_power_user() {
        let result = make_test_result();
        // PowerUser sees estimates, stability, percentiles, history
        format_verbose_sections(&result, UserProfile::PowerUser, false, Theme::Dark);
    }

    #[test]
    fn test_format_verbose_sections_casual() {
        let result = make_test_result();
        // Casual only sees estimates
        format_verbose_sections(&result, UserProfile::Casual, false, Theme::Dark);
    }

    #[test]
    fn test_format_verbose_sections_gamer() {
        let result = make_test_result();
        // Gamer sees bufferbloat but not stability
        format_verbose_sections(&result, UserProfile::Gamer, false, Theme::Dark);
    }

    #[test]
    fn test_format_verbose_sections_remote_worker() {
        let result = make_test_result();
        // RemoteWorker sees stability and history
        format_verbose_sections(&result, UserProfile::RemoteWorker, false, Theme::Dark);
    }

    #[test]
    fn test_format_verbose_sections_minimal() {
        let result = make_test_result();
        format_verbose_sections(&result, UserProfile::default(), true, Theme::Dark);
    }

    #[test]
    fn test_format_verbose_sections_integration() {
        // Exercise the full integration path
        format_verbose_sections(
            &make_test_result(),
            UserProfile::default(),
            false,
            Theme::Dark,
        );
    }

    #[test]
    fn test_format_verbose_sections_empty() {
        // Should not panic with all None values
        let mut result = make_test_result();
        result.ping = None;
        result.jitter = None;
        result.download = None;
        result.upload = None;
        result.download_samples = None;
        result.upload_samples = None;
        result.ping_samples = None;
        format_verbose_sections(&result, UserProfile::default(), false, Theme::Dark);
    }

    #[test]
    fn test_format_data_kb() {
        assert_eq!(crate::common::format_data_size(5120), "5.0 KB");
    }

    #[test]
    fn test_format_data_mb() {
        assert_eq!(crate::common::format_data_size(5_242_880), "5.0 MB");
    }

    #[test]
    fn test_format_data_gb() {
        assert_eq!(crate::common::format_data_size(1_073_741_824), "1.00 GB");
    }
}