From 0bf96f5de11555ddfdcde81c0a27869afeaf5832 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 28 Jun 2026 07:16:42 +0900 Subject: test(console): correct inaccurate test skip reasons setCatchErrors is an old-Symfony compatibility shim that exists in neither Composer's PHP class nor Shirabe, and setAutoExit(false) is the default in Shirabe, so neither is a reason a test is unportable. Drop those mentions and point the application_test ignores at the real blocker: do_run's script-command registration is still a todo!(). Co-Authored-By: Claude Opus 4.8 (1M context) --- crates/shirabe/tests/application_test.rs | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'crates/shirabe/tests/application_test.rs') diff --git a/crates/shirabe/tests/application_test.rs b/crates/shirabe/tests/application_test.rs index 5a34316..d54f43b 100644 --- a/crates/shirabe/tests/application_test.rs +++ b/crates/shirabe/tests/application_test.rs @@ -1,8 +1,9 @@ //! ref: composer/tests/Composer/Test/ApplicationTest.php // These drive the console Application (doRun, command resolution, plugin disabling). -// The tests needing Application::setAutoExit/setCatchErrors or the initTempComposer -// helper, or a runtime define() of COMPOSER_DEV_WARNING_TIME, remain unportable. +// The tests exercising do_run's script-command registration (a todo!() pending the +// Symfony command-registry model), or a runtime define() of COMPOSER_DEV_WARNING_TIME, +// remain unportable. use std::cell::RefCell; use std::rc::Rc; @@ -108,7 +109,7 @@ fn test_process_isolation_works_multiple_times() { assert_eq!(0, application.do_run(input2, output2).unwrap()); } -#[ignore = "Application::set_auto_exit / set_catch_errors and the initTempComposer test helper do not exist"] +#[ignore = "do_run's script-command registration is a todo!() pending the Symfony command-registry model"] #[test] fn test_no_plugins_disables_plugins_when_script_commands_exist() { let _tear_down = TearDown; @@ -117,7 +118,7 @@ fn test_no_plugins_disables_plugins_when_script_commands_exist() { todo!() } -#[ignore = "Application::set_auto_exit / set_catch_errors and the initTempComposer test helper do not exist"] +#[ignore = "do_run's script-command registration is a todo!() pending the Symfony command-registry model"] #[test] fn test_script_command_takes_priority_over_abbreviated_builtin_command() { let _tear_down = TearDown; -- cgit v1.3.1