fix reflexivity

Author: steveklabnik

src/lib.rs

@@ -53,6 +53,10 @@ pub fn is_different<A: AsRef<Path>, B: AsRef<Path>>(a_base: A, b_base: B) -> Res
         return Ok(true);
     }
 
+    Ok(compare(&a_base, &b_base)? || compare(&b_base, &a_base)?)
+}
+
+fn compare(a_base: &Path, b_base: &Path) -> Result<bool, Error> {
     // next, we walk all of the entries in a and compare them to b
     for entry in WalkDir::new(a_base) {
         let entry = entry?;
@@ -64,8 +68,6 @@ pub fn is_different<A: AsRef<Path>, B: AsRef<Path>>(a_base: A, b_base: B) -> Res
         // and then join that with b to get the path in b
         let b = b_base.join(no_prefix);
 
-        println!("comparing {} and {}", a.display(), b.display());
-
         if a.is_dir() {
             if b.is_dir() {
                 // can't compare the contents of directories, so just continue
@@ -91,7 +93,7 @@ pub fn is_different<A: AsRef<Path>, B: AsRef<Path>>(a_base: A, b_base: B) -> Res
         }
     }
 
-    // if we made it here, everything in a is in b!
+    // if we made it here, we're good!
     Ok(false)
 }
 

tests/reflexive/dir1/one.txt

@@ -0,0 +1 @@
+one

tests/reflexive/dir1/subdir/two.txt

@@ -0,0 +1 @@
+two

tests/reflexive/dir2/one.txt

@@ -0,0 +1 @@
+one

tests/reflexive/dir2/subdir/three.txt

@@ -0,0 +1 @@
+three

tests/reflexive/dir2/subdir/two.txt

@@ -0,0 +1 @@
+two

tests/smoke.rs

@@ -28,4 +28,9 @@ fn fileanddir() {
 #[test]
 fn oneempty() {
     assert!(dir_diff::is_different("tests/oneempty/dir1", "tests/oneempty/dir2").unwrap());
+}
+
+#[test]
+fn reflexive() {
+    assert!(dir_diff::is_different("tests/reflexive/dir1", "tests/reflexive/dir2").unwrap());
 }