test: update checkpoint test to include suppressMessage parameter

daniel-lxs · daniel-lxs · commit cfaa4714b55a · 2025-09-05T16:59:56.000-05:00
- Fixed test expectation to match the new function signature
- saveCheckpoint now expects both allowEmpty and suppressMessage parameters
diff --git a/src/core/checkpoints/__tests__/checkpoint.test.ts b/src/core/checkpoints/__tests__/checkpoint.test.ts
@@ -111,7 +111,7 @@ describe("Checkpoint functionality", () => {
 			// saveCheckpoint should have been called
 			expect(mockCheckpointService.saveCheckpoint).toHaveBeenCalledWith(
 				expect.stringContaining("Task: test-task-id"),
-				{ allowEmpty: true },
+				{ allowEmpty: true, suppressMessage: false },
 			)
 
 			// Result should contain the commit hash

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ describe("Checkpoint functionality", () => {`
`111`	`111`	`// saveCheckpoint should have been called`
`112`	`112`	`expect(mockCheckpointService.saveCheckpoint).toHaveBeenCalledWith(`
`113`	`113`	`expect.stringContaining("Task: test-task-id"),`
`114`		`- { allowEmpty: true },`
	`114`	`+ { allowEmpty: true, suppressMessage: false },`
`115`	`115`	`)`
`116`	`116`
`117`	`117`	`// Result should contain the commit hash`