summary refs log tree commit diff
path: root/sys_util/src/seek_hole.rs
diff options
context:
space:
mode:
authorDavid Tolnay <dtolnay@chromium.org>2019-04-17 21:09:41 -0700
committerchrome-bot <chrome-bot@chromium.org>2019-07-11 16:15:38 -0700
commite33b55c4298c8f0d3dff7f04343e765559463d3a (patch)
tree6bde0d2e529f9b650270a95526a3a1a4186f7a63 /sys_util/src/seek_hole.rs
parent8f94a9ff71ce766c9e91fb2284d40f327148e708 (diff)
downloadcrosvm-e33b55c4298c8f0d3dff7f04343e765559463d3a.tar
crosvm-e33b55c4298c8f0d3dff7f04343e765559463d3a.tar.gz
crosvm-e33b55c4298c8f0d3dff7f04343e765559463d3a.tar.bz2
crosvm-e33b55c4298c8f0d3dff7f04343e765559463d3a.tar.lz
crosvm-e33b55c4298c8f0d3dff7f04343e765559463d3a.tar.xz
crosvm-e33b55c4298c8f0d3dff7f04343e765559463d3a.tar.zst
crosvm-e33b55c4298c8f0d3dff7f04343e765559463d3a.zip
tempfile: Unify the two tempdir implementations
Looks like we ended up with two totally different tempdir
implementations: one from CL:520706 and the other from CL:1409705.

This CL consolidates them into one implementation.

BUG=chromium:974059
TEST=tempfile: cargo test
TEST=crosvm: cargo check --all-features
TEST=devices: cargo check --tests
TEST=sys_util: cargo check --tests
TEST=local kokoro
TEST=./build_test

Cq-Depend: chromium:1574668
Change-Id: Id70e963c9986ed2fc5f160819c4a7f9f16092b3b
Signed-off-by: Daniel Verkamp <dverkamp@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1573227
Tested-by: kokoro <noreply+kokoro@google.com>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Diffstat (limited to 'sys_util/src/seek_hole.rs')
-rw-r--r--sys_util/src/seek_hole.rs11
1 files changed, 5 insertions, 6 deletions
diff --git a/sys_util/src/seek_hole.rs b/sys_util/src/seek_hole.rs
index 8215a8d..8cec06c 100644
--- a/sys_util/src/seek_hole.rs
+++ b/sys_util/src/seek_hole.rs
@@ -54,10 +54,9 @@ impl SeekHole for File {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::TempDir;
     use std::fs::File;
     use std::io::{Seek, SeekFrom, Write};
-    use std::path::PathBuf;
+    use tempfile::TempDir;
 
     fn seek_cur(file: &mut File) -> u64 {
         file.seek(SeekFrom::Current(0)).unwrap()
@@ -65,8 +64,8 @@ mod tests {
 
     #[test]
     fn seek_data() {
-        let tempdir = TempDir::new("/tmp/seek_data_test").unwrap();
-        let mut path = PathBuf::from(tempdir.as_path().unwrap());
+        let tempdir = TempDir::new().unwrap();
+        let mut path = tempdir.path().to_owned();
         path.push("test_file");
         let mut file = File::create(&path).unwrap();
 
@@ -112,8 +111,8 @@ mod tests {
 
     #[test]
     fn seek_hole() {
-        let tempdir = TempDir::new("/tmp/seek_hole_test").unwrap();
-        let mut path = PathBuf::from(tempdir.as_path().unwrap());
+        let tempdir = TempDir::new().unwrap();
+        let mut path = tempdir.path().to_owned();
         path.push("test_file");
         let mut file = File::create(&path).unwrap();