@jcjeon/integration-cli 0.2.0 → 0.2.1

package/apps/server/src/modules/changelog/changelog.service.spec.ts

@@ -1,531 +0,0 @@
-import * as childProcess from 'child_process';
-import * as fs from 'fs';
-import * as path from 'path';
-
-import { Test, TestingModule } from '@nestjs/testing';
-import { getRepositoryToken } from '@nestjs/typeorm';
-
-import { AgentChangelogEntity } from '../../database/entities/agent-changelog.entity';
-import { GitChangelogService } from './changelog.service';
-
-jest.mock('fs');
-jest.mock('child_process', () => ({
-  execSync: jest.fn(),
-  execFileSync: jest.fn(),
-}));
-
-const mockRepo = () => ({
-  create: jest.fn((dto: any) => ({ ...dto })),
-  save: jest.fn((e: any) => Promise.resolve(Array.isArray(e) ? e : [e])),
-  findOne: jest.fn(),
-  find: jest.fn(),
-});
-
-describe('GitChangelogService', () => {
-  let service: GitChangelogService;
-  let repo: ReturnType<typeof mockRepo>;
-
-  const execSync = childProcess.execSync as jest.Mock;
-  const execFileSync = childProcess.execFileSync as jest.Mock;
-
-  beforeEach(async () => {
-    const module: TestingModule = await Test.createTestingModule({
-      providers: [
-        GitChangelogService,
-        { provide: getRepositoryToken(AgentChangelogEntity), useFactory: mockRepo },
-      ],
-    }).compile();
-
-    service = module.get(GitChangelogService);
-    repo = module.get(getRepositoryToken(AgentChangelogEntity));
-    jest.clearAllMocks();
-  });
-
-  // ─── isGitRepo ───────────────────────────────────────────────────────────
-
-  describe('isGitRepo', () => {
-    it('git repo이면 true를 반환한다', () => {
-      execSync.mockReturnValue('');
-
-      expect(service.isGitRepo('/some/dir')).toBe(true);
-      expect(execSync).toHaveBeenCalledWith('git rev-parse --git-dir', expect.objectContaining({ cwd: '/some/dir' }));
-    });
-
-    it('git repo가 아니면 false를 반환한다', () => {
-      execSync.mockImplementation(() => { throw new Error('not a git repo'); });
-
-      expect(service.isGitRepo('/not/a/repo')).toBe(false);
-    });
-  });
-
-  // ─── getCurrentHead ──────────────────────────────────────────────────────
-
-  describe('getCurrentHead', () => {
-    it('현재 HEAD 커밋 SHA를 반환한다', () => {
-      execSync.mockReturnValue('abc123def456\n');
-
-      const result = service.getCurrentHead('/repo');
-
-      expect(result).toBe('abc123def456');
-    });
-  });
-
-  // ─── getCurrentBranch ────────────────────────────────────────────────────
-
-  describe('getCurrentBranch', () => {
-    it('현재 브랜치명을 반환한다', () => {
-      execSync.mockReturnValue('main\n');
-
-      const result = service.getCurrentBranch('/repo');
-
-      expect(result).toBe('main');
-    });
-
-    it('특수문자를 대시로 치환하고 40자로 자른다', () => {
-      const longBranch = 'feature/some-branch-name-that-is-very-very-long-and-has/slashes';
-      execSync.mockReturnValue(longBranch + '\n');
-
-      const result = service.getCurrentBranch('/repo');
-
-      expect(result).not.toContain('/');
-      expect(result.length).toBeLessThanOrEqual(40);
-    });
-
-    it('에러 발생 시 "unknown"을 반환한다', () => {
-      execSync.mockImplementation(() => { throw new Error('git error'); });
-
-      const result = service.getCurrentBranch('/repo');
-
-      expect(result).toBe('unknown');
-    });
-  });
-
-  // ─── getRepoRoot ─────────────────────────────────────────────────────────
-
-  describe('getRepoRoot', () => {
-    it('git repo의 루트 디렉토리를 반환한다', () => {
-      execSync.mockReturnValue('/path/to/repo\n');
-
-      const result = service.getRepoRoot('/path/to/repo/subdir');
-
-      expect(result).toBe('/path/to/repo');
-    });
-  });
-
-  // ─── mergeAll ────────────────────────────────────────────────────────────
-
-  describe('mergeAll', () => {
-    it('전체 변경사항을 병합하고 성공 결과를 반환한다', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(true);
-      execSync.mockReturnValue('/repo\n'); // getRepoRoot uses execSync
-      execFileSync
-        .mockReturnValueOnce('agent-branch\n') // getCurrentBranch in worktree
-        .mockReturnValueOnce(''); // merge
-
-      const result = service.mergeAll('/tmp/worktrees/agent-1', '/repo');
-
-      expect(result.success).toBe(true);
-      expect(result.message).toBe('전체 병합이 완료되었습니다.');
-    });
-
-    it('worktree 경로가 없으면 실패 메시지를 반환한다', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(false);
-
-      const result = service.mergeAll('/not/exist', '/repo');
-
-      expect(result.success).toBe(false);
-      expect(result.message).toContain('worktree 경로가 존재하지 않습니다');
-    });
-
-    it('merge 실패 시 false를 반환하고 abort를 시도한다', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(true);
-      execSync.mockReturnValue('/repo\n'); // getRepoRoot (main flow + catch block)
-      execFileSync
-        .mockReturnValueOnce('agent-branch\n') // getCurrentBranch in worktree
-        .mockImplementationOnce(() => { throw new Error('merge conflict'); }) // merge fails
-        .mockReturnValueOnce(''); // abort in catch
-
-      const result = service.mergeAll('/tmp/worktrees/agent-1', '/repo');
-
-      expect(result.success).toBe(false);
-    });
-  });
-
-  // ─── mergeFile ───────────────────────────────────────────────────────────
-
-  describe('mergeFile', () => {
-    it('단일 파일을 병합하고 성공 결과를 반환한다', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(true);
-      execSync.mockReturnValue('/repo\n'); // getRepoRoot
-      execFileSync
-        .mockReturnValueOnce('agent-branch\n') // getCurrentBranch in worktree
-        .mockReturnValueOnce(''); // checkout
-
-      const result = service.mergeFile('/tmp/worktrees/agent-1', '/repo', 'src/app.ts');
-
-      expect(result.success).toBe(true);
-      expect(result.message).toContain('src/app.ts');
-    });
-
-    it('worktree 경로가 없으면 실패 메시지를 반환한다', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(false);
-
-      const result = service.mergeFile('/not/exist', '/repo', 'file.ts');
-
-      expect(result.success).toBe(false);
-      expect(result.message).toContain('worktree 경로가 존재하지 않습니다');
-    });
-
-    it('checkout 실패 시 에러 메시지를 반환한다', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(true);
-      execSync.mockReturnValue('/repo\n'); // getRepoRoot
-      execFileSync
-        .mockReturnValueOnce('agent-branch\n') // getCurrentBranch
-        .mockImplementationOnce(() => { throw new Error('file not found'); }); // checkout fails
-
-      const result = service.mergeFile('/tmp/worktrees/agent-1', '/repo', 'missing.ts');
-
-      expect(result.success).toBe(false);
-      expect(result.message).toContain('file not found');
-    });
-
-    it('상위 디렉토리 pathspec은 거부한다', () => {
-      (fs.existsSync as jest.Mock).mockReturnValue(true);
-
-      const result = service.mergeFile('/tmp/worktrees/agent-1', '/repo', '../secret.ts');
-
-      expect(result.success).toBe(false);
-      expect(result.message).toContain('유효하지 않은 파일 경로입니다');
-      expect(execSync).not.toHaveBeenCalled();
-      expect(execFileSync).not.toHaveBeenCalled();
-    });
-  });
-
-  // ─── getLatestRunId ──────────────────────────────────────────────────────
-
-  describe('getLatestRunId', () => {
-    it('가장 최근 runId를 반환한다', async () => {
-      repo.findOne.mockResolvedValue({ runId: 5 });
-
-      const result = await service.getLatestRunId('task-1');
-
-      expect(result).toBe(5);
-      expect(repo.findOne).toHaveBeenCalledWith({
-        where: { taskId: 'task-1' },
-        order: { runId: 'DESC' },
-        select: ['runId'],
-      });
-    });
-
-    it('기록이 없으면 null을 반환한다', async () => {
-      repo.findOne.mockResolvedValue(null);
-
-      const result = await service.getLatestRunId('empty-task');
-
-      expect(result).toBeNull();
-    });
-  });
-
-  // ─── getByTask ───────────────────────────────────────────────────────────
-
-  describe('getByTask', () => {
-    it('task의 changelog를 에이전트별로 그룹핑해서 반환한다', async () => {
-      repo.findOne.mockResolvedValue({ runId: 3 });
-      repo.find.mockResolvedValue([
-        { id: 1, taskId: 'task-1', agentId: 1, runId: 3, filePath: 'src/a.ts', changeType: 'modified', additions: 5, deletions: 2, patch: 'diff...' },
-        { id: 2, taskId: 'task-1', agentId: 1, runId: 3, filePath: 'src/b.ts', changeType: 'added', additions: 10, deletions: 0, patch: 'diff...' },
-        { id: 3, taskId: 'task-1', agentId: 2, runId: 3, filePath: 'src/c.ts', changeType: 'deleted', additions: 0, deletions: 3, patch: 'diff...' },
-      ]);
-
-      const result = await service.getByTask('task-1');
-
-      expect(result).toHaveLength(2);
-      expect(result[0].agentId).toBe(1);
-      expect(result[0].files).toHaveLength(2);
-      expect(result[1].agentId).toBe(2);
-      expect(result[1].files).toHaveLength(1);
-    });
-
-    it('runId를 직접 지정하면 해당 run의 결과만 반환한다', async () => {
-      repo.find.mockResolvedValue([
-        { id: 1, taskId: 'task-1', agentId: 1, runId: 2, filePath: 'src/x.ts', changeType: 'modified', additions: 1, deletions: 1, patch: null },
-      ]);
-
-      const result = await service.getByTask('task-1', 2);
-
-      expect(repo.find).toHaveBeenCalledWith(expect.objectContaining({
-        where: { taskId: 'task-1', runId: 2 },
-      }));
-      expect(result).toHaveLength(1);
-    });
-
-    it('changelog가 없으면 빈 배열을 반환한다', async () => {
-      repo.findOne.mockResolvedValue(null);
-      repo.find.mockResolvedValue([]);
-
-      const result = await service.getByTask('task-1');
-
-      expect(result).toEqual([]);
-    });
-  });
-
-  // ─── directory snapshot changelog ────────────────────────────────────────
-
-  describe('directory snapshot changelog', () => {
-    const dirent = (name: string, type: 'file' | 'dir') => ({
-      name,
-      isDirectory: () => type === 'dir',
-      isFile: () => type === 'file',
-    }) as fs.Dirent;
-
-    it('일반 디렉토리 스냅샷을 생성하고 무거운 폴더를 건너뛴다', () => {
-      (fs.readdirSync as jest.Mock).mockImplementation((dir: string) => {
-        if (dir === '/work') return [dirent('src', 'dir'), dirent('root.txt', 'file'), dirent('node_modules', 'dir')];
-        if (dir === path.join('/work', 'src')) return [dirent('app.ts', 'file')];
-        return [];
-      });
-      (fs.readFileSync as jest.Mock).mockImplementation((file: string) => Buffer.from(`content:${file}`));
-
-      const snapshot = service.createDirectorySnapshot('/work');
-
-      expect(snapshot.has('root.txt')).toBe(true);
-      expect(snapshot.has('src/app.ts')).toBe(true);
-      expect(snapshot.has('node_modules/pkg/index.js')).toBe(false);
-    });
-
-    it('일반 디렉토리 변경사항을 changelog로 저장한다', async () => {
-      const before = new Map([
-        ['src/app.ts', { hash: 'old-hash', content: 'old\n', isBinary: false }],
-        ['src/remove.ts', { hash: 'remove-hash', content: 'remove\n', isBinary: false }],
-      ]);
-      const after = new Map([
-        ['src/app.ts', { hash: 'new-hash', content: 'new\n', isBinary: false }],
-        ['src/add.ts', { hash: 'add-hash', content: 'add\n', isBinary: false }],
-      ]);
-
-      jest.spyOn(service, 'createDirectorySnapshot').mockReturnValue(after);
-      (fs.mkdirSync as jest.Mock).mockReturnValue(undefined);
-      (fs.writeFileSync as jest.Mock).mockReturnValue(undefined);
-
-      const result = await service.captureDirectoryAndSave('task-1', 1, '/work', before, 3);
-
-      expect(result).toBe(3);
-      expect(repo.save).toHaveBeenCalledWith(expect.arrayContaining([
-        expect.objectContaining({ filePath: 'src/add.ts', changeType: 'added', runId: 3 }),
-        expect.objectContaining({ filePath: 'src/app.ts', changeType: 'modified', runId: 3 }),
-        expect.objectContaining({ filePath: 'src/remove.ts', changeType: 'deleted', runId: 3 }),
-      ]));
-      expect(fs.writeFileSync).toHaveBeenCalledWith(
-        expect.stringContaining('agent-1'),
-        expect.stringContaining('diff --git'),
-        'utf8',
-      );
-    });
-
-    it('일반 디렉토리 변경사항이 없으면 DB에 저장하지 않는다', async () => {
-      const before = new Map([
-        ['src/app.ts', { hash: 'same-hash', content: 'same\n', isBinary: false }],
-      ]);
-
-      jest.spyOn(service, 'createDirectorySnapshot').mockReturnValue(before);
-
-      const result = await service.captureDirectoryAndSave('task-1', 1, '/work', before, 3);
-
-      expect(result).toBe(0);
-      expect(repo.save).not.toHaveBeenCalled();
-    });
-  });
-
-  // ─── parseDiff (private, via any 접근) ──────────────────────────────────
-
-  describe('parseDiff', () => {
-    const parseDiff = (diffText: string) => (service as any).parseDiff(diffText);
-
-    it('modified 파일을 파싱한다', () => {
-      const diff = `diff --git a/src/app.ts b/src/app.ts
-index abc..def 100644
---- a/src/app.ts
-+++ b/src/app.ts
-@@ -1,3 +1,4 @@
- const x = 1;
--const y = 2;
-+const y = 3;
-+const z = 4;
-`;
-      const result = parseDiff(diff);
-
-      expect(result).toHaveLength(1);
-      expect(result[0].filePath).toBe('src/app.ts');
-      expect(result[0].changeType).toBe('modified');
-      expect(result[0].additions).toBe(2);
-      expect(result[0].deletions).toBe(1);
-    });
-
-    it('새로 추가된 파일을 파싱한다', () => {
-      const diff = `diff --git a/src/new.ts b/src/new.ts
-new file mode 100644
-index 000..abc
---- /dev/null
-+++ b/src/new.ts
-@@ -0,0 +1,2 @@
-+export const x = 1;
-+export const y = 2;
-`;
-      const result = parseDiff(diff);
-
-      expect(result[0].changeType).toBe('added');
-      expect(result[0].additions).toBe(2);
-      expect(result[0].deletions).toBe(0);
-    });
-
-    it('삭제된 파일을 파싱한다', () => {
-      const diff = `diff --git a/src/old.ts b/src/old.ts
-deleted file mode 100644
-index abc..000
---- a/src/old.ts
-+++ /dev/null
-@@ -1,2 +0,0 @@
--export const x = 1;
--export const y = 2;
-`;
-      const result = parseDiff(diff);
-
-      expect(result[0].changeType).toBe('deleted');
-      expect(result[0].deletions).toBe(2);
-    });
-
-    it('rename된 파일을 파싱한다', () => {
-      const diff = `diff --git a/src/old.ts b/src/renamed.ts
-rename from src/old.ts
-rename to src/renamed.ts
-similarity index 100%
-`;
-      const result = parseDiff(diff);
-
-      expect(result[0].changeType).toBe('renamed');
-      expect(result[0].filePath).toBe('src/renamed.ts');
-    });
-
-    it('여러 파일을 포함한 diff를 파싱한다', () => {
-      const diff = `diff --git a/src/a.ts b/src/a.ts
-index abc..def 100644
---- a/src/a.ts
-+++ b/src/a.ts
-@@ -1 +1,2 @@
- const a = 1;
-+const b = 2;
-diff --git a/src/b.ts b/src/b.ts
-new file mode 100644
---- /dev/null
-+++ b/src/b.ts
-@@ -0,0 +1 @@
-+export default {};
-`;
-      const result = parseDiff(diff);
-
-      expect(result).toHaveLength(2);
-      expect(result[0].filePath).toBe('src/a.ts');
-      expect(result[1].filePath).toBe('src/b.ts');
-    });
-
-    it('빈 diff는 빈 배열을 반환한다', () => {
-      expect(parseDiff('')).toEqual([]);
-    });
-
-    it('200KB 초과 patch는 truncated 처리한다', () => {
-      const longContent = '+' + 'x'.repeat(210_000);
-      const diff = `diff --git a/big.ts b/big.ts
-index abc..def 100644
---- a/big.ts
-+++ b/big.ts
-@@ -1 +1 @@
-${longContent}
-`;
-      const result = parseDiff(diff);
-
-      expect(result[0].patch).toContain('(truncated)');
-    });
-
-    it('+++ / --- 헤더 라인은 추가/삭제 카운트에 포함하지 않는다', () => {
-      const diff = `diff --git a/src/x.ts b/src/x.ts
-index abc..def 100644
---- a/src/x.ts
-+++ b/src/x.ts
-@@ -1,2 +1,2 @@
--old line
-+new line
-`;
-      const result = parseDiff(diff);
-
-      expect(result[0].additions).toBe(1);
-      expect(result[0].deletions).toBe(1);
-    });
-  });
-
-  // ─── captureAndSave ──────────────────────────────────────────────────────
-
-  describe('captureAndSave', () => {
-    it('변경사항을 커밋하고 DB에 저장한 뒤 snapshotSha를 반환한다', async () => {
-      execSync
-        .mockReturnValueOnce('') // git add -A
-        .mockReturnValueOnce('') // git commit
-        .mockReturnValueOnce('newsha123\n') // git rev-parse HEAD
-        .mockReturnValueOnce(`diff --git a/src/x.ts b/src/x.ts
-index abc..def 100644
---- a/src/x.ts
-+++ b/src/x.ts
-@@ -1 +1 @@
--old
-+new
-`); // git diff
-
-      const result = await service.captureAndSave('task-1', 1, '/worktree', 'startsha', 'task commit', 2);
-
-      expect(result).toBe('newsha123');
-      expect(repo.save).toHaveBeenCalled();
-    });
-
-    it('변경사항이 없으면 DB 저장 없이 snapshotSha를 반환한다', async () => {
-      execSync
-        .mockReturnValueOnce('')
-        .mockReturnValueOnce('')
-        .mockReturnValueOnce('newsha123\n')
-        .mockReturnValueOnce('   '); // empty diff
-
-      const result = await service.captureAndSave('task-1', 1, '/worktree', 'startsha', 'task commit');
-
-      expect(result).toBe('newsha123');
-      expect(repo.save).not.toHaveBeenCalled();
-    });
-
-    it('에러 발생 시 null을 반환한다', async () => {
-      execSync.mockImplementation(() => { throw new Error('git error'); });
-
-      const result = await service.captureAndSave('task-1', 1, '/worktree', 'startsha', 'task commit');
-
-      expect(result).toBeNull();
-    });
-  });
-
-  // ─── mergeToMain ─────────────────────────────────────────────────────────
-
-  describe('mergeToMain', () => {
-    it('스냅샷 커밋을 main repo에 merge한다', () => {
-      execSync.mockReturnValue('');
-
-      expect(() => service.mergeToMain('/repo', 'snapshotsha', 1)).not.toThrow();
-      expect(execSync).toHaveBeenCalledWith(
-        expect.stringContaining('merge --no-ff'),
-        expect.anything(),
-      );
-    });
-
-    it('merge 충돌 시 abort를 시도하고 경고만 남긴다', () => {
-      execSync
-        .mockImplementationOnce(() => { throw new Error('conflict'); })
-        .mockReturnValueOnce(''); // abort
-
-      expect(() => service.mergeToMain('/repo', 'sha', 1)).not.toThrow();
-    });
-  });
-});