Move shared_helpers test to a dedicated module

Author: Kobzol

src/bootstrap/src/utils/shared_helpers.rs

@@ -6,6 +6,9 @@
 
 #![allow(dead_code)]
 
+#[cfg(test)]
+mod tests;
+
 use std::env;
 use std::ffi::OsString;
 use std::fs::OpenOptions;

src/bootstrap/src/utils/tests/shared_helpers_tests.rs renamed to src/bootstrap/src/utils/shared_helpers/tests.rs

@@ -1,10 +1,3 @@
-//! The `shared_helpers` module can't have its own tests submodule, because
-//! that would cause problems for the shim binaries that include it via
-//! `#[path]`, so instead those unit tests live here.
-//!
-//! To prevent tidy from complaining about this file not being named `tests.rs`,
-//! it lives inside a submodule directory named `tests`.
-
 use crate::utils::shared_helpers::parse_value_from_args;
 
 #[test]

src/bootstrap/src/utils/tests/mod.rs

@@ -1,2 +1 @@
 pub mod git;
-mod shared_helpers_tests;