From fb1317b59c65c71498c14ecb351773ac05f139a0 Mon Sep 17 00:00:00 2001
From: Steven Enamakel <enamakel@tinyhumans.ai>
Date: Thu, 1 Oct 2026 22:14:40 +0300
Subject: [PATCH] test: rename legacy test.rs and *_test.rs files to *_tests.rs

Each module keeps its name; its declaration gains a #[path] to the renamed file.
Only names change, no test code does. Drops the legacy-files note from the guidance.

Co-authored-by: Medulla <medulla@tinyhumans.ai>
---
 AGENTS.md                                                      | 3 ---
 crates/tinysearch-bus/src/errors/mod.rs                        | 1 +
 crates/tinysearch-bus/src/errors/{test.rs => mod_tests.rs}     | 0
 crates/tinysearch-bus/src/names/mod.rs                         | 1 +
 crates/tinysearch-bus/src/names/{test.rs => mod_tests.rs}      | 0
 crates/tinysearch-bus/src/search/catalog/mod.rs                | 1 +
 .../src/search/catalog/{test.rs => mod_tests.rs}               | 0
 crates/tinysearch-bus/src/search/mod.rs                        | 1 +
 crates/tinysearch-bus/src/search/{test.rs => mod_tests.rs}     | 0
 crates/tinysearch-bus/src/version/mod.rs                       | 1 +
 crates/tinysearch-bus/src/version/{test.rs => mod_tests.rs}    | 0
 crates/tinysearch/src/error/mod.rs                             | 1 +
 crates/tinysearch/src/error/{test.rs => mod_tests.rs}          | 0
 crates/tinysearch/src/provider/direct/parallel.rs              | 1 +
 .../provider/direct/parallel/{test.rs => parallel_tests.rs}    | 0
 crates/tinysearch/src/provider/direct/tavily.rs                | 1 +
 .../src/provider/direct/tavily/{test.rs => tavily_tests.rs}    | 0
 crates/tinysearch/src/provider/direct/tinyfish.rs              | 1 +
 .../provider/direct/tinyfish/{test.rs => tinyfish_tests.rs}    | 0
 crates/tinysearch/src/provider/mod.rs                          | 1 +
 crates/tinysearch/src/provider/{test.rs => mod_tests.rs}       | 0
 crates/tinysearch/src/search/mod.rs                            | 1 +
 crates/tinysearch/src/search/{test.rs => mod_tests.rs}         | 0
 crates/tinysearch/src/search/roles/mod.rs                      | 1 +
 crates/tinysearch/src/search/roles/{test.rs => mod_tests.rs}   | 0
 crates/tinysearch/src/tinybus_module/mod.rs                    | 1 +
 crates/tinysearch/src/tinybus_module/{test.rs => mod_tests.rs} | 0
 27 files changed, 13 insertions(+), 3 deletions(-)
 rename crates/tinysearch-bus/src/errors/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch-bus/src/names/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch-bus/src/search/catalog/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch-bus/src/search/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch-bus/src/version/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch/src/error/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch/src/provider/direct/parallel/{test.rs => parallel_tests.rs} (100%)
 rename crates/tinysearch/src/provider/direct/tavily/{test.rs => tavily_tests.rs} (100%)
 rename crates/tinysearch/src/provider/direct/tinyfish/{test.rs => tinyfish_tests.rs} (100%)
 rename crates/tinysearch/src/provider/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch/src/search/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch/src/search/roles/{test.rs => mod_tests.rs} (100%)
 rename crates/tinysearch/src/tinybus_module/{test.rs => mod_tests.rs} (100%)

diff --git a/AGENTS.md b/AGENTS.md
index 2efcd73..2677bad 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -46,6 +46,3 @@ PRs go to the canonical upstream repository and should be ready for review.
 - Integration tests stay in the crate's `tests/` directory.
 - OpenHuman's `scripts/externalize-inline-tests.mjs <repo-root> --write` moves
   inline test modules out mechanically; without `--write` it only reports.
-- Existing `test.rs` and `<module>_test.rs` files predate this rule. Rename each
-  to `<module>_tests.rs` (keep its `mod` name, add the `#[path]` attribute) the
-  next time you touch it.
diff --git a/crates/tinysearch-bus/src/errors/mod.rs b/crates/tinysearch-bus/src/errors/mod.rs
index f817ab2..f4de152 100644
--- a/crates/tinysearch-bus/src/errors/mod.rs
+++ b/crates/tinysearch-bus/src/errors/mod.rs
@@ -67,4 +67,5 @@ pub fn code_of(message: &str) -> Option<&'static str> {
 }
 
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch-bus/src/errors/test.rs b/crates/tinysearch-bus/src/errors/mod_tests.rs
similarity index 100%
rename from crates/tinysearch-bus/src/errors/test.rs
rename to crates/tinysearch-bus/src/errors/mod_tests.rs
diff --git a/crates/tinysearch-bus/src/names/mod.rs b/crates/tinysearch-bus/src/names/mod.rs
index 5bcdd8a..2b47cc7 100644
--- a/crates/tinysearch-bus/src/names/mod.rs
+++ b/crates/tinysearch-bus/src/names/mod.rs
@@ -39,4 +39,5 @@ pub fn role_for_tool(name: &str) -> Option<Role> {
 /// Members in interface dispatch order.
 pub const METHODS: &[&str] = &[methods::LIST_TOOLS, methods::EXECUTE_TOOL];
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch-bus/src/names/test.rs b/crates/tinysearch-bus/src/names/mod_tests.rs
similarity index 100%
rename from crates/tinysearch-bus/src/names/test.rs
rename to crates/tinysearch-bus/src/names/mod_tests.rs
diff --git a/crates/tinysearch-bus/src/search/catalog/mod.rs b/crates/tinysearch-bus/src/search/catalog/mod.rs
index 05ce0fc..30fe9d1 100644
--- a/crates/tinysearch-bus/src/search/catalog/mod.rs
+++ b/crates/tinysearch-bus/src/search/catalog/mod.rs
@@ -370,4 +370,5 @@ pub fn select_tools(
 }
 
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch-bus/src/search/catalog/test.rs b/crates/tinysearch-bus/src/search/catalog/mod_tests.rs
similarity index 100%
rename from crates/tinysearch-bus/src/search/catalog/test.rs
rename to crates/tinysearch-bus/src/search/catalog/mod_tests.rs
diff --git a/crates/tinysearch-bus/src/search/mod.rs b/crates/tinysearch-bus/src/search/mod.rs
index 4488bc4..ad4c8dc 100644
--- a/crates/tinysearch-bus/src/search/mod.rs
+++ b/crates/tinysearch-bus/src/search/mod.rs
@@ -8,4 +8,5 @@ pub use catalog::{
 };
 pub use types::*;
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch-bus/src/search/test.rs b/crates/tinysearch-bus/src/search/mod_tests.rs
similarity index 100%
rename from crates/tinysearch-bus/src/search/test.rs
rename to crates/tinysearch-bus/src/search/mod_tests.rs
diff --git a/crates/tinysearch-bus/src/version/mod.rs b/crates/tinysearch-bus/src/version/mod.rs
index 43208f4..3ff8c57 100644
--- a/crates/tinysearch-bus/src/version/mod.rs
+++ b/crates/tinysearch-bus/src/version/mod.rs
@@ -44,4 +44,5 @@ fn binds(host: (u32, u32), module: (u32, u32)) -> bool {
 }
 
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch-bus/src/version/test.rs b/crates/tinysearch-bus/src/version/mod_tests.rs
similarity index 100%
rename from crates/tinysearch-bus/src/version/test.rs
rename to crates/tinysearch-bus/src/version/mod_tests.rs
diff --git a/crates/tinysearch/src/error/mod.rs b/crates/tinysearch/src/error/mod.rs
index ca764c5..37541e4 100644
--- a/crates/tinysearch/src/error/mod.rs
+++ b/crates/tinysearch/src/error/mod.rs
@@ -75,4 +75,5 @@ impl Error {
 /// Standard result type.
 pub type Result<T> = std::result::Result<T, Error>;
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch/src/error/test.rs b/crates/tinysearch/src/error/mod_tests.rs
similarity index 100%
rename from crates/tinysearch/src/error/test.rs
rename to crates/tinysearch/src/error/mod_tests.rs
diff --git a/crates/tinysearch/src/provider/direct/parallel.rs b/crates/tinysearch/src/provider/direct/parallel.rs
index 2ac5822..d43f45c 100644
--- a/crates/tinysearch/src/provider/direct/parallel.rs
+++ b/crates/tinysearch/src/provider/direct/parallel.rs
@@ -319,4 +319,5 @@ async fn async_response(
 }
 
 #[cfg(test)]
+#[path = "parallel/parallel_tests.rs"]
 mod test;
diff --git a/crates/tinysearch/src/provider/direct/parallel/test.rs b/crates/tinysearch/src/provider/direct/parallel/parallel_tests.rs
similarity index 100%
rename from crates/tinysearch/src/provider/direct/parallel/test.rs
rename to crates/tinysearch/src/provider/direct/parallel/parallel_tests.rs
diff --git a/crates/tinysearch/src/provider/direct/tavily.rs b/crates/tinysearch/src/provider/direct/tavily.rs
index 608510a..eb8cd01 100644
--- a/crates/tinysearch/src/provider/direct/tavily.rs
+++ b/crates/tinysearch/src/provider/direct/tavily.rs
@@ -81,4 +81,5 @@ pub(super) fn unwrap(request: &ExecuteToolRequest, mut value: Value) -> Result<V
 }
 
 #[cfg(test)]
+#[path = "tavily/tavily_tests.rs"]
 mod test;
diff --git a/crates/tinysearch/src/provider/direct/tavily/test.rs b/crates/tinysearch/src/provider/direct/tavily/tavily_tests.rs
similarity index 100%
rename from crates/tinysearch/src/provider/direct/tavily/test.rs
rename to crates/tinysearch/src/provider/direct/tavily/tavily_tests.rs
diff --git a/crates/tinysearch/src/provider/direct/tinyfish.rs b/crates/tinysearch/src/provider/direct/tinyfish.rs
index d26afe2..38de8b8 100644
--- a/crates/tinysearch/src/provider/direct/tinyfish.rs
+++ b/crates/tinysearch/src/provider/direct/tinyfish.rs
@@ -106,4 +106,5 @@ pub(super) fn prepare(request: &ExecuteToolRequest, key: &str) -> Result<Prepare
 }
 
 #[cfg(test)]
+#[path = "tinyfish/tinyfish_tests.rs"]
 mod test;
diff --git a/crates/tinysearch/src/provider/direct/tinyfish/test.rs b/crates/tinysearch/src/provider/direct/tinyfish/tinyfish_tests.rs
similarity index 100%
rename from crates/tinysearch/src/provider/direct/tinyfish/test.rs
rename to crates/tinysearch/src/provider/direct/tinyfish/tinyfish_tests.rs
diff --git a/crates/tinysearch/src/provider/mod.rs b/crates/tinysearch/src/provider/mod.rs
index 469a09b..5dba72d 100644
--- a/crates/tinysearch/src/provider/mod.rs
+++ b/crates/tinysearch/src/provider/mod.rs
@@ -405,4 +405,5 @@ mod normalize;
 use normalize::normalize;
 
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch/src/provider/test.rs b/crates/tinysearch/src/provider/mod_tests.rs
similarity index 100%
rename from crates/tinysearch/src/provider/test.rs
rename to crates/tinysearch/src/provider/mod_tests.rs
diff --git a/crates/tinysearch/src/search/mod.rs b/crates/tinysearch/src/search/mod.rs
index 39fd754..f0590a3 100644
--- a/crates/tinysearch/src/search/mod.rs
+++ b/crates/tinysearch/src/search/mod.rs
@@ -316,4 +316,5 @@ fn within_usize_bounds(
 }
 
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch/src/search/test.rs b/crates/tinysearch/src/search/mod_tests.rs
similarity index 100%
rename from crates/tinysearch/src/search/test.rs
rename to crates/tinysearch/src/search/mod_tests.rs
diff --git a/crates/tinysearch/src/search/roles/mod.rs b/crates/tinysearch/src/search/roles/mod.rs
index 3ed8e15..9d8da28 100644
--- a/crates/tinysearch/src/search/roles/mod.rs
+++ b/crates/tinysearch/src/search/roles/mod.rs
@@ -173,6 +173,7 @@ pub(super) fn provider_arguments(
 }
 
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
 
 /// A caller may send `depth: "deep"` from an older declaration (a resumed
diff --git a/crates/tinysearch/src/search/roles/test.rs b/crates/tinysearch/src/search/roles/mod_tests.rs
similarity index 100%
rename from crates/tinysearch/src/search/roles/test.rs
rename to crates/tinysearch/src/search/roles/mod_tests.rs
diff --git a/crates/tinysearch/src/tinybus_module/mod.rs b/crates/tinysearch/src/tinybus_module/mod.rs
index 4ac3dd8..9535344 100644
--- a/crates/tinysearch/src/tinybus_module/mod.rs
+++ b/crates/tinysearch/src/tinybus_module/mod.rs
@@ -49,4 +49,5 @@ tinybus_module::module_export! {
 }
 
 #[cfg(test)]
+#[path = "mod_tests.rs"]
 mod test;
diff --git a/crates/tinysearch/src/tinybus_module/test.rs b/crates/tinysearch/src/tinybus_module/mod_tests.rs
similarity index 100%
rename from crates/tinysearch/src/tinybus_module/test.rs
rename to crates/tinysearch/src/tinybus_module/mod_tests.rs