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/command/remove_command_test.rs | 9 --------- 1 file changed, 9 deletions(-) (limited to 'crates/shirabe/tests/command/remove_command_test.rs') diff --git a/crates/shirabe/tests/command/remove_command_test.rs b/crates/shirabe/tests/command/remove_command_test.rs index 75ef5075..252496ea 100644 --- a/crates/shirabe/tests/command/remove_command_test.rs +++ b/crates/shirabe/tests/command/remove_command_test.rs @@ -86,7 +86,6 @@ fn test_exception_when_running_unused_without_lock_file() { #[test] #[serial] -#[ignore] fn test_warning_when_removing_non_existent_package() { let tear_down = init_temp_composer(None, None, None, true); create_installed_json(&[], &[], true); @@ -259,7 +258,6 @@ fn test_message_output_when_no_unused_packages_to_remove() { #[test] #[serial] -#[ignore] fn test_remove_unused_package() { let tear_down = init_temp_composer( Some(&serde_json::json!({ @@ -325,7 +323,6 @@ fn test_remove_unused_package() { #[test] #[serial] -#[ignore] fn test_remove_package_by_name() { let tear_down = init_temp_composer( Some(&serde_json::json!({ @@ -410,7 +407,6 @@ fn test_remove_package_by_name() { #[test] #[serial] -#[ignore] fn test_remove_package_by_name_with_dry_run() { let tear_down = init_temp_composer( Some(&serde_json::json!({ @@ -497,7 +493,6 @@ fn test_remove_package_by_name_with_dry_run() { #[test] #[serial] -#[ignore] fn test_remove_allowed_plugin_package_with_no_other_allowed_plugins() { let tear_down = init_temp_composer( Some(&serde_json::json!({ @@ -555,7 +550,6 @@ fn test_remove_allowed_plugin_package_with_no_other_allowed_plugins() { #[test] #[serial] -#[ignore] fn test_remove_allowed_plugin_package_with_other_allowed_plugins() { let tear_down = init_temp_composer( Some(&serde_json::json!({ @@ -607,7 +601,6 @@ fn test_remove_allowed_plugin_package_with_other_allowed_plugins() { #[test] #[serial] -#[ignore] fn test_remove_packages_by_vendor() { let tear_down = init_temp_composer( Some(&serde_json::json!({ @@ -691,7 +684,6 @@ fn test_remove_packages_by_vendor() { #[test] #[serial] -#[ignore] fn test_remove_packages_by_vendor_with_dry_run() { let tear_down = init_temp_composer( Some(&serde_json::json!({ @@ -969,7 +961,6 @@ fn run_update_inherited_dependencies_flag_case( #[test] #[serial] -#[ignore] fn test_update_inherited_dependencies_flag_is_passed_to_post_remove_installer() { // 'update with all dependencies' run_update_inherited_dependencies_flag_case( -- cgit v1.3.1