From 37ed5b8c6d4cda30e668d0221eb281431dbc8c67 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Mon, 20 Jul 2026 16:04:45 +0900 Subject: test(ignore): document root causes for unannotated #[ignore] tests 74 tests carried a bare #[ignore] with no explanation. Re-ran each: 25 now pass and had the attribute removed; the remaining 49 got a concise reason (todo!() stubs, regex-crate PCRE gaps, PhpMixed type mismatches, config bool-coercion bugs, missing skipped_load wiring in PoolBuilder, etc.) so future work can find and fix them by grep. No production code or test logic/assertions were changed. Co-Authored-By: Claude Sonnet 5 --- crates/shirabe/tests/util/ini_helper_test.rs | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'crates/shirabe/tests/util/ini_helper_test.rs') diff --git a/crates/shirabe/tests/util/ini_helper_test.rs b/crates/shirabe/tests/util/ini_helper_test.rs index d1e2071e..76705a09 100644 --- a/crates/shirabe/tests/util/ini_helper_test.rs +++ b/crates/shirabe/tests/util/ini_helper_test.rs @@ -43,7 +43,6 @@ fn set_env(paths: &[&str]) { } #[test] -#[ignore] fn test_with_no_ini() { let paths = [""]; @@ -56,7 +55,8 @@ fn test_with_no_ini() { } #[test] -#[ignore] +#[ignore = "XdebugHandler::get_all_ini_files() is stubbed to always return [\"\"], ignoring \ +COMPOSER_ORIGINAL_INIS entirely"] fn test_with_loaded_ini_only() { let paths = ["loaded.ini"]; @@ -65,7 +65,8 @@ fn test_with_loaded_ini_only() { } #[test] -#[ignore] +#[ignore = "XdebugHandler::get_all_ini_files() is stubbed to always return [\"\"], ignoring \ +COMPOSER_ORIGINAL_INIS entirely"] fn test_with_loaded_ini_and_additional() { let paths = ["loaded.ini", "one.ini", "two.ini"]; @@ -78,7 +79,8 @@ fn test_with_loaded_ini_and_additional() { } #[test] -#[ignore] +#[ignore = "XdebugHandler::get_all_ini_files() is stubbed to always return [\"\"], ignoring \ +COMPOSER_ORIGINAL_INIS entirely"] fn test_without_loaded_ini_and_additional() { let paths = ["", "one.ini", "two.ini"]; -- cgit v1.3.1