test: improve mock cleanup in evaluator and runner tests

- Move mock.restore() from beforeEach to afterAll in evaluator.unit.test.ts
- Move mock.restore() from beforeEach to afterAll in runner.unit.test.ts
- Add cleanup comments for clarity
- Update Bun package manager from 1.1.0 to 1.3.0 in package.json

🤖 Generated with Claude via commitment

Author: arittr

package.json

@@ -41,7 +41,7 @@
   ],
   "license": "ISC",
   "name": "@arittr/commitment",
-  "packageManager": "bun@1.1.0",
+  "packageManager": "bun@1.3.0",
   "scripts": {
     "build": "bun run build.ts",
     "check-types": "bun tsc --noEmit",

src/eval/__tests__/evaluator.unit.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, it, mock, spyOn } from 'bun:test';
+import { afterAll, beforeEach, describe, expect, it, mock, spyOn } from 'bun:test';
 /**
  * Unit tests for Evaluator module
  *
@@ -31,17 +31,18 @@ describe('Evaluator', () => {
   let mockChatGptAgent: ChatGPTAgent;
 
   beforeEach(() => {
-    // Reset mocks
-    mock.restore();
-
     // Create evaluator instance
     evaluator = new Evaluator();
 
     // Get mocked ChatGPT agent instance
-
     mockChatGptAgent = (evaluator as unknown as { agent: ChatGPTAgent }).agent;
   });
 
+  afterAll(() => {
+    // Clean up module mocks after this test suite
+    mock.restore();
+  });
+
   describe('evaluate', () => {
     it('should call ChatGPT agent with correct parameters', async () => {
       // Arrange

src/eval/__tests__/runner.unit.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, it, mock, spyOn } from 'bun:test';
+import { afterAll, beforeEach, describe, expect, it, mock, spyOn } from 'bun:test';
 /**
  * Unit tests for EvalRunner module
  *
@@ -63,10 +63,14 @@ describe('EvalRunner', () => {
   let runner: EvalRunner;
 
   beforeEach(() => {
-    mock.restore();
     runner = new EvalRunner();
   });
 
+  afterAll(() => {
+    // Clean up module mocks after this test suite
+    mock.restore();
+  });
+
   describe('loadFixture', () => {
     it('should load mocked fixture with all required fields', async () => {
       // Arrange