@sanity/cli 6.0.0-alpha.3 → 6.0.0-alpha.5

package/dist/commands/__tests__/debug.test.js

@@ -1,28 +1,36 @@
 import { runCommand } from '@oclif/test';
-import { findProjectRoot, getCliConfig, getCliToken, getConfig } from '@sanity/cli-core';
+import { getCliToken, getConfig } from '@sanity/cli-core';
 import { mockApi, testCommand } from '@sanity/cli-test';
 import nock from 'nock';
 import { afterEach, describe, expect, test, vi } from 'vitest';
 import { findSanityModulesVersions } from '../../actions/versions/findSanityModulesVersions.js';
 import { Debug } from '../debug.js';
-// Mock CLI core functions using relative paths
-vi.mock('../../../../cli-core/src/services/getCliToken.js', ()=>({
-        getCliToken: vi.fn()
-    }));
-vi.mock('../../../../cli-core/src/services/cliUserConfig.js', ()=>({
+vi.mock('@sanity/cli-core', async ()=>{
+    const actual = await vi.importActual('@sanity/cli-core');
+    return {
+        ...actual,
+        getCliToken: vi.fn(),
         getConfig: vi.fn()
-    }));
-vi.mock('../../../../cli-core/src/config/findProjectRoot.js', ()=>({
-        findProjectRoot: vi.fn()
-    }));
-// Mock the CLI config function like in manage.test.ts
-vi.mock('../../../../cli-core/src/config/cli/getCliConfig.js', ()=>({
-        getCliConfig: vi.fn()
-    }));
-// Mock version-related functions
+    };
+});
 vi.mock('../../actions/versions/findSanityModulesVersions.js', ()=>({
         findSanityModulesVersions: vi.fn()
     }));
+const defaultProjectRoot = {
+    directory: '/test/project',
+    path: '/test/project/sanity.cli.ts',
+    type: 'studio'
+};
+const defaultCliConfig = {
+    api: {
+        projectId: 'project123'
+    }
+};
+const defaultMocks = {
+    cliConfig: defaultCliConfig,
+    projectRoot: defaultProjectRoot,
+    token: 'mock-auth-token'
+};
 afterEach(()=>{
     vi.clearAllMocks();
     const pending = nock.pendingMocks();
@@ -53,7 +61,6 @@ describe('#debug', ()=>{
     `);
     });
     test('shows debug information with authentication and config details', async ()=>{
-        // Mock authentication
         vi.mocked(getCliToken).mockResolvedValue('mock-auth-token');
         vi.mocked(getConfig).mockImplementation(async (key)=>{
             if (key === 'authToken') return 'mock-auth-token';
@@ -66,19 +73,6 @@ describe('#debug', ()=>{
             };
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                dataset: 'production',
-                projectId: 'project123'
-            }
-        });
-        // Mock version info with some packages
         vi.mocked(findSanityModulesVersions).mockResolvedValue([
             {
                 declared: '^3.0.0',
@@ -99,8 +93,17 @@ describe('#debug', ()=>{
                 needsUpdate: true
             }
         ]);
-        const { stdout } = await testCommand(Debug, []);
-        // Check that basic debug output structure is present
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: {
+                ...defaultMocks,
+                cliConfig: {
+                    api: {
+                        dataset: 'production',
+                        projectId: 'project123'
+                    }
+                }
+            }
+        });
         expect(stdout).toContain('User:');
         expect(stdout).toContain('Authentication:');
         expect(stdout).toContain('<redacted>');
@@ -112,82 +115,61 @@ describe('#debug', ()=>{
         expect(stdout).toContain('(latest: 3.1.0)');
     });
     test('shows redacted auth token by default', async ()=>{
-        // Mock basic authentication
         vi.mocked(getCliToken).mockResolvedValue('mock-auth-token');
         vi.mocked(getConfig).mockImplementation(async (key)=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock minimal project setup
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('<redacted>');
         expect(stdout).toContain('(run with --secrets to reveal token)');
         expect(stdout).not.toContain('mock-auth-token');
     });
     test('shows actual auth token with --secrets flag', async ()=>{
-        // Mock authentication with specific token
         vi.mocked(getCliToken).mockResolvedValue('secret-token-12345');
         vi.mocked(getConfig).mockImplementation(async (key)=>{
             if (key === 'authToken') return 'secret-token-12345';
             return undefined;
         });
-        // Mock minimal project setup
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         const { stdout } = await testCommand(Debug, [
             '--secrets'
-        ]);
+        ], {
+            mocks: {
+                ...defaultMocks,
+                token: 'secret-token-12345'
+            }
+        });
         expect(stdout).toContain('secret-token-12345');
         expect(stdout).not.toContain('<redacted>');
         expect(stdout).not.toContain('(run with --secrets to reveal token)');
     });
     test('handles unauthenticated user', async ()=>{
-        // Mock no authentication
         vi.mocked(getCliToken).mockResolvedValue(undefined);
         vi.mocked(getConfig).mockImplementation(async ()=>undefined);
-        const { error } = await testCommand(Debug, []);
-        // When not authenticated, the command should error trying to get client
-        expect(error?.message).toContain('Failed to gather debug information');
+        vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
+        const { error, stdout } = await testCommand(Debug, [], {
+            mocks: {
+                ...defaultMocks,
+                token: undefined
+            }
+        });
+        // When not authenticated, either the command errors or shows "Not logged in" message
+        if (error) {
+            expect(error.message).toContain('Failed to gather debug information');
+        } else {
+            expect(stdout).toContain('Not logged in');
+        }
     });
     test('shows package versions with update information', async ()=>{
-        // Mock basic authentication
         vi.mocked(getCliToken).mockResolvedValue('mock-auth-token');
         vi.mocked(getConfig).mockImplementation(async (key)=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project setup
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
-        // Mock version info with mixed update statuses
         vi.mocked(findSanityModulesVersions).mockResolvedValue([
             {
                 declared: '^3.0.0',
@@ -217,7 +199,9 @@ describe('#debug', ()=>{
                 needsUpdate: true
             }
         ]);
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('Package versions:');
         expect(stdout).toContain('sanity');
         expect(stdout).toContain('3.0.0');
@@ -227,7 +211,9 @@ describe('#debug', ()=>{
     });
     test('handles errors gracefully', async ()=>{
         vi.mocked(getCliToken).mockRejectedValue(new Error('Auth system unavailable'));
-        const { error } = await testCommand(Debug, []);
+        const { error } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(error?.message).toContain('Failed to gather debug information');
         expect(error?.message).toContain('Auth system unavailable');
     });
@@ -238,17 +224,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration but no project info from API
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -264,7 +239,9 @@ describe('#debug', ()=>{
             apiVersion: 'v2025-08-06',
             uri: '/projects/project123'
         }).reply(404);
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('User:');
         expect(stdout).toContain("Email: 'test@example.com'");
         expect(stdout).toContain("ID: 'user123'");
@@ -277,17 +254,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -317,7 +283,9 @@ describe('#debug', ()=>{
             ],
             studioHost: 'test-project'
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('Project:');
         expect(stdout).toContain("Display name: 'Test Project'");
         expect(stdout).toContain("ID: 'project123'");
@@ -327,21 +295,20 @@ describe('#debug', ()=>{
         // Mock no authentication
         vi.mocked(getCliToken).mockResolvedValue(undefined);
         vi.mocked(getConfig).mockImplementation(async ()=>undefined);
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Command should fail when no auth token is present since it requires authentication
-        const { error } = await testCommand(Debug, []);
-        expect(error?.message).toContain('Failed to gather debug information');
+        const { error, stdout } = await testCommand(Debug, [], {
+            mocks: {
+                ...defaultMocks,
+                token: undefined
+            }
+        });
+        // When not authenticated, either the command errors or shows "Not logged in" message
+        if (error) {
+            expect(error.message).toContain('Failed to gather debug information');
+        } else {
+            expect(stdout).toContain('Not logged in');
+        }
     });
     test('handles case when no project config is present', async ()=>{
         // Mock authentication
@@ -350,16 +317,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration with no projectId (invalid config)
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -371,7 +328,15 @@ describe('#debug', ()=>{
             name: 'Test User'
         });
         // No project API mock needed since no valid projectId
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: {
+                ...defaultMocks,
+                cliConfig: {
+                    api: {
+                    }
+                }
+            }
+        });
         expect(stdout).toContain('Global config');
         expect(stdout).toContain('Project config');
         expect(stdout).toContain('Missing required "api.projectId" key');
@@ -383,17 +348,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         // Mock findSanityModulesVersions to return empty array (no versions)
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
@@ -424,26 +378,19 @@ describe('#debug', ()=>{
             ],
             studioHost: 'test-project'
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('Global config');
         expect(stdout).toContain('Package versions:');
     // Should show the heading but no packages since the array is empty
     });
     test('handles error case with unknown error type', async ()=>{
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         // Mock getCliToken to throw a non-Error object to trigger unknown error path
         vi.mocked(getCliToken).mockRejectedValue('string error');
-        const { error } = await testCommand(Debug, []);
+        const { error } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(error).toBeTruthy();
         expect(error?.message).toContain('Failed to gather debug information');
         expect(error?.message).toContain('Unknown error');
@@ -453,17 +400,6 @@ describe('#debug', ()=>{
         vi.mocked(getCliToken).mockResolvedValue('mock-auth-token');
         // Mock getConfig to throw an error
         vi.mocked(getConfig).mockRejectedValue(new Error('Config access error'));
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -484,7 +420,9 @@ describe('#debug', ()=>{
             members: [],
             studioHost: 'test-project'
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         // Should continue to work despite global config error
         expect(stdout).toContain('Global config');
         expect(stdout).toContain('User:');
@@ -497,17 +435,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return an error
         mockApi({
@@ -516,7 +443,9 @@ describe('#debug', ()=>{
         }).reply(500, {
             error: 'Internal server error'
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('User:');
         // Should show error message in red
         expect(stdout).toMatch(/Request failed with status code 500|Failed to fetch user info|Internal server error/);
@@ -528,17 +457,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -556,7 +474,9 @@ describe('#debug', ()=>{
         }).reply(404, {
             error: 'Project not found'
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('User:');
         expect(stdout).toContain("Email: 'test@example.com'");
         // Should not contain Project section since it failed to load
@@ -569,17 +489,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -597,7 +506,9 @@ describe('#debug', ()=>{
         }).reply(200, ()=>{
             return null;
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('User:');
         expect(stdout).toContain("Email: 'test@example.com'");
         // Project error is handled internally but not displayed to user
@@ -610,17 +521,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -640,7 +540,9 @@ describe('#debug', ()=>{
             id: 'project123',
             studioHost: 'test-project'
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('Project:');
         expect(stdout).toContain("Display name: 'Test Project'");
         expect(stdout).toContain("Roles: [ '<none>' ]");
@@ -652,17 +554,6 @@ describe('#debug', ()=>{
             if (key === 'authToken') return 'mock-auth-token';
             return undefined;
         });
-        // Mock project configuration
-        vi.mocked(findProjectRoot).mockResolvedValue({
-            directory: '/test/project',
-            path: '/test/project/sanity.cli.ts',
-            type: 'studio'
-        });
-        vi.mocked(getCliConfig).mockResolvedValue({
-            api: {
-                projectId: 'project123'
-            }
-        });
         vi.mocked(findSanityModulesVersions).mockResolvedValue([]);
         // Mock the /me API endpoint to return user info
         mockApi({
@@ -687,7 +578,9 @@ describe('#debug', ()=>{
             ],
             studioHost: 'test-project'
         });
-        const { stdout } = await testCommand(Debug, []);
+        const { stdout } = await testCommand(Debug, [], {
+            mocks: defaultMocks
+        });
         expect(stdout).toContain('Project:');
         expect(stdout).toContain("Display name: 'Test Project'");
         expect(stdout).toContain("Roles: [ '<none>' ]");