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>
---
 .../tests/command/repository_command_test.rs       | 121 ++++++++++++++++-----
 1 file changed, 95 insertions(+), 26 deletions(-)

(limited to 'crates/shirabe/tests/command/repository_command_test.rs')

diff --git a/crates/shirabe/tests/command/repository_command_test.rs b/crates/shirabe/tests/command/repository_command_test.rs
index 7611785..33a4451 100644
--- a/crates/shirabe/tests/command/repository_command_test.rs
+++ b/crates/shirabe/tests/command/repository_command_test.rs
@@ -1,28 +1,97 @@
 //! ref: composer/tests/Composer/Test/Command/RepositoryCommandTest.php
 
-macro_rules! stub {
-    ($name:ident) => {
-        #[test]
-        #[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
-        fn $name() {
-            todo!()
-        }
-    };
-}
-
-stub!(test_list_with_no_repositories);
-stub!(test_list_with_repositories_as_list);
-stub!(test_list_with_repositories_as_assoc);
-stub!(test_add_repository_with_type_and_url);
-stub!(test_add_repository_with_json);
-stub!(test_remove_repository);
-stub!(test_set_and_get_url_in_repository_assoc);
-stub!(test_set_and_get_url_in_repository_list);
-stub!(test_disable_and_enable_packagist);
-stub!(test_invalid_arg_combination_throws);
-stub!(test_prepend_repository_by_name_list_to_assoc);
-stub!(test_append_repository_by_name_list_to_assoc);
-stub!(test_prepend_repository_assoc_with_packagist_disabled);
-stub!(test_append_repository_assoc_with_packagist_disabled);
-stub!(test_add_before_and_after_by_name);
-stub!(test_add_same_name_replaces_existing);
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_list_with_no_repositories() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_list_with_repositories_as_list() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_list_with_repositories_as_assoc() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_add_repository_with_type_and_url() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_add_repository_with_json() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_remove_repository() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_set_and_get_url_in_repository_assoc() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_set_and_get_url_in_repository_list() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_disable_and_enable_packagist() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_invalid_arg_combination_throws() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_prepend_repository_by_name_list_to_assoc() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_append_repository_by_name_list_to_assoc() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_prepend_repository_assoc_with_packagist_disabled() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_append_repository_assoc_with_packagist_disabled() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_add_before_and_after_by_name() {
+    todo!()
+}
+
+#[test]
+#[ignore = "requires the ApplicationTester/initTempComposer harness, which is not yet ported"]
+fn test_add_same_name_replaces_existing() {
+    todo!()
+}
-- 
cgit v1.3.1