tests, optimizations, refactors

Author: liwilliam2021

src/core/checkpoints/__tests__/checkpoint.test.ts

@@ -1358,12 +1358,17 @@ describe("Cline", () => {
 			})
 
 			it("should handle pending ask operations gracefully when task is aborted", async () => {
-				const [cline, task] = Task.create({
+				const [cline, taskPromise] = Task.create({
 					provider: mockProvider,
 					apiConfiguration: mockApiConfig,
 					task: "test task",
 				})
 
+				// Handle the task promise to prevent unhandled rejection
+				taskPromise.catch(() => {
+					// Expected error when task is aborted
+				})
+
 				// Start an ask operation but don't respond to it
 				const askPromise = cline.ask("tool", "Test question")
 
@@ -1381,12 +1386,17 @@ describe("Cline", () => {
 			})
 
 			it("should not throw 'Current ask promise was ignored' error when task is aborted", async () => {
-				const [cline, task] = Task.create({
+				const [cline, taskPromise] = Task.create({
 					provider: mockProvider,
 					apiConfiguration: mockApiConfig,
 					task: "test task",
 				})
 
+				// Handle the task promise to prevent unhandled rejection
+				taskPromise.catch(() => {
+					// Expected error when task is aborted
+				})
+
 				// Start multiple ask operations
 				const askPromise1 = cline.ask("tool", "Question 1")
 				const askPromise2 = cline.ask("tool", "Question 2")
@@ -1400,12 +1410,17 @@ describe("Cline", () => {
 			})
 
 			it("should resolve pending ask with messageResponse when abortTask is called", async () => {
-				const [cline, task] = Task.create({
+				const [cline, taskPromise] = Task.create({
 					provider: mockProvider,
 					apiConfiguration: mockApiConfig,
 					task: "test task",
 				})
 
+				// Handle the task promise to prevent unhandled rejection
+				taskPromise.catch(() => {
+					// Expected error when task is aborted
+				})
+
 				// Spy on the ask response properties
 				// Start an ask operation
 				const askPromise = cline.ask("tool", "Test question")