feat: better error when no record found

src/lib.rs

@@ -69,7 +69,7 @@ pub fn run(cli: Args, mode: impl util::TestingMode, stream: &mut impl Write) ->
         // the graves_to_exhume.
         if cli.seance && record.open().is_ok() {
             let gravepath = util::join_absolute(graveyard, dunce::canonicalize(cwd)?);
-            for grave in record.seance(&gravepath) {
+            for grave in record.seance(&gravepath)? {
                 graves_to_exhume.push(grave);
             }
         }
@@ -112,7 +112,7 @@ pub fn run(cli: Args, mode: impl util::TestingMode, stream: &mut impl Write) ->
 
     if cli.seance {
         let gravepath = util::join_absolute(graveyard, dunce::canonicalize(cwd)?);
-        for grave in record.seance(&gravepath) {
+        for grave in record.seance(&gravepath)? {
             writeln!(stream, "{}", grave.display())?;
         }
         return Ok(());

src/record.rs

@@ -123,13 +123,16 @@ impl Record {
     }
 
     /// Returns an iterator over all graves in the record that are under gravepath
-    pub fn seance<'a>(&'a self, gravepath: &'a PathBuf) -> impl Iterator<Item = PathBuf> + 'a {
-        let record_file = self.open().unwrap();
-        BufReader::new(record_file)
+    pub fn seance<'a>(
+        &'a self,
+        gravepath: &'a PathBuf,
+    ) -> io::Result<impl Iterator<Item = PathBuf> + 'a> {
+        let record_file = self.open()?;
+        Ok(BufReader::new(record_file)
             .lines()
             .map_while(Result::ok)
             .map(|line| PathBuf::from(RecordItem::new(&line).dest))
-            .filter(move |d| d.starts_with(gravepath))
+            .filter(move |d| d.starts_with(gravepath)))
     }
 
     /// Write deletion history to record

tests/integration_tests.rs

@@ -699,3 +699,20 @@ fn test_graveyard_subcommand(#[values(false, true)] seance: bool) {
         .success()
         .stdout(expected_str);
 }
+
+#[rstest]
+fn read_empty_record() {
+    let _env_lock = aquire_lock();
+    let test_env = TestEnv::new();
+    let cwd = env::current_dir().unwrap();
+    let record = record::Record::new(&test_env.graveyard);
+    let gravepath = &util::join_absolute(&test_env.graveyard, dunce::canonicalize(cwd).unwrap());
+    let result = record.seance(gravepath);
+    assert!(result.is_err());
+    if let Err(e) = result {
+        assert_eq!(e.kind(), ErrorKind::NotFound);
+        assert_eq!(e.to_string(), "Failed to read record!");
+    } else {
+        panic!("Expected an error");
+    }
+}