From 9e1caa42e455e7d21e489a8099f37e690c260080 Mon Sep 17 00:00:00 2001 From: Igor Gnatenko Date: Thu, 10 May 2018 11:45:33 +0200 Subject: [PATCH] deps: update tempfile to 3 Signed-off-by: Igor Gnatenko --- Cargo.toml | 3 +-- src/unistd.rs | 12 ++++++------ test/sys/test_socket.rs | 8 ++++---- test/sys/test_uio.rs | 9 ++++----- test/test.rs | 1 - test/test_fcntl.rs | 6 ++---- test/test_mount.rs | 19 +++++++------------ test/test_stat.rs | 12 ++++++------ test/test_unistd.rs | 11 ++++------- 9 files changed, 34 insertions(+), 47 deletions(-) diff --git a/Cargo.toml b/Cargo.toml index 9f580a9a49..063097ab14 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -24,8 +24,7 @@ cc = "1" bytes = "0.4.8" lazy_static = "1" rand = "0.4" -tempdir = "0.3" -tempfile = "2" +tempfile = "3" [target.'cfg(target_os = "freebsd")'.dev-dependencies] sysctl = "0.1" diff --git a/src/unistd.rs b/src/unistd.rs index 8022aa0b2c..756a4d69f5 100644 --- a/src/unistd.rs +++ b/src/unistd.rs @@ -421,15 +421,15 @@ pub fn fchdir(dirfd: RawFd) -> Result<()> { /// # Example /// /// ```rust -/// extern crate tempdir; +/// extern crate tempfile; /// extern crate nix; /// /// use nix::unistd; /// use nix::sys::stat; -/// use tempdir::TempDir; +/// use tempfile::TempDir; /// /// fn main() { -/// let tmp_dir1 = TempDir::new("test_mkdir").unwrap(); +/// let tmp_dir1 = TempDir::new().unwrap(); /// let tmp_dir2 = tmp_dir1.path().join("new_dir"); /// /// // create new directory and give read, write and execute rights to the owner @@ -464,15 +464,15 @@ pub fn mkdir(path: &P, mode: Mode) -> Result<()> { /// # Example /// /// ```rust -/// extern crate tempdir; +/// extern crate tempfile; /// extern crate nix; /// /// use nix::unistd; /// use nix::sys::stat; -/// use tempdir::TempDir; +/// use tempfile::TempDir; /// /// fn main() { -/// let tmp_dir = TempDir::new("test_fifo").unwrap(); +/// let tmp_dir = TempDir::new().unwrap(); /// let fifo_path = tmp_dir.path().join("foo.pipe"); /// /// // create new fifo and give read, write and execute rights to the owner diff --git a/test/sys/test_socket.rs b/test/sys/test_socket.rs index 35e3bf9052..d2a5268c7f 100644 --- a/test/sys/test_socket.rs +++ b/test/sys/test_socket.rs @@ -90,9 +90,9 @@ pub fn test_abstract_uds_addr() { pub fn test_getsockname() { use nix::sys::socket::{socket, AddressFamily, SockType, SockFlag}; use nix::sys::socket::{bind, SockAddr}; - use tempdir::TempDir; + use tempfile; - let tempdir = TempDir::new("test_getsockname").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let sockname = tempdir.path().join("sock"); let sock = socket(AddressFamily::Unix, SockType::Stream, SockFlag::empty(), None) .expect("socket failed"); @@ -254,9 +254,9 @@ pub fn test_unixdomain() { use nix::sys::socket::{bind, socket, connect, listen, accept, SockAddr}; use nix::unistd::{read, write, close}; use std::thread; - use tempdir::TempDir; + use tempfile; - let tempdir = TempDir::new("test_unixdomain").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let sockname = tempdir.path().join("sock"); let s1 = socket(AddressFamily::Unix, SockType::Stream, SockFlag::empty(), None).expect("socket failed"); diff --git a/test/sys/test_uio.rs b/test/sys/test_uio.rs index 9f56c810a1..cefc8bba84 100644 --- a/test/sys/test_uio.rs +++ b/test/sys/test_uio.rs @@ -5,8 +5,7 @@ use std::{cmp, iter}; use std::fs::{OpenOptions}; use std::os::unix::io::AsRawFd; -use tempdir::TempDir; -use tempfile::tempfile; +use tempfile::{self, tempfile}; #[test] fn test_writev() { @@ -114,7 +113,7 @@ fn test_pwrite() { fn test_pread() { use std::io::Write; - let tempdir = TempDir::new("nix-test_pread").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let path = tempdir.path().join("pread_test_file"); let mut file = OpenOptions::new().write(true).read(true).create(true) @@ -142,7 +141,7 @@ fn test_pwritev() { IoVec::from_slice(&to_write[64..128]), ]; - let tempdir = TempDir::new("nix-test_pwritev").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); // pwritev them into a temporary file let path = tempdir.path().join("pwritev_test_file"); @@ -166,7 +165,7 @@ fn test_preadv() { let to_write: Vec = (0..200).collect(); let expected: Vec = (100..200).collect(); - let tempdir = TempDir::new("nix-test_preadv").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let path = tempdir.path().join("preadv_test_file"); diff --git a/test/test.rs b/test/test.rs index 8083b8f9d6..da55235af2 100644 --- a/test/test.rs +++ b/test/test.rs @@ -7,7 +7,6 @@ extern crate nix; extern crate lazy_static; extern crate libc; extern crate rand; -extern crate tempdir; extern crate tempfile; mod sys; diff --git a/test/test_fcntl.rs b/test/test_fcntl.rs index 57b35837df..bcc523bfcc 100644 --- a/test/test_fcntl.rs +++ b/test/test_fcntl.rs @@ -1,8 +1,7 @@ use nix::fcntl::{openat, open, OFlag, readlink, readlinkat}; use nix::sys::stat::Mode; use nix::unistd::{close, read}; -use tempdir::TempDir; -use tempfile::NamedTempFile; +use tempfile::{self, NamedTempFile}; use std::io::prelude::*; use std::os::unix::fs; @@ -30,8 +29,7 @@ fn test_openat() { #[test] fn test_readlink() { - let tempdir = TempDir::new("nix-test_readdir") - .unwrap_or_else(|e| panic!("tempdir failed: {}", e)); + let tempdir = tempfile::tempdir().unwrap(); let src = tempdir.path().join("a"); let dst = tempdir.path().join("b"); println!("a: {:?}, b: {:?}", &src, &dst); diff --git a/test/test_mount.rs b/test/test_mount.rs index 89416a43d1..2e4f19ccf4 100644 --- a/test/test_mount.rs +++ b/test/test_mount.rs @@ -5,7 +5,7 @@ extern crate libc; extern crate nix; -extern crate tempdir; +extern crate tempfile; #[cfg(target_os = "linux")] mod test_mount { @@ -23,7 +23,7 @@ mod test_mount { use nix::sys::stat::{self, Mode}; use nix::unistd::getuid; - use tempdir::TempDir; + use tempfile; static SCRIPT_CONTENTS: &'static [u8] = b"#!/bin/sh exit 23"; @@ -32,8 +32,7 @@ exit 23"; const NONE: Option<&'static [u8]> = None; pub fn test_mount_tmpfs_without_flags_allows_rwx() { - let tempdir = TempDir::new("nix-test_mount") - .unwrap_or_else(|e| panic!("tempdir failed: {}", e)); + let tempdir = tempfile::tempdir().unwrap(); mount(NONE, tempdir.path(), @@ -89,8 +88,7 @@ exit 23"; } pub fn test_mount_rdonly_disallows_write() { - let tempdir = TempDir::new("nix-test_mount") - .unwrap_or_else(|e| panic!("tempdir failed: {}", e)); + let tempdir = tempfile::tempdir().unwrap(); mount(NONE, tempdir.path(), @@ -107,8 +105,7 @@ exit 23"; } pub fn test_mount_noexec_disallows_exec() { - let tempdir = TempDir::new("nix-test_mount") - .unwrap_or_else(|e| panic!("tempdir failed: {}", e)); + let tempdir = tempfile::tempdir().unwrap(); mount(NONE, tempdir.path(), @@ -146,13 +143,11 @@ exit 23"; } pub fn test_mount_bind() { - let tempdir = TempDir::new("nix-test_mount") - .unwrap_or_else(|e| panic!("tempdir failed: {}", e)); + let tempdir = tempfile::tempdir().unwrap(); let file_name = "test"; { - let mount_point = TempDir::new("nix-test_mount") - .unwrap_or_else(|e| panic!("tempdir failed: {}", e)); + let mount_point = tempfile::tempdir().unwrap(); mount(Some(tempdir.path()), mount_point.path(), diff --git a/test/test_stat.rs b/test/test_stat.rs index c042ce195f..4135052efe 100644 --- a/test/test_stat.rs +++ b/test/test_stat.rs @@ -9,7 +9,7 @@ use nix::sys::stat::{self, fchmod, fchmodat, fstat, lstat, stat}; use nix::sys::stat::{FileStat, Mode, FchmodatFlags}; use nix::unistd::chdir; use nix::Result; -use tempdir::TempDir; +use tempfile; #[allow(unused_comparisons)] // uid and gid are signed on Windows, but not on other platforms. This function @@ -56,7 +56,7 @@ fn assert_lstat_results(stat_result: Result) { #[test] fn test_stat_and_fstat() { - let tempdir = TempDir::new("nix-test_stat_and_fstat").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let filename = tempdir.path().join("foo.txt"); let file = File::create(&filename).unwrap(); @@ -69,7 +69,7 @@ fn test_stat_and_fstat() { #[test] fn test_fstatat() { - let tempdir = TempDir::new("nix-test_fstatat").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let filename = tempdir.path().join("foo.txt"); File::create(&filename).unwrap(); let dirfd = fcntl::open(tempdir.path(), @@ -84,7 +84,7 @@ fn test_fstatat() { #[test] fn test_stat_fstat_lstat() { - let tempdir = TempDir::new("nix-test_stat_fstat_lstat").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let filename = tempdir.path().join("bar.txt"); let linkname = tempdir.path().join("barlink"); @@ -106,7 +106,7 @@ fn test_stat_fstat_lstat() { #[test] fn test_fchmod() { - let tempdir = TempDir::new("nix-test_fchmod").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let filename = tempdir.path().join("foo.txt"); let file = File::create(&filename).unwrap(); @@ -128,7 +128,7 @@ fn test_fchmod() { #[test] fn test_fchmodat() { - let tempdir = TempDir::new("nix-test_fchmodat").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let filename = "foo.txt"; let fullpath = tempdir.path().join(filename); File::create(&fullpath).unwrap(); diff --git a/test/test_unistd.rs b/test/test_unistd.rs index fe33b1d06c..d36a3d3934 100644 --- a/test/test_unistd.rs +++ b/test/test_unistd.rs @@ -1,5 +1,3 @@ -extern crate tempdir; - use nix::fcntl::{fcntl, FcntlArg, FdFlag, OFlag}; use nix::unistd::*; use nix::unistd::ForkResult::*; @@ -11,8 +9,7 @@ use std::ffi::CString; use std::fs::File; use std::io::Write; use std::os::unix::prelude::*; -use tempfile::tempfile; -use tempdir::TempDir; +use tempfile::{self, tempfile}; use libc::{self, _exit, off_t}; #[test] @@ -84,7 +81,7 @@ fn test_mkstemp_directory() { #[test] fn test_mkfifo() { - let tempdir = TempDir::new("nix-test_mkfifo").unwrap(); + let tempdir = tempfile::tempdir().unwrap(); let mkfifo_fifo = tempdir.path().join("mkfifo_fifo"); mkfifo(&mkfifo_fifo, Mode::S_IRUSR).unwrap(); @@ -286,7 +283,7 @@ fn test_fchdir() { #[allow(unused_variables)] let m = ::CWD_MTX.lock().expect("Mutex got poisoned by another test"); - let tmpdir = TempDir::new("test_fchdir").unwrap(); + let tmpdir = tempfile::tempdir().unwrap(); let tmpdir_path = tmpdir.path().canonicalize().unwrap(); let tmpdir_fd = File::open(&tmpdir_path).unwrap().into_raw_fd(); @@ -302,7 +299,7 @@ fn test_getcwd() { #[allow(unused_variables)] let m = ::CWD_MTX.lock().expect("Mutex got poisoned by another test"); - let tmpdir = TempDir::new("test_getcwd").unwrap(); + let tmpdir = tempfile::tempdir().unwrap(); let tmpdir_path = tmpdir.path().canonicalize().unwrap(); assert!(chdir(&tmpdir_path).is_ok()); assert_eq!(getcwd().unwrap(), tmpdir_path);