From a5f116f937e7b206568390b17f5bfddacad96e19 Mon Sep 17 00:00:00 2001
From: "Dustin J. Mitchell" <dustin@v.igoro.us>
Date: Mon, 23 Sep 2024 19:32:25 -0400
Subject: [PATCH] Additional tests for Replica methods (#459)

Address some missing test coverage identified by `cargo mutants` in #457.
---
 taskchampion/src/replica.rs | 43 +++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/taskchampion/src/replica.rs b/taskchampion/src/replica.rs
index d853a3956..befd65b0f 100644
--- a/taskchampion/src/replica.rs
+++ b/taskchampion/src/replica.rs
@@ -601,6 +601,11 @@ mod tests {
 
         // num_undo_points includes only the undo point
         assert_eq!(rep.num_undo_points().unwrap(), 1);
+
+        // A second undo point is counted.
+        let ops = vec![Operation::UndoPoint];
+        rep.commit_operations(ops).unwrap();
+        assert_eq!(rep.num_undo_points().unwrap(), 2);
     }
 
     #[test]
@@ -636,6 +641,13 @@ mod tests {
         assert_eq!(all_tasks.len(), 2);
         assert_eq!(all_tasks.get(&uuid1).unwrap().get_uuid(), uuid1);
         assert_eq!(all_tasks.get(&uuid2).unwrap().get_uuid(), uuid2);
+
+        let mut all_uuids = rep.all_task_uuids().unwrap();
+        all_uuids.sort();
+        let mut exp_uuids = vec![uuid1, uuid2];
+        exp_uuids.sort();
+        assert_eq!(all_uuids.len(), 2);
+        assert_eq!(all_uuids, exp_uuids);
     }
 
     #[test]
@@ -759,6 +771,37 @@ mod tests {
         Ok(())
     }
 
+    #[test]
+    fn commit_reversed_operations() -> Result<()> {
+        let uuid1 = Uuid::new_v4();
+        let uuid2 = Uuid::new_v4();
+        let uuid3 = Uuid::new_v4();
+
+        let mut rep = Replica::new_inmemory();
+
+        let mut ops = Operations::new();
+        ops.push(Operation::UndoPoint);
+        rep.create_task(uuid1, &mut ops).unwrap();
+        ops.push(Operation::UndoPoint);
+        rep.create_task(uuid2, &mut ops).unwrap();
+        rep.commit_operations(dbg!(ops))?;
+        assert_eq!(rep.num_undo_points().unwrap(), 2);
+
+        // Trying to reverse-commit the wrong operations fails.
+        let ops = vec![Operation::Delete {
+            uuid: uuid3,
+            old_task: TaskMap::new(),
+        }];
+        assert!(!rep.commit_reversed_operations(ops)?);
+
+        // Commiting the correct operations succeeds
+        let ops = rep.get_undo_operations()?;
+        assert!(rep.commit_reversed_operations(ops)?);
+        assert_eq!(rep.num_undo_points().unwrap(), 1);
+
+        Ok(())
+    }
+
     #[test]
     fn get_and_modify() {
         let mut rep = Replica::new_inmemory();