clean up tests to keep only the higher value ones

Author: jsourcebot

packages/web/src/ee/features/chat/mcp/oauthScopeUtils.test.ts

@@ -13,10 +13,6 @@ describe('normalizeMcpRequestedOAuthScopes', () => {
         ]);
     });
 
-    test('returns an empty array for empty input', () => {
-        expect(normalizeMcpRequestedOAuthScopes([])).toEqual([]);
-    });
-
     test('filters blank strings', () => {
         expect(normalizeMcpRequestedOAuthScopes(['', '  ', 'read'])).toEqual(['read']);
     });
@@ -35,18 +31,6 @@ describe('buildMcpOAuthScopeEntries', () => {
         ]);
     });
 
-    test('enables offline_access by default when present in available scopes', () => {
-        const entries = buildMcpOAuthScopeEntries({
-            availableOAuthScopes: ['offline_access', 'read'],
-            requestedOAuthScopes: ['read'],
-        });
-
-        expect(entries).toEqual([
-            { scope: 'offline_access', enabled: true },
-            { scope: 'read', enabled: true },
-        ]);
-    });
-
     test('enables offline_access by default even when not in requestedOAuthScopes', () => {
         const entries = buildMcpOAuthScopeEntries({
             availableOAuthScopes: ['offline_access', 'read', 'write'],
@@ -104,12 +88,4 @@ describe('getEnabledMcpOAuthScopeNames', () => {
 
         expect(scopes).toEqual(['offline_access', 'read']);
     });
-
-    test('returns an empty array when no scopes are enabled', () => {
-        const scopes = getEnabledMcpOAuthScopeNames([
-            { scope: 'read', enabled: false },
-        ]);
-
-        expect(scopes).toEqual([]);
-    });
 });