test(mocks,config): update mocks and test configs for withProgress API

- Add withProgress mock implementation to vscode mock
- Add ProgressLocation enum to vscode mock
- Update temperature value from 0.4 to 0.3 in tests
- Remove unnecessary async/await from activate() calls
- Update test assertions for configuration handling

Author: tsdevau

test/__mocks__/vscode.ts

@@ -45,6 +45,11 @@ const mockWindow = {
     show: jest.fn(),
     dispose: jest.fn(),
   }),
+  withProgress: jest.fn().mockImplementation((options, task) => {
+    return task({
+      report: jest.fn(),
+    })
+  }),
 }
 
 const mockExtensions = {
@@ -74,3 +79,8 @@ export const Uri = {
 // Export any additional VSCode APIs that tests need
 export const EventEmitter = jest.fn()
 export const TextDocument = jest.fn()
+export const ProgressLocation = {
+  Notification: 1,
+  SourceControl: 2,
+  Window: 3,
+}

test/configurationEdgeCases.test.ts

@@ -62,7 +62,7 @@ describe("Configuration Edge Cases", () => {
     mockConfig = {
       model: "claude-3-5-sonnet-latest",
       maxTokens: 1024,
-      temperature: 0.4,
+      temperature: 0.3,
       allowedTypes: ["feat", "fix", "refactor", "chore", "docs", "style", "test", "perf", "ci"],
     }
 
@@ -185,7 +185,7 @@ describe("Configuration Edge Cases", () => {
 
       expect(mockAnthropicCreate).toHaveBeenCalledWith(
         expect.objectContaining({
-          temperature: 0.4,
+          temperature: 0.3,
         }),
       )
     })

test/configurationHandling.test.ts

@@ -79,7 +79,7 @@ describe("Configuration Handling", () => {
       return config[key]
     })
 
-    await activate(mockContext)
+    activate(mockContext)
     await vscode.commands.executeCommand("diffCommit.generateCommitMessage")
 
     expect(configGetMock).toHaveBeenCalledWith("model")
@@ -92,7 +92,7 @@ describe("Configuration Handling", () => {
   test("should use default values when configuration is missing", async () => {
     configGetMock.mockReturnValue(undefined)
 
-    await activate(mockContext)
+    activate(mockContext)
     await vscode.commands.executeCommand("diffCommit.generateCommitMessage")
 
     expect(configGetMock).toHaveBeenCalled()
@@ -106,7 +106,7 @@ describe("Configuration Handling", () => {
       return undefined
     })
 
-    await activate(mockContext)
+    activate(mockContext)
     await vscode.commands.executeCommand("diffCommit.generateCommitMessage")
 
     expect(configGetMock).toHaveBeenCalledWith("customInstructions")
@@ -115,7 +115,7 @@ describe("Configuration Handling", () => {
   test("should handle missing workspace folder", async () => {
     ;(vscode.workspace.workspaceFolders as any) = undefined
 
-    await activate(mockContext)
+    activate(mockContext)
     await vscode.commands.executeCommand("diffCommit.generateCommitMessage")
 
     expect(vscode.window.showErrorMessage).toHaveBeenCalledWith("No workspace folder found")
@@ -124,7 +124,7 @@ describe("Configuration Handling", () => {
   test("should handle missing Git extension", async () => {
     ;(vscode.extensions.getExtension as jest.Mock).mockReturnValue(undefined)
 
-    await activate(mockContext)
+    activate(mockContext)
     await vscode.commands.executeCommand("diffCommit.generateCommitMessage")
 
     expect(vscode.window.showErrorMessage).toHaveBeenCalledWith("Git extension not found")
@@ -146,7 +146,7 @@ describe("Configuration Handling", () => {
       return undefined
     })
 
-    await activate(mockContext)
+    activate(mockContext)
     await vscode.commands.executeCommand("diffCommit.generateCommitMessage")
 
     expect(vscode.window.showErrorMessage).toHaveBeenCalledWith("No Git repository found")
@@ -178,7 +178,7 @@ describe("Configuration Handling", () => {
       return undefined
     })
 
-    await activate(mockContext)
+    activate(mockContext)
     await vscode.commands.executeCommand("diffCommit.generateCommitMessage")
 
     expect(vscode.window.showErrorMessage).toHaveBeenCalledWith("No changes detected")