Improve test isolation

Glitched · Glitched · commit fd335d2de2cc · 2025-07-14T17:58:55.000-04:00
- Add afterEach to restore all mocks
- Remove unnecessary mockClear() calls
- Ensures tests are properly isolated
diff --git a/src/server/auth/handlers/authorize.test.ts b/src/server/auth/handlers/authorize.test.ts
@@ -102,6 +102,10 @@ describe('Authorization Handler', () => {
     app.use('/authorize', handler);
   });
 
+  afterEach(() => {
+    jest.restoreAllMocks();
+  });
+
   describe('HTTP method validation', () => {
     it('rejects non-GET/POST methods', async () => {
       const response = await supertest(app)
@@ -305,7 +309,6 @@ describe('Authorization Handler', () => {
 
     it('propagates nonce parameter for OpenID Connect flows', async () => {
       const mockProviderWithNonce = jest.spyOn(mockProvider, 'authorize');
-      mockProviderWithNonce.mockClear();
       
       const response = await supertest(app)
         .get('/authorize')
@@ -334,7 +337,6 @@ describe('Authorization Handler', () => {
 
     it('handles authorization without nonce parameter', async () => {
       const mockProviderWithoutNonce = jest.spyOn(mockProvider, 'authorize');
-      mockProviderWithoutNonce.mockClear();
       
       const response = await supertest(app)
         .get('/authorize')
