@sk8metal/michi-cli 0.0.9 → 0.1.0

package/scripts/__tests__/spec-impl-workflow.test.ts

@@ -0,0 +1,429 @@
+/**
+ * spec-impl-workflow.ts のテスト
+ */
+
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+
+// モジュールのモック
+vi.mock('dotenv', () => ({ config: vi.fn() }));
+
+// JIRAClientをモック
+const mockTransitionIssue = vi.fn();
+const mockAddComment = vi.fn();
+
+class MockJIRAClient {
+  transitionIssue = mockTransitionIssue;
+  addComment = mockAddComment;
+}
+
+vi.mock('../jira-sync.js', () => ({
+  JIRAClient: MockJIRAClient,
+}));
+
+// config-loaderをモック
+vi.mock('../utils/config-loader.js', () => ({
+  getConfig: vi.fn(() => ({
+    jira: {
+      statusMapping: {
+        inProgress: 'In Progress',
+        readyForReview: 'Ready for Review',
+      },
+    },
+  })),
+}));
+
+// spec-loaderをモック
+const mockGetJiraInfoFromSpec = vi.fn();
+const mockCheckJiraInfoStatus = vi.fn();
+
+vi.mock('../utils/spec-loader.js', () => ({
+  getJiraInfoFromSpec: (...args: unknown[]) => mockGetJiraInfoFromSpec(...args),
+  checkJiraInfoStatus: (...args: unknown[]) => mockCheckJiraInfoStatus(...args),
+}));
+
+// Octokitをモック
+const mockPullsCreate = vi.fn();
+
+class MockOctokit {
+  pulls = {
+    create: mockPullsCreate,
+  };
+}
+
+vi.mock('@octokit/rest', () => ({
+  Octokit: MockOctokit,
+}));
+
+describe('spec-impl-workflow', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    // 環境変数の設定
+    process.env.ATLASSIAN_URL = 'https://test.atlassian.net';
+    process.env.ATLASSIAN_EMAIL = 'test@example.com';
+    process.env.ATLASSIAN_API_TOKEN = 'test-token';
+    process.env.GITHUB_TOKEN = 'github-token';
+    process.env.GITHUB_REPO = 'owner/repo';
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+    delete process.env.ATLASSIAN_URL;
+    delete process.env.ATLASSIAN_EMAIL;
+    delete process.env.ATLASSIAN_API_TOKEN;
+    delete process.env.GITHUB_TOKEN;
+    delete process.env.GITHUB_REPO;
+  });
+
+  // ==========================================================================
+  // 新 API (runSpecImplStart, runSpecImplComplete)
+  // ==========================================================================
+
+  describe('runSpecImplStart', () => {
+    it('spec.json から JIRA 情報を取得して Epic と Story を進行中に移動', async () => {
+      mockGetJiraInfoFromSpec.mockReturnValue({
+        epicKey: 'PROJ-123',
+        storyKeys: ['PROJ-124'],
+        firstStoryKey: 'PROJ-124',
+        projectKey: 'PROJ',
+        epicUrl: 'https://test.atlassian.net/browse/PROJ-123',
+      });
+      mockCheckJiraInfoStatus.mockReturnValue({
+        hasJiraInfo: true,
+        hasEpic: true,
+        hasStories: true,
+        missing: [],
+      });
+      mockTransitionIssue.mockResolvedValue(undefined);
+
+      const { runSpecImplStart } = await import('../spec-impl-workflow.js');
+
+      const result = await runSpecImplStart({
+        featureName: 'user-auth',
+      });
+
+      expect(mockGetJiraInfoFromSpec).toHaveBeenCalledWith(
+        'user-auth',
+        process.cwd(),
+      );
+      expect(mockTransitionIssue).toHaveBeenCalledWith(
+        'PROJ-123',
+        'In Progress',
+      );
+      expect(mockTransitionIssue).toHaveBeenCalledWith(
+        'PROJ-124',
+        'In Progress',
+      );
+      expect(result.jiraInfo).toBeTruthy();
+      expect(result.jiraInfo?.epicKey).toBe('PROJ-123');
+    });
+
+    it('skipJira=true の場合は JIRA 連携をスキップ', async () => {
+      mockGetJiraInfoFromSpec.mockReturnValue({
+        epicKey: null,
+        storyKeys: [],
+        firstStoryKey: null,
+        projectKey: null,
+        epicUrl: null,
+      });
+      mockCheckJiraInfoStatus.mockReturnValue({
+        hasJiraInfo: false,
+        hasEpic: false,
+        hasStories: false,
+        missing: ['Epic', 'Story'],
+      });
+
+      const { runSpecImplStart } = await import('../spec-impl-workflow.js');
+
+      const result = await runSpecImplStart({
+        featureName: 'user-auth',
+        skipJira: true,
+      });
+
+      expect(mockTransitionIssue).not.toHaveBeenCalled();
+      expect(result.jiraInfo).toBeNull();
+    });
+
+    it('JIRA 情報がない場合は JiraInfoNotFoundError をスロー', async () => {
+      mockGetJiraInfoFromSpec.mockReturnValue({
+        epicKey: null,
+        storyKeys: [],
+        firstStoryKey: null,
+        projectKey: null,
+        epicUrl: null,
+      });
+      mockCheckJiraInfoStatus.mockReturnValue({
+        hasJiraInfo: false,
+        hasEpic: false,
+        hasStories: false,
+        missing: ['Epic', 'Story'],
+      });
+
+      const { runSpecImplStart, JiraInfoNotFoundError } = await import(
+        '../spec-impl-workflow.js'
+      );
+
+      await expect(
+        runSpecImplStart({ featureName: 'user-auth' }),
+      ).rejects.toThrow(JiraInfoNotFoundError);
+    });
+  });
+
+  describe('runSpecImplComplete', () => {
+    beforeEach(() => {
+      mockPullsCreate.mockResolvedValue({
+        data: {
+          html_url: 'https://github.com/owner/repo/pull/42',
+        },
+      });
+    });
+
+    it('PR 作成、ステータス変更、コメント追加を実行', async () => {
+      mockGetJiraInfoFromSpec.mockReturnValue({
+        epicKey: 'PROJ-123',
+        storyKeys: ['PROJ-124'],
+        firstStoryKey: 'PROJ-124',
+        projectKey: 'PROJ',
+        epicUrl: 'https://test.atlassian.net/browse/PROJ-123',
+      });
+      mockCheckJiraInfoStatus.mockReturnValue({
+        hasJiraInfo: true,
+        hasEpic: true,
+        hasStories: true,
+        missing: [],
+      });
+      mockTransitionIssue.mockResolvedValue(undefined);
+      mockAddComment.mockResolvedValue(undefined);
+
+      const { runSpecImplComplete } = await import('../spec-impl-workflow.js');
+
+      const result = await runSpecImplComplete({
+        featureName: 'user-auth',
+      });
+
+      // PR 作成
+      expect(mockPullsCreate).toHaveBeenCalledWith({
+        owner: 'owner',
+        repo: 'repo',
+        title: 'feat: user-auth',
+        body: expect.stringContaining('PROJ-123'),
+        head: 'feature/user-auth',
+        base: 'main',
+      });
+
+      // ステータス変更
+      expect(mockTransitionIssue).toHaveBeenCalledWith(
+        'PROJ-123',
+        'Ready for Review',
+      );
+      expect(mockTransitionIssue).toHaveBeenCalledWith(
+        'PROJ-124',
+        'Ready for Review',
+      );
+
+      // コメント追加
+      expect(mockAddComment).toHaveBeenCalledWith(
+        'PROJ-123',
+        expect.stringContaining('https://github.com/owner/repo/pull/42'),
+      );
+
+      expect(result.prUrl).toBe('https://github.com/owner/repo/pull/42');
+    });
+
+    it('skipJira=true の場合は JIRA 連携をスキップしつつ PR は作成', async () => {
+      mockGetJiraInfoFromSpec.mockReturnValue({
+        epicKey: null,
+        storyKeys: [],
+        firstStoryKey: null,
+        projectKey: null,
+        epicUrl: null,
+      });
+      mockCheckJiraInfoStatus.mockReturnValue({
+        hasJiraInfo: false,
+        hasEpic: false,
+        hasStories: false,
+        missing: ['Epic', 'Story'],
+      });
+
+      const { runSpecImplComplete } = await import('../spec-impl-workflow.js');
+
+      const result = await runSpecImplComplete({
+        featureName: 'user-auth',
+        skipJira: true,
+      });
+
+      expect(mockPullsCreate).toHaveBeenCalled();
+      expect(mockTransitionIssue).not.toHaveBeenCalled();
+      expect(mockAddComment).not.toHaveBeenCalled();
+      expect(result.prUrl).toBe('https://github.com/owner/repo/pull/42');
+    });
+
+    it('カスタムブランチ名を使用できる', async () => {
+      mockGetJiraInfoFromSpec.mockReturnValue({
+        epicKey: 'PROJ-123',
+        storyKeys: [],
+        firstStoryKey: null,
+        projectKey: 'PROJ',
+        epicUrl: null,
+      });
+      mockCheckJiraInfoStatus.mockReturnValue({
+        hasJiraInfo: true,
+        hasEpic: true,
+        hasStories: false,
+        missing: ['Story'],
+      });
+      mockTransitionIssue.mockResolvedValue(undefined);
+
+      const { runSpecImplComplete } = await import('../spec-impl-workflow.js');
+
+      await runSpecImplComplete({
+        featureName: 'user-auth',
+        branchName: 'custom/branch-name',
+      });
+
+      expect(mockPullsCreate).toHaveBeenCalledWith(
+        expect.objectContaining({
+          head: 'custom/branch-name',
+        }),
+      );
+    });
+  });
+
+  describe('JiraInfoNotFoundError', () => {
+    it('正しいエラーメッセージを持つ', async () => {
+      const { JiraInfoNotFoundError } = await import(
+        '../spec-impl-workflow.js'
+      );
+
+      const error = new JiraInfoNotFoundError('user-auth', ['Epic', 'Story']);
+
+      expect(error.name).toBe('JiraInfoNotFoundError');
+      expect(error.featureName).toBe('user-auth');
+      expect(error.missing).toEqual(['Epic', 'Story']);
+      expect(error.message).toContain('user-auth');
+      expect(error.message).toContain('Epic, Story');
+      expect(error.message).toContain('michi jira:sync');
+    });
+  });
+
+  // ==========================================================================
+  // 旧 API（後方互換性のため維持、非推奨）
+  // ==========================================================================
+
+  describe('onSpecImplStart (deprecated)', () => {
+    it('JIRAを進行中に移動する', async () => {
+      mockTransitionIssue.mockResolvedValueOnce(undefined);
+
+      const { onSpecImplStart } = await import('../spec-impl-workflow.js');
+
+      await onSpecImplStart({
+        featureName: 'user-auth',
+        jiraKey: 'PROJ-123',
+      });
+
+      expect(mockTransitionIssue).toHaveBeenCalledWith(
+        'PROJ-123',
+        'In Progress',
+      );
+    });
+
+    it('JIRA認証情報が不足している場合はエラー', async () => {
+      delete process.env.ATLASSIAN_API_TOKEN;
+
+      // モジュールを再インポート
+      vi.resetModules();
+      const { onSpecImplStart } = await import('../spec-impl-workflow.js');
+
+      await expect(
+        onSpecImplStart({
+          featureName: 'user-auth',
+          jiraKey: 'PROJ-123',
+        }),
+      ).rejects.toThrow(/Missing JIRA credentials/);
+    });
+  });
+
+  describe('onSpecImplEnd', () => {
+    beforeEach(() => {
+      // PRの作成をモック
+      mockPullsCreate.mockResolvedValue({
+        data: {
+          html_url: 'https://github.com/owner/repo/pull/42',
+        },
+      });
+    });
+
+    it('PR作成、ステータス変更、コメント追加を実行', async () => {
+      mockTransitionIssue.mockResolvedValueOnce(undefined);
+      mockAddComment.mockResolvedValueOnce(undefined);
+
+      const { onSpecImplEnd } = await import('../spec-impl-workflow.js');
+
+      const result = await onSpecImplEnd({
+        featureName: 'user-auth',
+        jiraKey: 'PROJ-123',
+      });
+
+      // PR URLが返される
+      expect(result.prUrl).toBe('https://github.com/owner/repo/pull/42');
+
+      // PR作成の検証
+      expect(mockPullsCreate).toHaveBeenCalledWith({
+        owner: 'owner',
+        repo: 'repo',
+        title: 'feat: user-auth',
+        body: expect.stringContaining('PROJ-123'),
+        head: 'feature/user-auth',
+        base: 'main',
+      });
+
+      // JIRAステータス変更の検証
+      expect(mockTransitionIssue).toHaveBeenCalledWith(
+        'PROJ-123',
+        'Ready for Review',
+      );
+
+      // コメント追加の検証
+      expect(mockAddComment).toHaveBeenCalledWith(
+        'PROJ-123',
+        expect.stringContaining('https://github.com/owner/repo/pull/42'),
+      );
+    });
+
+    it('カスタムブランチ名を使用できる', async () => {
+      mockTransitionIssue.mockResolvedValueOnce(undefined);
+      mockAddComment.mockResolvedValueOnce(undefined);
+
+      const { onSpecImplEnd } = await import('../spec-impl-workflow.js');
+
+      await onSpecImplEnd({
+        featureName: 'user-auth',
+        jiraKey: 'PROJ-123',
+        branchName: 'custom/branch-name',
+      });
+
+      expect(mockPullsCreate).toHaveBeenCalledWith(
+        expect.objectContaining({
+          head: 'custom/branch-name',
+        }),
+      );
+    });
+
+    it('GitHub認証情報が不足している場合はエラー', async () => {
+      delete process.env.GITHUB_TOKEN;
+
+      vi.resetModules();
+      const { onSpecImplEnd } = await import('../spec-impl-workflow.js');
+
+      await expect(
+        onSpecImplEnd({
+          featureName: 'user-auth',
+          jiraKey: 'PROJ-123',
+        }),
+      ).rejects.toThrow(/Missing GitHub credentials/);
+    });
+
+    // 注意: 旧 API (onSpecImplEnd) は JIRA エラーをキャッチせずスローします。
+    // 新 API (runSpecImplComplete) は JIRA エラーを警告として扱い、PR URL を返します。
+    // 旧 API を使用する場合は、呼び出し側でエラーハンドリングが必要です。
+  });
+});

package/scripts/__tests__/spec-loader.test.ts

@@ -0,0 +1,199 @@
+/**
+ * spec-loader.ts のテスト
+ */
+
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+
+// spec-updater をモック
+vi.mock('../utils/spec-updater.js', () => ({
+  loadSpecJson: vi.fn(),
+}));
+
+import { loadSpecJson } from '../utils/spec-updater.js';
+import {
+  getJiraInfoFromSpec,
+  checkJiraInfoStatus,
+  canIntegrateWithJira,
+  type JiraInfo,
+  type JiraInfoStatus,
+} from '../utils/spec-loader.js';
+
+const mockLoadSpecJson = vi.mocked(loadSpecJson);
+
+describe('spec-loader', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  describe('getJiraInfoFromSpec', () => {
+    it('すべての JIRA 情報がある場合に正しく取得する', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: {
+          epicKey: 'PROJ-123',
+          storyKeys: ['PROJ-124', 'PROJ-125'],
+          projectKey: 'PROJ',
+          epicUrl: 'https://test.atlassian.net/browse/PROJ-123',
+        },
+      });
+
+      const result = getJiraInfoFromSpec('user-auth');
+
+      expect(mockLoadSpecJson).toHaveBeenCalledWith('user-auth', process.cwd());
+      expect(result).toEqual<JiraInfo>({
+        epicKey: 'PROJ-123',
+        storyKeys: ['PROJ-124', 'PROJ-125'],
+        firstStoryKey: 'PROJ-124',
+        projectKey: 'PROJ',
+        epicUrl: 'https://test.atlassian.net/browse/PROJ-123',
+      });
+    });
+
+    it('JIRA 情報がない場合は null を返す', () => {
+      mockLoadSpecJson.mockReturnValue({});
+
+      const result = getJiraInfoFromSpec('user-auth');
+
+      expect(result).toEqual<JiraInfo>({
+        epicKey: null,
+        storyKeys: [],
+        firstStoryKey: null,
+        projectKey: null,
+        epicUrl: null,
+      });
+    });
+
+    it('Story のみの場合', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: {
+          storyKeys: ['PROJ-124'],
+        },
+      });
+
+      const result = getJiraInfoFromSpec('user-auth');
+
+      expect(result).toEqual<JiraInfo>({
+        epicKey: null,
+        storyKeys: ['PROJ-124'],
+        firstStoryKey: 'PROJ-124',
+        projectKey: null,
+        epicUrl: null,
+      });
+    });
+
+    it('Epic のみの場合', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: {
+          epicKey: 'PROJ-123',
+        },
+      });
+
+      const result = getJiraInfoFromSpec('user-auth');
+
+      expect(result).toEqual<JiraInfo>({
+        epicKey: 'PROJ-123',
+        storyKeys: [],
+        firstStoryKey: null,
+        projectKey: null,
+        epicUrl: null,
+      });
+    });
+
+    it('カスタム projectRoot を指定できる', () => {
+      mockLoadSpecJson.mockReturnValue({ jira: { epicKey: 'PROJ-123' } });
+
+      getJiraInfoFromSpec('user-auth', '/custom/path');
+
+      expect(mockLoadSpecJson).toHaveBeenCalledWith(
+        'user-auth',
+        '/custom/path',
+      );
+    });
+  });
+
+  describe('checkJiraInfoStatus', () => {
+    it('すべての JIRA 情報がある場合', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: {
+          epicKey: 'PROJ-123',
+          storyKeys: ['PROJ-124'],
+        },
+      });
+
+      const result = checkJiraInfoStatus('user-auth');
+
+      expect(result).toEqual<JiraInfoStatus>({
+        hasJiraInfo: true,
+        hasEpic: true,
+        hasStories: true,
+        missing: [],
+      });
+    });
+
+    it('JIRA 情報がまったくない場合', () => {
+      mockLoadSpecJson.mockReturnValue({});
+
+      const result = checkJiraInfoStatus('user-auth');
+
+      expect(result).toEqual<JiraInfoStatus>({
+        hasJiraInfo: false,
+        hasEpic: false,
+        hasStories: false,
+        missing: ['Epic', 'Story'],
+      });
+    });
+
+    it('Epic のみがある場合', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: { epicKey: 'PROJ-123' },
+      });
+
+      const result = checkJiraInfoStatus('user-auth');
+
+      expect(result).toEqual<JiraInfoStatus>({
+        hasJiraInfo: true,
+        hasEpic: true,
+        hasStories: false,
+        missing: ['Story'],
+      });
+    });
+
+    it('Story のみがある場合', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: { storyKeys: ['PROJ-124'] },
+      });
+
+      const result = checkJiraInfoStatus('user-auth');
+
+      expect(result).toEqual<JiraInfoStatus>({
+        hasJiraInfo: true,
+        hasEpic: false,
+        hasStories: true,
+        missing: ['Epic'],
+      });
+    });
+  });
+
+  describe('canIntegrateWithJira', () => {
+    it('Epic がある場合は連携可能', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: { epicKey: 'PROJ-123' },
+      });
+
+      expect(canIntegrateWithJira('user-auth')).toBe(true);
+    });
+
+    it('Epic がない場合は連携不可', () => {
+      mockLoadSpecJson.mockReturnValue({
+        jira: { storyKeys: ['PROJ-124'] },
+      });
+
+      expect(canIntegrateWithJira('user-auth')).toBe(false);
+    });
+
+    it('JIRA 情報がない場合は連携不可', () => {
+      mockLoadSpecJson.mockReturnValue({});
+
+      expect(canIntegrateWithJira('user-auth')).toBe(false);
+    });
+  });
+});