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

@@ -28,3 +28,43 @@ pub fn remove_dir_contents(dir_path: &Path) -> Result<(), io::Error> {
 
     Ok(())
 }
+
+#[cfg(test)]
+mod test {
+    extern crate tempdir;
+    use crate::remove_dir_all;
+    use crate::remove_dir_contents;
+    use std::fs::{self, File};
+    use std::io;
+
+    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(),
+            )),
+        }
+    }
+
+    #[test]
+    fn mkdir_rm() -> Result<(), io::Error> {
+        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)?;
+        remove_dir_all(&ours)?;
+        remove_dir_contents(&ours)?;
+        Ok(())
+    }
+}