Fix file based history when capacity is set to zero

Signed-off-by: Andrei Stan <[email protected]>

Author: andreistan26

src/history/base.rs

@@ -427,4 +427,17 @@ mod test {
 
         Ok(())
     }
+
+    #[test]
+    fn history_size_zero() -> Result<()> {
+        #[cfg(not(any(feature = "sqlite", feature = "sqlite-dynlib")))]
+        let mut history = crate::FileBackedHistory::new(0);
+        history.save(create_item(1, "/home/me", "cd ~/Downloads", 0))?;
+        assert_eq!(history.count_all()?, 0);
+        let _ = history.sync();
+        history.clear()?;
+        drop(history);
+
+        Ok(())
+    }
 }

src/history/file_backed.rs

@@ -59,7 +59,7 @@ impl History for FileBackedHistory {
             .entries
             .back()
             .map_or(true, |previous| previous != &entry)
-            && !entry.is_empty()
+            && !entry.is_empty() && self.capacity > 0
         {
             if self.entries.len() == self.capacity {
                 // History is "full", so we delete the oldest entry first,
@@ -234,7 +234,7 @@ impl History for FileBackedHistory {
                     .collect::<std::io::Result<VecDeque<_>>>()?;
                 if from_file.len() + own_entries.len() > self.capacity {
                     (
-                        from_file.split_off(from_file.len() - (self.capacity - own_entries.len())),
+                        from_file.split_off(from_file.len() - (self.capacity.saturating_sub(own_entries.len()))),
                         true,
                     )
                 } else {