fix(test): Move 'cargo_home' from 'install' to 'paths'

This is used outside of `cargo install` contexts and this makes it more
discoverable for those use cases.

Author: epage

crates/cargo-test-support/src/install.rs

@@ -1,14 +1,13 @@
-use crate::paths;
 use std::env::consts::EXE_SUFFIX;
-use std::path::{Path, PathBuf};
+use std::path::Path;
 
 /// Used by `cargo install` tests to assert an executable binary
 /// has been installed. Example usage:
 /// ```no_run
 /// use cargo_test_support::install::assert_has_installed_exe;
-/// use cargo_test_support::install::cargo_home;
+/// use cargo_test_support::paths;
 ///
-/// assert_has_installed_exe(cargo_home(), "foo");
+/// assert_has_installed_exe(paths::cargo_home(), "foo");
 /// ```
 #[track_caller]
 pub fn assert_has_installed_exe<P: AsRef<Path>>(path: P, name: &'static str) {
@@ -24,10 +23,6 @@ fn check_has_installed_exe<P: AsRef<Path>>(path: P, name: &'static str) -> bool
     path.as_ref().join("bin").join(exe(name)).is_file()
 }
 
-pub fn cargo_home() -> PathBuf {
-    paths::home().join(".cargo")
-}
-
 pub fn exe(name: &str) -> String {
     format!("{}{}", name, EXE_SUFFIX)
 }

crates/cargo-test-support/src/paths.rs

@@ -110,6 +110,10 @@ pub fn home() -> PathBuf {
     path
 }
 
+pub fn cargo_home() -> PathBuf {
+    home().join(".cargo")
+}
+
 pub trait CargoPathExt {
     fn to_url(&self) -> url::Url;
 

tests/testsuite/binary_name.rs

@@ -2,7 +2,7 @@
 
 use cargo_test_support::install::assert_has_installed_exe;
 use cargo_test_support::install::assert_has_not_installed_exe;
-use cargo_test_support::install::cargo_home;
+use cargo_test_support::paths;
 use cargo_test_support::prelude::*;
 use cargo_test_support::project;
 use cargo_test_support::str;
@@ -221,7 +221,7 @@ Hello, crabs!
         .masquerade_as_nightly_cargo(&["different-binary-name"])
         .run();
 
-    assert_has_installed_exe(cargo_home(), "007bar");
+    assert_has_installed_exe(paths::cargo_home(), "007bar");
 
     p.cargo("uninstall")
         .with_stderr_data(str![[r#"
@@ -231,7 +231,7 @@ Hello, crabs!
         .masquerade_as_nightly_cargo(&["different-binary-name"])
         .run();
 
-    assert_has_not_installed_exe(cargo_home(), "007bar");
+    assert_has_not_installed_exe(paths::cargo_home(), "007bar");
 }
 
 #[cargo_test]

tests/testsuite/build_script.rs

@@ -6,7 +6,7 @@ use std::io;
 use std::thread;
 
 use cargo_test_support::compare::assert_e2e;
-use cargo_test_support::install::cargo_home;
+use cargo_test_support::paths::cargo_home;
 use cargo_test_support::prelude::*;
 use cargo_test_support::registry::Package;
 use cargo_test_support::str;

tests/testsuite/concurrent.rs

@@ -9,7 +9,8 @@ use std::{env, str};
 
 use cargo_test_support::cargo_process;
 use cargo_test_support::git;
-use cargo_test_support::install::{assert_has_installed_exe, cargo_home};
+use cargo_test_support::install::assert_has_installed_exe;
+use cargo_test_support::paths;
 use cargo_test_support::prelude::*;
 use cargo_test_support::registry::Package;
 use cargo_test_support::str;
@@ -46,8 +47,8 @@ fn multiple_installs() {
     execs().run_output(&a);
     execs().run_output(&b);
 
-    assert_has_installed_exe(cargo_home(), "foo");
-    assert_has_installed_exe(cargo_home(), "bar");
+    assert_has_installed_exe(paths::cargo_home(), "foo");
+    assert_has_installed_exe(paths::cargo_home(), "bar");
 }
 
 #[cargo_test]
@@ -75,8 +76,8 @@ fn concurrent_installs() {
     execs().run_output(&a);
     execs().run_output(&b);
 
-    assert_has_installed_exe(cargo_home(), "foo");
-    assert_has_installed_exe(cargo_home(), "bar");
+    assert_has_installed_exe(paths::cargo_home(), "foo");
+    assert_has_installed_exe(paths::cargo_home(), "bar");
 }
 
 #[cargo_test]
@@ -104,7 +105,7 @@ fn one_install_should_be_bad() {
     execs().run_output(&a);
     execs().run_output(&b);
 
-    assert_has_installed_exe(cargo_home(), "foo");
+    assert_has_installed_exe(paths::cargo_home(), "foo");
 }
 
 #[cargo_test]

tests/testsuite/features2.rs

@@ -3,7 +3,7 @@
 use std::fs::File;
 
 use cargo_test_support::cross_compile::{self, alternate};
-use cargo_test_support::install::cargo_home;
+use cargo_test_support::paths;
 use cargo_test_support::prelude::*;
 use cargo_test_support::publish::validate_crate_contents;
 use cargo_test_support::registry::{Dependency, Package};
@@ -2208,8 +2208,8 @@ fn minimal_download() {
         .build();
 
     let clear = || {
-        cargo_home().join("registry/cache").rm_rf();
-        cargo_home().join("registry/src").rm_rf();
+        paths::cargo_home().join("registry/cache").rm_rf();
+        paths::cargo_home().join("registry/src").rm_rf();
         p.build_dir().rm_rf();
     };