Add test case for remove_dir_contents

We check a few error behaviours too.

Signed-off-by: Ian Jackson <[email protected]>

Author: ijackson

Cargo.toml

@@ -24,3 +24,4 @@ libc = "0.2"
 
 [dev-dependencies]
 doc-comment = "0.3"
+tempdir = "0.3"

src/portable.rs

@@ -1,7 +1,10 @@
-use std::fs;
+use std::fs::{self, File};
 use std::io;
 use std::path::Path;
 
+#[cfg(test)]
+extern crate tempdir;
+
 /// Deletes the contents of `dir_path`, but not the directory iteself.
 ///
 /// If `dir_path` is a symlink to a directory, deletes the contents
@@ -28,3 +31,34 @@ pub fn remove_dir_contents(dir_path: &Path) -> Result<(), io::Error> {
 
     Ok(())
 }
+
+#[test]
+fn mkdir_rm() -> Result<(), io::Error> {
+    fn expect_failure<T>(k: io::ErrorKind, r: io::Result<T>) -> io::Result<()> {
+        match r {
+            Err(e) if e.kind() == k => Ok(()),
+            Err(e) => Err(e),
+            Ok(_) => Err(io::Error::new(
+                io::ErrorKind::Other,
+                "unexpected success".to_string(),
+            )),
+        }
+    };
+
+    let tmp = tempdir::TempDir::new("t.remove_dir_all")?;
+    let ours = tmp.path().join("t.mkdir");
+    let file = ours.join("file");
+    fs::create_dir(&ours)?;
+    File::create(&file)?;
+    File::open(&file)?;
+
+    expect_failure(io::ErrorKind::Other, remove_dir_contents(&file))?;
+
+    remove_dir_contents(&ours)?;
+    expect_failure(io::ErrorKind::NotFound, File::open(&file))?;
+
+    remove_dir_contents(&ours)?;
+    crate::remove_dir_all(&ours)?;
+    remove_dir_contents(&ours)?;
+    Ok(())
+}