chore: improve test assertions and cleanup

Author: jescalada

test/1.test.ts

@@ -52,8 +52,6 @@ describe('init', () => {
 
   // Example test: use vi.doMock to override the config module
   it('should return an array of enabled auth methods when overridden', async () => {
-    vi.resetModules(); // Clear module cache
-
     // fs must be mocked BEFORE importing the config module
     // We also mock existsSync to ensure the file "exists"
     vi.doMock('fs', async (importOriginal) => {
@@ -87,6 +85,9 @@ describe('init', () => {
   afterEach(function () {
     // Restore all stubs
     vi.restoreAllMocks();
+
+    // Clear module cache
+    vi.resetModules();
   });
 
   // Runs after all tests

test/extractRawBody.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, beforeEach, expect, vi, Mock } from 'vitest';
+import { describe, it, beforeEach, expect, vi, Mock, afterAll } from 'vitest';
 import { PassThrough } from 'stream';
 
 // Tell Vitest to mock dependencies
@@ -33,7 +33,9 @@ describe('extractRawBody middleware', () => {
     };
 
     next = vi.fn();
+  });
 
+  afterAll(() => {
     (rawBody as Mock).mockClear();
     (chain.executeChain as Mock).mockClear();
   });

test/generated-config.test.ts

@@ -302,12 +302,9 @@ describe('Generated Config (QuickType)', () => {
     });
 
     it('should test validation error paths', () => {
-      try {
-        // Try to parse something that looks like valid JSON but has wrong structure
-        Convert.toGitProxyConfig('{"proxyUrl": 123, "authentication": "not-array"}');
-      } catch (error) {
-        assert.instanceOf(error, Error);
-      }
+      assert.throws(() =>
+        Convert.toGitProxyConfig('{"proxyUrl": 123, "authentication": "not-array"}'),
+      );
     });
 
     it('should test date and null handling', () => {