From f0f5f084c883dc4f5b6e61603e82cd1c2092fd9d Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Sun, 21 Jun 2026 13:11:49 +0900
Subject: test(tests): expand stub macros into plain test functions

The per-file stub!/encode_stub!/etc. macros generated #[ignore]d test
functions but obscured the individual test bodies. Expanding them inline
removes the macro indirection so future ports can fill in each function
directly.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../shirabe/tests/command/remove_command_test.rs   | 121 ++++++++++++++++-----
 1 file changed, 95 insertions(+), 26 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 4e3497a..297d136 100644
--- a/crates/shirabe/tests/command/remove_command_test.rs
+++ b/crates/shirabe/tests/command/remove_command_test.rs
@@ -1,28 +1,97 @@
 //! ref: composer/tests/Composer/Test/Command/RemoveCommandTest.php
 
-macro_rules! stub {
-    ($name:ident) => {
-        #[test]
-        #[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
-        fn $name() {
-            todo!()
-        }
-    };
-}
-
-stub!(test_exception_running_with_no_remove_packages);
-stub!(test_exception_when_running_unused_without_lock_file);
-stub!(test_warning_when_removing_non_existent_package);
-stub!(test_warning_when_removing_package_from_wrong_type);
-stub!(test_warning_when_removing_package_with_deprecated_dependencies_flag);
-stub!(test_message_output_when_no_unused_packages_to_remove);
-stub!(test_remove_unused_package);
-stub!(test_remove_package_by_name);
-stub!(test_remove_package_by_name_with_dry_run);
-stub!(test_remove_allowed_plugin_package_with_no_other_allowed_plugins);
-stub!(test_remove_allowed_plugin_package_with_other_allowed_plugins);
-stub!(test_remove_packages_by_vendor);
-stub!(test_remove_packages_by_vendor_with_dry_run);
-stub!(test_warning_when_removing_packages_by_vendor_from_wrong_type);
-stub!(test_package_still_present_error_when_no_install_flag_used);
-stub!(test_update_inherited_dependencies_flag_is_passed_to_post_remove_installer);
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_exception_running_with_no_remove_packages() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_exception_when_running_unused_without_lock_file() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_warning_when_removing_non_existent_package() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_warning_when_removing_package_from_wrong_type() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_warning_when_removing_package_with_deprecated_dependencies_flag() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_message_output_when_no_unused_packages_to_remove() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_unused_package() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_package_by_name() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_package_by_name_with_dry_run() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_allowed_plugin_package_with_no_other_allowed_plugins() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_allowed_plugin_package_with_other_allowed_plugins() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_packages_by_vendor() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_packages_by_vendor_with_dry_run() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_warning_when_removing_packages_by_vendor_from_wrong_type() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_package_still_present_error_when_no_install_flag_used() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_update_inherited_dependencies_flag_is_passed_to_post_remove_installer() {
+    todo!()
+}
-- 
cgit v1.3.1