better verbosity

src/index.rs

@@ -62,7 +62,6 @@ impl Index {
     }
 
     pub fn remember(&mut self, item: RepositoryItem) {
-        println!("remembering {}", item);
         self.items.push(item.into());
     }
 

src/repository.rs

@@ -67,7 +67,6 @@ impl<'a> Repository<'a> {
         }
 
         if source_path.is_file() {
-            println!("storing {} as {}", source_path.display(), destination_path.display());
             fs::create_dir_all(destination_path.parent().unwrap())?;
             let version = Repository::calculate_version(source_path)?;
             fs::copy(source_path, destination_path)?;
@@ -95,12 +94,6 @@ impl<'a> Repository<'a> {
     }
 
     pub fn item_by_source_path(&self, path: &Path) -> Result<Option<RepositoryItem>, BakareError> {
-        println!(
-            "trying to find {} in a repo [{}] of {} items",
-            path.display(),
-            self.path.display(),
-            self.index.len()
-        );
         if !path.is_absolute() {
             return Err(BakareError::RepositoryPathNotAbsolute);
         }

src/repository_item.rs

@@ -32,13 +32,11 @@ impl RepositoryItem {
         }
         let parent = target_path.parent().unwrap();
         if !parent.exists() {
-            println!("Creating {}", parent.display());
             fs::create_dir_all(parent)?;
         }
         if !self.absolute_path.exists() {
             return Err(BakareError::CorruptedRepoNoFile);
         }
-        println!("Saving {} to {}", self.absolute_path.display(), target_path.display());
         fs::copy(&self.absolute_path, &target_path)?;
 
         Ok(())

tests/system_tests.rs

@@ -137,7 +137,7 @@ fn assert_directory_trees_have_same_contents(left: &Path, right: &Path) -> Resul
         let mut br = vec![];
         fl.read_to_end(&mut bl)?;
         fr.read_to_end(&mut br)?;
-        assert_eq!(bl, bl);
+        assert_eq!(bl, br);
     }
     Ok(())
 }