diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-28 07:16:42 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-28 07:20:32 +0900 |
| commit | 0bf96f5de11555ddfdcde81c0a27869afeaf5832 (patch) | |
| tree | f44aa2e365dec17b532b784667ba836d734857e8 /crates/shirabe/tests/application_test.rs | |
| parent | f74d43409220ac209d01367bfc40dbc2c85be252 (diff) | |
| download | php-shirabe-0bf96f5de11555ddfdcde81c0a27869afeaf5832.tar.gz php-shirabe-0bf96f5de11555ddfdcde81c0a27869afeaf5832.tar.zst php-shirabe-0bf96f5de11555ddfdcde81c0a27869afeaf5832.zip | |
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) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/tests/application_test.rs')
| -rw-r--r-- | crates/shirabe/tests/application_test.rs | 9 |
1 files changed, 5 insertions, 4 deletions
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; |
