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>

1 files changed, 133 insertions, 26 deletions

diff --git a/crates/shirabe/tests/downloader/git_downloader_test.rs b/crates/shirabe/tests/downloader/git_downloader_test.rs
index ad5e305..c820254 100644
--- a/crates/shirabe/tests/downloader/git_downloader_test.rs
+++ b/crates/shirabe/tests/downloader/git_downloader_test.rs
@@ -44,31 +44,138 @@ impl Drop for TearDown {
 // These construct a GitDownloader with a mocked IO/Config and a mocked ProcessExecutor to
 // feed git command output; mocking is not available, and a real HttpDownloader reaches
 // curl_multi_init (todo!()).
-macro_rules! stub {
-    ($name:ident) => {
-        #[test]
-        #[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
-        fn $name() {
-            let working_dir = set_up();
-            let _tear_down = TearDown::new(working_dir.path().to_path_buf());
-            let _ = &working_dir;
-            todo!()
-        }
-    };
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_download_for_package_without_source_reference() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_download() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_download_with_cache() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_download_uses_various_protocols_and_sets_push_url_for_github() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_download_and_set_push_url_use_custom_various_protocols_for_github() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_download_throws_runtime_exception_if_git_command_fails() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_updatefor_package_without_source_reference() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_update() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_update_with_new_repo_url() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
 }
 
-stub!(test_download_for_package_without_source_reference);
-stub!(test_download);
-stub!(test_download_with_cache);
-stub!(test_download_uses_various_protocols_and_sets_push_url_for_github);
-stub!(test_download_and_set_push_url_use_custom_various_protocols_for_github);
-stub!(test_download_throws_runtime_exception_if_git_command_fails);
-stub!(test_updatefor_package_without_source_reference);
-stub!(test_update);
-stub!(test_update_with_new_repo_url);
-stub!(test_update_throws_runtime_exception_if_git_command_fails);
-stub!(test_update_doesnt_throws_runtime_exception_if_git_command_fails_at_first_but_is_able_to_recover);
-stub!(test_downgrade_shows_appropriate_message);
-stub!(test_not_using_downgrading_with_references);
-stub!(test_remove);
-stub!(test_get_installation_source);
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_update_throws_runtime_exception_if_git_command_fails() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_update_doesnt_throws_runtime_exception_if_git_command_fails_at_first_but_is_able_to_recover()
+ {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_downgrade_shows_appropriate_message() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_not_using_downgrading_with_references() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_remove() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks ProcessExecutor/IO and needs an HttpDownloader (curl_multi_init todo!())"]
+fn test_get_installation_source() {
+    let working_dir = set_up();
+    let _tear_down = TearDown::new(working_dir.path().to_path_buf());
+    let _ = &working_dir;
+    todo!()
+}