diff options
author | David Tolnay <dtolnay@chromium.org> | 2019-04-17 21:09:41 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-07-11 16:15:38 -0700 |
commit | e33b55c4298c8f0d3dff7f04343e765559463d3a (patch) | |
tree | 6bde0d2e529f9b650270a95526a3a1a4186f7a63 /sys_util/src/seek_hole.rs | |
parent | 8f94a9ff71ce766c9e91fb2284d40f327148e708 (diff) | |
download | crosvm-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.rs | 11 |
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(); |