Use tempdir for tests

Cargo.toml

@@ -17,6 +17,7 @@ parity-util-mem = { version = "*", default-features = false, features = ["jemall
 [dev-dependencies]
 assert_cmd = "0.12"
 nix = "0.17"
+tempfile = "3.1.0"
 
 [workspace]
 members = [

tests/invalid_order_arguments.rs

@@ -16,11 +16,13 @@
 
 use assert_cmd::cargo::cargo_bin;
 use std::process::Command;
+use tempfile::tempdir;
 
 #[test]
 #[cfg(unix)]
 fn invalid_order_arguments() {
-	let base_path = "invalid_order_arguments";
+	let tmp = tempdir().expect("could not create temp dir");
+	let base_path = tmp.path().to_str().expect("path should consist of valid utf8 characters");
 
 	let status = Command::new(cargo_bin("polkadot"))
 		.args(&["--dev", "invalid_order_arguments", "-d", base_path, "-y"])

tests/purge_chain_works.rs

@@ -15,7 +15,8 @@
 // along with Substrate.  If not, see <http://www.gnu.org/licenses/>.
 
 use assert_cmd::cargo::cargo_bin;
-use std::{convert::TryInto, process::Command, thread, time::Duration, fs, path::PathBuf};
+use std::{convert::TryInto, process::Command, thread, time::Duration};
+use tempfile::tempdir;
 
 mod common;
 
@@ -25,9 +26,9 @@ fn purge_chain_works() {
 	use nix::sys::signal::{kill, Signal::SIGINT};
 	use nix::unistd::Pid;
 
-	let base_path = "purge_chain_test";
+	let tmpdir = tempdir().expect("could not create temp dir");
+	let base_path = tmpdir.path().to_str().expect("path should consist of valid utf8 characters");
 
-	let _ = fs::remove_dir_all(base_path);
 	let mut cmd = Command::new(cargo_bin("polkadot"))
 		.args(&["--dev", "-d", base_path])
 		.spawn()
@@ -49,6 +50,6 @@ fn purge_chain_works() {
 	assert!(status.success());
 
 	// Make sure that the `dev` chain folder exists, but the `db` is deleted.
-	assert!(PathBuf::from(base_path).join("chains/dev/").exists());
-	assert!(!PathBuf::from(base_path).join("chains/dev/db").exists());
+	assert!(tmpdir.path().join("chains/dev/").exists());
+	assert!(!tmpdir.path().join("chains/dev/db").exists());
 }

tests/running_the_node_and_interrupt.rs

@@ -15,7 +15,8 @@
 // along with Substrate.  If not, see <http://www.gnu.org/licenses/>.
 
 use assert_cmd::cargo::cargo_bin;
-use std::{convert::TryInto, process::Command, thread, time::Duration, fs};
+use std::{convert::TryInto, process::Command, thread, time::Duration};
+use tempfile::tempdir;
 
 mod common;
 
@@ -26,9 +27,11 @@ fn running_the_node_works_and_can_be_interrupted() {
 	use nix::unistd::Pid;
 
 	fn run_command_and_kill(signal: Signal) {
-		let _ = fs::remove_dir_all("interrupt_test");
+		let tmp = tempdir().expect("coult not create temp dir");
+		let base_path = tmp.path().to_str().expect("path should be valid utf8");
+
 		let mut cmd = Command::new(cargo_bin("polkadot"))
-			.args(&["--dev", "-d", "interrupt_test"])
+			.args(&["--dev", "-d", base_path])
 			.spawn()
 			.unwrap();