chore: testActiveDirectoryAuth cleanup, remove old test packages from cli

Author: jescalada

packages/git-proxy-cli/package.json

@@ -10,10 +10,6 @@
     "yargs": "^17.7.2",
     "@finos/git-proxy": "file:../.."
   },
-  "devDependencies": {
-    "chai": "^4.5.0",
-    "ts-mocha": "^11.1.0"
-  },
   "scripts": {
     "build": "tsc",
     "lint": "eslint \"./*.ts\" --fix",

test/testActiveDirectoryAuth.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, beforeEach, expect, vi, type Mock } from 'vitest';
+import { describe, it, beforeEach, expect, vi, type Mock, afterEach } from 'vitest';
 
 let ldapStub: { isUserInAdGroup: Mock };
 let dbStub: { updateUser: Mock };
@@ -33,9 +33,6 @@ const newConfig = JSON.stringify({
 
 describe('ActiveDirectory auth method', () => {
   beforeEach(async () => {
-    vi.clearAllMocks();
-    vi.resetModules();
-
     ldapStub = {
       isUserInAdGroup: vi.fn(),
     };
@@ -84,6 +81,11 @@ describe('ActiveDirectory auth method', () => {
     configure(passportStub as any);
   });
 
+  afterEach(() => {
+    vi.clearAllMocks();
+    vi.resetModules();
+  });
+
   it('should authenticate a valid user and mark them as admin', async () => {
     const mockReq = {};
     const mockProfile = {