feat: add tests for chmod-items handler and mock filesystem interactions

Author: shtse8

__tests__/handlers/chmod-items.test.ts

@@ -0,0 +1,68 @@
+import { vi, describe, it, expect, beforeEach } from 'vitest';
+
+// 設置模擬模塊
+vi.mock('node:fs', () => ({
+  promises: {
+    chmod: vi.fn().mockName('fs.chmod')
+  }
+}));
+
+vi.mock('../../src/utils/path-utils', () => ({
+  resolvePath: vi.fn().mockImplementation((path) =>
+    `/project-root/${path}`
+  ).mockName('pathUtils.resolvePath'),
+  PROJECT_ROOT: '/project-root'
+}));
+
+describe('chmod-items handler', () => {
+  let handler: any;
+  let fsMock: any;
+  let pathUtilsMock: any;
+
+  beforeEach(async () => {
+    // 動態導入模擬模塊
+    fsMock = (await import('node:fs')).promises;
+    pathUtilsMock = await import('../../src/utils/path-utils');
+    
+    // 重置模擬
+    vi.resetAllMocks();
+    
+    // 設置默認模擬實現
+    pathUtilsMock.resolvePath.mockImplementation((path: string) => 
+      `/project-root/${path}`
+    );
+    fsMock.chmod.mockResolvedValue(undefined);
+    
+    // 動態導入處理程序
+    const { chmodItemsToolDefinition } = await import('../../src/handlers/chmod-items');
+    handler = chmodItemsToolDefinition.handler;
+  });
+
+  it('should change permissions for valid paths', async () => {
+    const result = await handler({
+      paths: ['file1.txt', 'dir/file2.txt'],
+      mode: '755'
+    });
+
+    expect(fsMock.chmod).toHaveBeenCalledTimes(2);
+    expect(JSON.parse(result.content[0].text)).toEqual([
+      { path: 'file1.txt', mode: '755', success: true },
+      { path: 'dir/file2.txt', mode: '755', success: true }
+    ]);
+  });
+
+  it('should handle multiple operations with mixed results', async () => {
+    fsMock.chmod
+      .mockResolvedValueOnce(undefined)
+      .mockRejectedValueOnce({ code: 'EPERM' });
+
+    const result = await handler({
+      paths: ['file1.txt', 'file2.txt'],
+      mode: '755'
+    });
+
+    const output = JSON.parse(result.content[0].text);
+    expect(output[0].success).toBe(true);
+    expect(output[1].success).toBe(false);
+  });
+});

__tests__/utils/stats-utils.test.ts

@@ -0,0 +1,81 @@
+import { describe, it, expect } from 'vitest';
+import { formatStats, FormattedStats } from '../../src/utils/stats-utils';
+
+function makeMockStats(partial: Partial<Record<keyof FormattedStats, any>> = {}): any {
+  // Provide default values and allow overrides
+  return {
+    isFile: () => partial.isFile ?? true,
+    isDirectory: () => partial.isDirectory ?? false,
+    isSymbolicLink: () => partial.isSymbolicLink ?? false,
+    size: partial.size ?? 1234,
+    atime: partial.atime ?? new Date('2024-01-01T01:02:03.000Z'),
+    mtime: partial.mtime ?? new Date('2024-01-02T01:02:03.000Z'),
+    ctime: partial.ctime ?? new Date('2024-01-03T01:02:03.000Z'),
+    birthtime: partial.birthtime ?? new Date('2024-01-04T01:02:03.000Z'),
+    mode: partial.mode ?? 0o755,
+    uid: partial.uid ?? 1000,
+    gid: partial.gid ?? 1000,
+  };
+}
+
+describe('formatStats', () => {
+  it('formats a regular file', () => {
+    const stats = makeMockStats({ isFile: true, isDirectory: false, isSymbolicLink: false, mode: 0o644 });
+    const result = formatStats('foo\\bar.txt', '/abs/foo/bar.txt', stats as any);
+    expect(result).toEqual({
+      path: 'foo/bar.txt',
+      isFile: true,
+      isDirectory: false,
+      isSymbolicLink: false,
+      size: 1234,
+      atime: '2024-01-01T01:02:03.000Z',
+      mtime: '2024-01-02T01:02:03.000Z',
+      ctime: '2024-01-03T01:02:03.000Z',
+      birthtime: '2024-01-04T01:02:03.000Z',
+      mode: '644',
+      uid: 1000,
+      gid: 1000,
+    });
+  });
+
+  it('formats a directory', () => {
+    const stats = makeMockStats({ isFile: false, isDirectory: true, isSymbolicLink: false, mode: 0o755 });
+    const result = formatStats('dir\\', '/abs/dir', stats as any);
+    expect(result.isDirectory).toBe(true);
+    expect(result.isFile).toBe(false);
+    expect(result.mode).toBe('755');
+  });
+
+  it('formats a symbolic link', () => {
+    const stats = makeMockStats({ isFile: false, isDirectory: false, isSymbolicLink: true, mode: 0o777 });
+    const result = formatStats('link', '/abs/link', stats as any);
+    expect(result.isSymbolicLink).toBe(true);
+    expect(result.mode).toBe('777');
+  });
+
+  it('pads mode with leading zeros', () => {
+    const stats = makeMockStats({ mode: 0o7 });
+    const result = formatStats('file', '/abs/file', stats as any);
+    expect(result.mode).toBe('007');
+  });
+
+  it('converts all date fields to ISO string', () => {
+    const stats = makeMockStats({
+      atime: new Date('2020-01-01T00:00:00.000Z'),
+      mtime: new Date('2020-01-02T00:00:00.000Z'),
+      ctime: new Date('2020-01-03T00:00:00.000Z'),
+      birthtime: new Date('2020-01-04T00:00:00.000Z'),
+    });
+    const result = formatStats('file', '/abs/file', stats as any);
+    expect(result.atime).toBe('2020-01-01T00:00:00.000Z');
+    expect(result.mtime).toBe('2020-01-02T00:00:00.000Z');
+    expect(result.ctime).toBe('2020-01-03T00:00:00.000Z');
+    expect(result.birthtime).toBe('2020-01-04T00:00:00.000Z');
+  });
+
+  it('replaces backslashes in path with forward slashes', () => {
+    const stats = makeMockStats();
+    const result = formatStats('foo\\bar\\baz.txt', '/abs/foo/bar/baz.txt', stats as any);
+    expect(result.path).toBe('foo/bar/baz.txt');
+  });
+});