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>

2 files changed, 164 insertions, 46 deletions

diff --git a/crates/shirabe/tests/package/version/version_guesser_test.rs b/crates/shirabe/tests/package/version/version_guesser_test.rs
index 9b974dd..75d4ae0 100644
--- a/crates/shirabe/tests/package/version/version_guesser_test.rs
+++ b/crates/shirabe/tests/package/version/version_guesser_test.rs
@@ -23,30 +23,94 @@ impl Drop for TearDown {
 
 // These drive VersionGuesser with a mocked ProcessExecutor feeding git/hg command output;
 // mocking is not available here.
-macro_rules! stub {
-    ($name:ident) => {
-        #[test]
-        #[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
-        fn $name() {
-            todo!()
-        }
-    };
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_hg_guess_version_returns_data() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_guess_version_returns_data() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_guess_version_does_not_see_custom_default_branch_as_non_feature_branch() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_guess_version_reads_and_respects_non_feature_branches_configuration_for_arbitrary_naming() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_guess_version_reads_and_respects_non_feature_branches_configuration_for_arbitrary_naming_regex()
+ {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_guess_version_reads_and_respects_non_feature_branches_configuration_for_arbitrary_naming_when_on_non_feature_branch()
+ {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_detached_head_becomes_dev_hash() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_detached_fetch_head_becomes_dev_hash_git2() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_detached_commit_head_becomes_dev_hash_git2() {
+    todo!()
 }
 
-stub!(test_hg_guess_version_returns_data);
-stub!(test_guess_version_returns_data);
-stub!(test_guess_version_does_not_see_custom_default_branch_as_non_feature_branch);
-stub!(
-    test_guess_version_reads_and_respects_non_feature_branches_configuration_for_arbitrary_naming
-);
-stub!(test_guess_version_reads_and_respects_non_feature_branches_configuration_for_arbitrary_naming_regex);
-stub!(test_guess_version_reads_and_respects_non_feature_branches_configuration_for_arbitrary_naming_when_on_non_feature_branch);
-stub!(test_detached_head_becomes_dev_hash);
-stub!(test_detached_fetch_head_becomes_dev_hash_git2);
-stub!(test_detached_commit_head_becomes_dev_hash_git2);
-stub!(test_tag_becomes_version);
-stub!(test_tag_becomes_pretty_version);
-stub!(test_invalid_tag_becomes_version);
-stub!(test_numeric_branches_show_nicely);
-stub!(test_remote_branches_are_selected);
-stub!(test_get_root_version_from_env);
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_tag_becomes_version() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_tag_becomes_pretty_version() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_invalid_tag_becomes_version() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_numeric_branches_show_nicely() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_remote_branches_are_selected() {
+    todo!()
+}
+
+#[test]
+#[ignore = "mocks a ProcessExecutor feeding git/hg output to drive VersionGuesser; mocking is not available"]
+fn test_get_root_version_from_env() {
+    todo!()
+}
diff --git a/crates/shirabe/tests/package/version/version_selector_test.rs b/crates/shirabe/tests/package/version/version_selector_test.rs
index ccd64ad..fd426ce 100644
--- a/crates/shirabe/tests/package/version/version_selector_test.rs
+++ b/crates/shirabe/tests/package/version/version_selector_test.rs
@@ -2,26 +2,80 @@
 
 // VersionSelector ranks candidate packages whose versions/constraints are parsed through a
 // look-around regex the regex crate cannot compile; the setup also mocks a repository.
-macro_rules! stub {
-    ($name:ident) => {
-        #[test]
-        #[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
-        fn $name() {
-            todo!()
-        }
-    };
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_latest_version_is_returned() {
+    todo!()
 }
 
-stub!(test_latest_version_is_returned);
-stub!(test_latest_version_is_returned_that_matches_php_requirements);
-stub!(test_latest_version_is_returned_that_matches_ext_requirements);
-stub!(test_latest_version_is_returned_that_matches_platform_ext);
-stub!(test_latest_version_is_returned_that_matches_composer_requirements);
-stub!(test_most_stable_version_is_returned);
-stub!(test_most_stable_version_is_returned_regardless_of_order);
-stub!(test_highest_version_is_returned);
-stub!(test_highest_version_matching_stability_is_returned);
-stub!(test_most_stable_unstable_version_is_returned);
-stub!(test_default_branch_alias_is_never_returned);
-stub!(test_false_returned_on_no_packages);
-stub!(test_find_recommended_require_version);
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_latest_version_is_returned_that_matches_php_requirements() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_latest_version_is_returned_that_matches_ext_requirements() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_latest_version_is_returned_that_matches_platform_ext() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_latest_version_is_returned_that_matches_composer_requirements() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_most_stable_version_is_returned() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_most_stable_version_is_returned_regardless_of_order() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_highest_version_is_returned() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_highest_version_matching_stability_is_returned() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_most_stable_unstable_version_is_returned() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_default_branch_alias_is_never_returned() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_false_returned_on_no_packages() {
+    todo!()
+}
+
+#[test]
+#[ignore = "not yet ported (VersionSelector over a mocked repository; constraint parsing uses a look-around regex)"]
+fn test_find_recommended_require_version() {
+    todo!()
+}