From 5186c79b12ea587b4471797af180cf3fd2652aa6 Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Sun, 9 Aug 2026 10:05:06 +0900
Subject: test(cli): drop the CLI smoke tests

These smoke tests only checked that `<cmd> --help` exited cleanly and
that running each command did not panic. The Composer-derived test
suite now covers the same commands with real assertions, so the smoke
tests no longer add anything.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 crates/shirabe/src/lib.rs | 163 ----------------------------------------------
 1 file changed, 163 deletions(-)

diff --git a/crates/shirabe/src/lib.rs b/crates/shirabe/src/lib.rs
index e5624988..afcc0cee 100644
--- a/crates/shirabe/src/lib.rs
+++ b/crates/shirabe/src/lib.rs
@@ -46,166 +46,3 @@ pub fn run(argv: Vec<String>) -> anyhow::Result<i32> {
     let input = std::rc::Rc::new(std::cell::RefCell::new(ArgvInput::new(Some(argv), None)?));
     application.run(Some(input), None)
 }
-
-#[cfg(test)]
-mod cli_tests {
-    use serial_test::serial;
-    use std::panic::{AssertUnwindSafe, catch_unwind};
-    use std::sync::Once;
-
-    const COMMANDS: &[&str] = &[
-        "about",
-        "archive",
-        "audit",
-        "browse",
-        "bump",
-        "check-platform-reqs",
-        "clear-cache",
-        "config",
-        "create-project",
-        "depends",
-        "diagnose",
-        "dump-autoload",
-        "exec",
-        "fund",
-        "global",
-        "init",
-        "install",
-        "licenses",
-        "outdated",
-        "prohibits",
-        "reinstall",
-        "remove",
-        "repository",
-        "require",
-        "run-script",
-        "search",
-        "self-update",
-        "show",
-        "status",
-        "suggests",
-        "update",
-        "validate",
-    ];
-
-    static QUIET_PANIC: Once = Once::new();
-
-    /// Runs the CLI with `args`. Returns true on clean exit, false on any panic / error / non-zero
-    /// exit.
-    fn run(args: &[&str]) -> bool {
-        QUIET_PANIC.call_once(|| std::panic::set_hook(Box::new(|_| {})));
-
-        // Each invocation must look like a fresh process.
-        //
-        // SAFETY: every test reaching this code is marked `#[serial]`, so no other thread reads or
-        // writes the environment concurrently with these calls.
-        unsafe {
-            std::env::remove_var("COLUMNS");
-            std::env::remove_var("LINES");
-        }
-
-        let mut argv = vec!["composer".to_string()];
-        argv.extend(args.iter().map(|s| s.to_string()));
-        matches!(
-            catch_unwind(AssertUnwindSafe(|| crate::run(argv))),
-            Ok(Ok(0))
-        )
-    }
-
-    #[test]
-    #[serial]
-    fn version_flag() {
-        assert!(run(&["--version"]));
-    }
-
-    #[test]
-    #[serial]
-    fn help_flag() {
-        assert!(run(&["--help"]));
-    }
-
-    #[test]
-    #[serial]
-    fn each_command_help() {
-        let failed: Vec<&&str> = COMMANDS.iter().filter(|c| !run(&[c, "--help"])).collect();
-        assert!(failed.is_empty(), "`<cmd> --help` failed for: {failed:?}");
-    }
-
-    /// Runs the CLI with `args` from inside an empty temporary directory. Returns true if the call did
-    /// not panic (any exit code, including non-zero or an `Err` return, counts as success).
-    fn run_no_panic(args: &[&str]) -> bool {
-        QUIET_PANIC.call_once(|| std::panic::set_hook(Box::new(|_| {})));
-
-        let original = std::env::current_dir().ok();
-        let dir = tempfile::tempdir().expect("create temp dir");
-        std::env::set_current_dir(dir.path()).expect("chdir to temp dir");
-
-        // SAFETY: every test reaching this code is marked `#[serial]`, so no other thread touches
-        // the environment or working directory concurrently.
-        unsafe {
-            std::env::remove_var("COLUMNS");
-            std::env::remove_var("LINES");
-        }
-
-        // Force non-interactive: the suite must not block on a prompt (e.g. `init`) when run under
-        // a TTY, where Composer's own logic would otherwise keep interaction enabled.
-        let mut argv = vec!["composer".to_string()];
-        argv.extend(args.iter().map(|s| s.to_string()));
-        argv.push("--no-interaction".to_string());
-        let result = catch_unwind(AssertUnwindSafe(|| crate::run(argv)));
-
-        if let Some(orig) = original {
-            let _ = std::env::set_current_dir(orig);
-        }
-
-        result.is_ok()
-    }
-
-    macro_rules! run_no_panic_tests {
-        ($( $(#[$attr:meta])* $name:ident => $cmd:expr ),* $(,)?) => {
-            $(
-                $(#[$attr])*
-                #[test]
-                #[serial]
-                fn $name() {
-                    assert!(run_no_panic(&[$cmd]), "`{}` panicked", $cmd);
-                }
-            )*
-        };
-    }
-
-    run_no_panic_tests! {
-        run_about => "about",
-        run_archive => "archive",
-        run_audit => "audit",
-        run_browse => "browse",
-        run_bump => "bump",
-        run_check_platform_reqs => "check-platform-reqs",
-        run_clear_cache => "clear-cache",
-        run_config => "config",
-        run_create_project => "create-project",
-        run_depends => "depends",
-        run_diagnose => "diagnose",
-        run_dump_autoload => "dump-autoload",
-        run_exec => "exec",
-        run_fund => "fund",
-        run_global => "global",
-        run_init => "init",
-        run_install => "install",
-        run_licenses => "licenses",
-        run_outdated => "outdated",
-        run_prohibits => "prohibits",
-        run_reinstall => "reinstall",
-        run_remove => "remove",
-        run_repository => "repository",
-        run_require => "require",
-        run_run_script => "run-script",
-        run_search => "search",
-        run_self_update => "self-update",
-        run_show => "show",
-        run_status => "status",
-        run_suggests => "suggests",
-        run_update => "update",
-        run_validate => "validate",
-    }
-}
-- 
cgit v1.3.1-4-g156e