@nexical/cli 0.11.0 → 0.11.1

package/test/unit/commands/init.test.ts

@@ -1,153 +1,188 @@
-import { logger, runCommand } from '@nexical/cli-core';
+import { runCommand } from '@nexical/cli-core';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import InitCommand from '../../../src/commands/init.js';
 import * as git from '../../../src/utils/git.js';
 import fs from 'fs-extra';
 
 vi.mock('@nexical/cli-core', async (importOriginal) => {
-    const mod = await importOriginal<typeof import('@nexical/cli-core')>();
-    return {
-        ...mod,
-        runCommand: vi.fn(),
-        logger: { code: vi.fn(), debug: vi.fn(), error: vi.fn(), success: vi.fn(), info: vi.fn(), warn: vi.fn() }
-    }
+  const mod = await importOriginal<typeof import('@nexical/cli-core')>();
+  return {
+    ...mod,
+    runCommand: vi.fn(),
+    logger: {
+      code: vi.fn(),
+      debug: vi.fn(),
+      error: vi.fn(),
+      success: vi.fn(),
+      info: vi.fn(),
+      warn: vi.fn(),
+    },
+  };
 });
 
 vi.mock('../../../src/utils/git.js', () => ({
-    clone: vi.fn(),
-    updateSubmodules: vi.fn(),
-    checkoutOrphan: vi.fn(),
-    addAll: vi.fn(),
-    commit: vi.fn(),
-    deleteBranch: vi.fn(),
-    renameBranch: vi.fn(),
-    removeRemote: vi.fn(),
-    branchExists: vi.fn(),
-    renameRemote: vi.fn(),
-    getRemoteUrl: vi.fn()
+  clone: vi.fn(),
+  updateSubmodules: vi.fn(),
+  checkoutOrphan: vi.fn(),
+  addAll: vi.fn(),
+  commit: vi.fn(),
+  deleteBranch: vi.fn(),
+  renameBranch: vi.fn(),
+  removeRemote: vi.fn(),
+  branchExists: vi.fn(),
+  renameRemote: vi.fn(),
+  getRemoteUrl: vi.fn(),
 }));
 
 vi.mock('fs-extra');
 
 describe('InitCommand', () => {
-    let command: InitCommand;
-    // Spy on process.exit but rely on catching the error if it throws (default vitest behavior)
-    // or mock it to throw a custom error we can check.
-    let mockExit: any;
-
-    beforeEach(() => {
-        vi.clearAllMocks();
-        command = new InitCommand({});
-        vi.spyOn(command, 'error').mockImplementation((() => { }) as any);
-        vi.spyOn(command, 'info').mockImplementation((() => { }) as any);
-        vi.spyOn(command, 'success').mockImplementation((() => { }) as any);
-
-        // Default fs mocks
-        vi.mocked(fs.pathExists as any).mockResolvedValue(false); // Target not exist
-        vi.mocked(fs.mkdir).mockResolvedValue(undefined);
-        vi.mocked(fs.readdir).mockResolvedValue([] as any);
-        vi.mocked(fs.copy).mockResolvedValue(undefined);
-        vi.mocked(fs.ensureDir).mockResolvedValue(undefined);
-
-        // Mock process.exit to throw a known error so we can stop execution and verify it
-        mockExit = vi.spyOn(process, 'exit').mockImplementation((code) => {
-            throw new Error(`Process.exit(${code})`);
-        });
+  let command: InitCommand;
+  // Spy on process.exit but rely on catching the error if it throws (default vitest behavior)
+  // or mock it to throw a custom error we can check.
+  beforeEach(() => {
+    vi.clearAllMocks();
+    command = new InitCommand({});
+    vi.spyOn(command, 'error').mockImplementation(() => {});
+    vi.spyOn(command, 'info').mockImplementation(() => {});
+    vi.spyOn(command, 'success').mockImplementation(() => {});
+
+    // Default fs mocks
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    vi.mocked(fs.pathExists as any).mockResolvedValue(false); // Target not exist
+    vi.mocked(fs.mkdir).mockResolvedValue(undefined);
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    vi.mocked(fs.readdir).mockResolvedValue([] as any);
+    vi.mocked(fs.copy).mockResolvedValue(undefined);
+    vi.mocked(fs.ensureDir).mockResolvedValue(undefined);
+
+    // Mock process.exit to throw a known error so we can stop execution and verify it
+    vi.spyOn(process, 'exit').mockImplementation((code) => {
+      throw new Error(`Process.exit(${code})`);
     });
-
-    afterEach(() => {
-        vi.resetAllMocks();
-    });
-
-    it('should have correct metadata', () => {
-        expect(InitCommand.description).toBeDefined();
-        expect(InitCommand.args).toBeDefined();
-        expect(InitCommand.requiresProject).toBe(false);
-    });
-
-    it('should initialize project with default repo', async () => {
-        const targetDir = 'new-project';
-        await command.run({ directory: targetDir, repo: 'https://default.com/repo' });
-
-        expect(fs.mkdir).toHaveBeenCalledWith(expect.stringContaining(targetDir), { recursive: true });
-
-        // Clone
-        expect(git.clone).toHaveBeenCalledWith('https://default.com/repo.git', expect.stringContaining(targetDir), { recursive: true });
-
-        // Submodules
-        expect(git.updateSubmodules).toHaveBeenCalledWith(expect.stringContaining(targetDir));
-
-        // Npm install
-        expect(runCommand).toHaveBeenCalledWith(
-            'npm install',
-            expect.stringContaining(targetDir)
-        );
-
-        // Remote rename
-        expect(git.renameRemote).toHaveBeenCalledWith('origin', 'upstream', expect.stringContaining(targetDir));
-
-        // Version and Config creation
-        expect(fs.writeFile).toHaveBeenCalledWith(expect.stringContaining('nexical.yaml'), expect.stringContaining('name: new-project'));
-        expect(fs.writeFile).toHaveBeenCalledWith(expect.stringContaining('VERSION'), '0.1.0');
-
-        expect(git.addAll).toHaveBeenCalledWith(expect.stringContaining(targetDir));
-        expect(git.commit).toHaveBeenCalledWith('Initial site commit', expect.stringContaining(targetDir));
-
-        expect(command.success).toHaveBeenCalledWith(expect.stringContaining('successfully'));
-    });
-
-    it('should skip version and config creation if they already exist', async () => {
-        const targetDir = 'existing-files';
-        vi.mocked(fs.pathExists as any).mockImplementation(async (p: string) => {
-            if (p.includes('nexical.yaml')) return true;
-            if (p.includes('VERSION')) return true;
-            return false;
-        });
-
-        await command.run({ directory: targetDir, repo: 'foo' });
-
-        expect(fs.writeFile).not.toHaveBeenCalledWith(expect.stringContaining('nexical.yaml'), expect.anything());
-        expect(fs.writeFile).not.toHaveBeenCalledWith(expect.stringContaining('VERSION'), expect.anything());
-    });
-
-    it('should handle gh@ syntax', async () => {
-        const targetDir = 'gh-project';
-        await command.run({ directory: targetDir, repo: 'gh@nexical/nexical-starter' });
-
-        expect(git.clone).toHaveBeenCalledWith(
-            'https://github.com/nexical/nexical-starter.git',
-            expect.stringContaining(targetDir),
-            { recursive: true }
-        );
+  });
+
+  afterEach(() => {
+    vi.resetAllMocks();
+  });
+
+  it('should have correct metadata', () => {
+    expect(InitCommand.description).toBeDefined();
+    expect(InitCommand.args).toBeDefined();
+    expect(InitCommand.requiresProject).toBe(false);
+  });
+
+  it('should initialize project with default repo', async () => {
+    const targetDir = 'new-project';
+    await command.run({ directory: targetDir, repo: 'https://default.com/repo' });
+
+    expect(fs.mkdir).toHaveBeenCalledWith(expect.stringContaining(targetDir), { recursive: true });
+
+    // Clone
+    expect(git.clone).toHaveBeenCalledWith(
+      'https://default.com/repo.git',
+      expect.stringContaining(targetDir),
+      { recursive: true },
+    );
+
+    // Submodules
+    expect(git.updateSubmodules).toHaveBeenCalledWith(expect.stringContaining(targetDir));
+
+    // Npm install
+    expect(runCommand).toHaveBeenCalledWith('npm install', expect.stringContaining(targetDir));
+
+    // Remote rename
+    expect(git.renameRemote).toHaveBeenCalledWith(
+      'origin',
+      'upstream',
+      expect.stringContaining(targetDir),
+    );
+
+    // Version and Config creation
+    expect(fs.writeFile).toHaveBeenCalledWith(
+      expect.stringContaining('nexical.yaml'),
+      expect.stringContaining('name: new-project'),
+    );
+    expect(fs.writeFile).toHaveBeenCalledWith(expect.stringContaining('VERSION'), '0.1.0');
+
+    expect(git.addAll).toHaveBeenCalledWith(expect.stringContaining(targetDir));
+    expect(git.commit).toHaveBeenCalledWith(
+      'Initial site commit',
+      expect.stringContaining(targetDir),
+    );
+
+    expect(command.success).toHaveBeenCalledWith(expect.stringContaining('successfully'));
+  });
+
+  it('should skip version and config creation if they already exist', async () => {
+    const targetDir = 'existing-files';
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    vi.mocked(fs.pathExists as any).mockImplementation(async (p: string) => {
+      if (p.includes('nexical.yaml')) return true;
+      if (p.includes('VERSION')) return true;
+      return false;
     });
 
-    it('should proceed if directory exists but is empty', async () => {
-        vi.mocked(fs.pathExists as any).mockResolvedValue(true);
-        vi.mocked(fs.readdir).mockResolvedValue([] as any);
-
-        await command.run({ directory: 'empty-dir', repo: 'foo' });
-
-        expect(fs.mkdir).not.toHaveBeenCalled(); // Should assume dir exists
-        expect(git.clone).toHaveBeenCalledWith('foo.git', expect.stringContaining('empty-dir'), { recursive: true });
+    await command.run({ directory: targetDir, repo: 'foo' });
+
+    expect(fs.writeFile).not.toHaveBeenCalledWith(
+      expect.stringContaining('nexical.yaml'),
+      expect.anything(),
+    );
+    expect(fs.writeFile).not.toHaveBeenCalledWith(
+      expect.stringContaining('VERSION'),
+      expect.anything(),
+    );
+  });
+
+  it('should handle gh@ syntax', async () => {
+    const targetDir = 'gh-project';
+    await command.run({ directory: targetDir, repo: 'gh@nexical/nexical-starter' });
+
+    expect(git.clone).toHaveBeenCalledWith(
+      'https://github.com/nexical/nexical-starter.git',
+      expect.stringContaining(targetDir),
+      { recursive: true },
+    );
+  });
+
+  it('should proceed if directory exists but is empty', async () => {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    vi.mocked(fs.pathExists as any).mockResolvedValue(true);
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    vi.mocked(fs.readdir).mockResolvedValue([] as any);
+
+    await command.run({ directory: 'empty-dir', repo: 'foo' });
+
+    expect(fs.mkdir).not.toHaveBeenCalled(); // Should assume dir exists
+    expect(git.clone).toHaveBeenCalledWith('foo.git', expect.stringContaining('empty-dir'), {
+      recursive: true,
     });
+  });
 
-    it('should fail if directory exists and is not empty', async () => {
-        // First exists check for targetDir
-        vi.mocked(fs.pathExists as any).mockResolvedValue(true);
-        vi.mocked(fs.readdir).mockResolvedValue(['file.txt'] as any);
+  it('should fail if directory exists and is not empty', async () => {
+    // First exists check for targetDir
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    vi.mocked(fs.pathExists as any).mockResolvedValue(true);
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    vi.mocked(fs.readdir).mockResolvedValue(['file.txt'] as any);
 
-        await expect(command.run({ directory: 'existing-dir', repo: 'foo' }))
-            .rejects.toThrow('Process.exit(1)');
+    await expect(command.run({ directory: 'existing-dir', repo: 'foo' })).rejects.toThrow(
+      'Process.exit(1)',
+    );
 
-        expect(command.error).toHaveBeenCalledWith(expect.stringContaining('not empty'));
-    });
+    expect(command.error).toHaveBeenCalledWith(expect.stringContaining('not empty'));
+  });
 
-    it('should handle git errors gracefully', async () => {
-        vi.mocked(git.clone).mockRejectedValueOnce(new Error('Git fail'));
+  it('should handle git errors gracefully', async () => {
+    vi.mocked(git.clone).mockRejectedValueOnce(new Error('Git fail'));
 
-        await expect(command.run({ directory: 'fail-project', repo: 'foo' }))
-            .rejects.toThrow('Process.exit(1)');
+    await expect(command.run({ directory: 'fail-project', repo: 'foo' })).rejects.toThrow(
+      'Process.exit(1)',
+    );
 
-        expect(command.error).toHaveBeenCalledWith(expect.stringContaining('Failed to initialize project'));
-    });
+    expect(command.error).toHaveBeenCalledWith(
+      expect.stringContaining('Failed to initialize project'),
+    );
+  });
 });