tempdir lives long enough for sure.

Svetlin Stefanov committed 4 years ago

Commit a41002f1 parent 8de6b3d1 file · +31 −17Browse files

Unified · Split

git-config/tests/file/from_paths/includes/conditional/gitdir.rs+31−17

     );
 }
-fn git_dir(root_dir: &TempDir, git_dir_suffix: &str) -> Box<Path> {
+fn git_dir(root_dir: &Path, git_dir_suffix: &str) -> PathBuf {
-    let git_dir = root_dir.path().join(git_dir_suffix).join(".git");
+    let git_dir = root_dir.join(git_dir_suffix).join(".git");
-    let git_dir = git_dir.as_path();
+    fs::create_dir_all(&git_dir).unwrap();
-    fs::create_dir_all(git_dir).unwrap();
+    git_dir
-    git_dir.into()
 }
 fn write_config(condition: &str, git_dir: &Path, overwrite_config_location: &ConfigLocation) {
 #[test]
 #[serial]
 fn relative_path_with_trailing_slash() {
+    let temp_dir = tempdir().unwrap();
+    let git_dir = git_dir(temp_dir.path(), "foo");
     assert_section_value(Options {
         condition: "gitdir:foo/",
-        git_dir: &git_dir(&tempdir().unwrap(), "foo"),
+        git_dir: &git_dir,
         ..Default::default()
     });
 }
         .as_os_str()
         .to_str()
         .unwrap();
+    let git_dir = git_dir(tmp_dir.path(), "foo");
     assert_section_value(Options {
         condition: &format!("gitdir:~/{}/foo/", root),
-        git_dir: &git_dir(&tmp_dir, "foo"),
+        git_dir: &git_dir,
         ..Default::default()
     });
 }
 #[test]
 #[serial]
 fn star_star_prefix_and_suffix() {
+    let temp_dir = tempdir().unwrap();
+    let git_dir = git_dir(temp_dir.path(), "foo");
     assert_section_value(Options {
         condition: "gitdir:**/foo/**",
-        git_dir: &git_dir(&tempdir().unwrap(), "foo"),
+        git_dir: &git_dir,
         ..Default::default()
     });
 }
 #[test]
 #[serial]
 fn dot_path_slash() {
+    let temp_dir = tempdir().unwrap();
+    let git_dir = git_dir(temp_dir.path(), "foo");
     assert_section_value(Options {
         condition: "gitdir:./",
-        git_dir: &git_dir(&tempdir().unwrap(), "foo"),
+        git_dir: &git_dir,
         config_location: ConfigLocation::User,
         ..Default::default()
     });
 #[test]
 #[serial]
 fn dot_path() {
+    let temp_dir = tempdir().unwrap();
+    let git_dir = git_dir(temp_dir.path(), "foo");
     assert_section_value(Options {
         condition: "gitdir:./foo/.git",
-        git_dir: &git_dir(&tempdir().unwrap(), "foo"),
+        git_dir: &git_dir,
         config_location: ConfigLocation::User,
         ..Default::default()
     });
 #[test]
 #[serial]
 fn case_insensitive() {
+    let temp_dir = tempdir().unwrap();
+    let git_dir = git_dir(temp_dir.path(), "foo");
     assert_section_value(Options {
         condition: "gitdir/i:FOO/",
-        git_dir: &git_dir(&tempdir().unwrap(), "foo"),
+        git_dir: &git_dir,
         ..Default::default()
     });
 }
 #[serial]
 #[ignore]
 fn pattern_with_backslash() {
+    let temp_dir = tempdir().unwrap();
+    let git_dir = git_dir(temp_dir.path(), "foo");
     assert_section_value(Options {
         condition: "gitdir:\\foo/",
-        git_dir: &git_dir(&tempdir().unwrap(), "foo"),
+        git_dir: &git_dir,
         expected: Value::Original,
         ..Default::default()
     });
 #[test]
 #[serial]
 fn star_star_in_the_middle() {
+    let temp_dir = tempdir().unwrap();
+    let git_dir = git_dir(temp_dir.path(), "foo/bar");
     assert_section_value(Options {
         condition: "gitdir:**/foo/**/bar/**",
-        git_dir: &git_dir(&tempdir().unwrap(), "foo/bar"),
+        git_dir: &git_dir,
         ..Default::default()
     });
 }
         .to_str()
         .unwrap();
-    git_dir(&tmp_dir, "foo");
+    let _ = git_dir(tmp_dir.path(), "foo");
     create_symlink(
         tmp_dir.path().join("bar").as_path(),
         tmp_dir.path().join("foo").as_path(),
 #[cfg(not(target_os = "windows"))]
 fn dot_path_with_symlink() {
     let tmp_dir = tempdir_in(home()).unwrap();
-    git_dir(&tmp_dir, "foo");
+    git_dir(tmp_dir.path(), "foo");
     create_symlink(
         tmp_dir.path().join("bar").as_path(),
         tmp_dir.path().join("foo").as_path(),
 #[cfg(not(target_os = "windows"))]
 fn dot_path_matching_symlink() {
     let tmp_dir = tempdir_in(home()).unwrap();
-    git_dir(&tmp_dir, "foo");
+    git_dir(tmp_dir.path(), "foo");
     create_symlink(
         tmp_dir.path().join("bar").as_path(),
         tmp_dir.path().join("foo").as_path(),
 #[cfg(not(target_os = "windows"))]
 fn dot_path_matching_symlink_with_icase() {
     let tmp_dir = tempdir_in(home()).unwrap();
-    git_dir(&tmp_dir, "foo");
+    git_dir(tmp_dir.path(), "foo");
     create_symlink(
         tmp_dir.path().join("bar").as_path(),
         tmp_dir.path().join("foo").as_path(),