@aaronshaf/ger 3.0.2 → 4.0.1

package/tests/list.test.ts

@@ -0,0 +1,220 @@
+import { describe, test, expect, beforeAll, afterAll, afterEach } from 'bun:test'
+import { HttpResponse, http } from 'msw'
+import { setupServer } from 'msw/node'
+import { Effect, Layer } from 'effect'
+import { listCommand } from '@/cli/commands/list'
+import { GerritApiServiceLive } from '@/api/gerrit'
+import { ConfigService } from '@/services/config'
+import { createMockConfigService } from './helpers/config-mock'
+import type { ChangeInfo } from '@/schemas/gerrit'
+
+const makeChange = (overrides: Partial<ChangeInfo> = {}): ChangeInfo => ({
+  id: 'project~main~I123',
+  _number: 12345,
+  project: 'my-project',
+  branch: 'main',
+  change_id: 'I123',
+  subject: 'Fix the important thing',
+  status: 'NEW',
+  created: '2025-01-15 10:00:00.000000000',
+  updated: '2025-01-15 12:00:00.000000000',
+  owner: { _account_id: 1, name: 'Alice', email: 'alice@x.com' },
+  ...overrides,
+})
+
+const mockChanges: ChangeInfo[] = [
+  makeChange({ _number: 1, subject: 'First change' }),
+  makeChange({
+    _number: 2,
+    subject: 'Second change with Code-Review',
+    labels: { 'Code-Review': { approved: { _account_id: 1 }, value: 2 } },
+  }),
+  makeChange({
+    _number: 3,
+    subject: 'Third change rejected',
+    labels: { 'Code-Review': { rejected: { _account_id: 2 }, value: -2 } },
+  }),
+]
+
+const server = setupServer(
+  http.get('*/a/accounts/self', () =>
+    HttpResponse.json({ _account_id: 1, name: 'Alice', email: 'alice@x.com' }),
+  ),
+  http.get('*/a/changes/', () => HttpResponse.json(mockChanges)),
+)
+
+beforeAll(() => server.listen({ onUnhandledRequest: 'bypass' }))
+afterAll(() => server.close())
+afterEach(() => server.resetHandlers())
+
+const mockConfig = createMockConfigService()
+
+describe('list command', () => {
+  test('renders table with header and rows', async () => {
+    const logs: string[] = []
+    const origLog = console.log
+    console.log = (...args: unknown[]) => logs.push(String(args[0]))
+
+    try {
+      await Effect.runPromise(
+        listCommand({}).pipe(
+          Effect.provide(GerritApiServiceLive),
+          Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+        ),
+      )
+    } finally {
+      console.log = origLog
+    }
+
+    const output = logs.join('\n')
+    expect(output).toContain('Change')
+    expect(output).toContain('Subject')
+    expect(output).toContain('CR')
+    expect(output).toContain('Verified')
+    expect(output).toContain('1')
+    expect(output).toContain('First change')
+  })
+
+  test('outputs JSON', async () => {
+    const logs: string[] = []
+    const origLog = console.log
+    console.log = (...args: unknown[]) => logs.push(String(args[0]))
+
+    try {
+      await Effect.runPromise(
+        listCommand({ json: true }).pipe(
+          Effect.provide(GerritApiServiceLive),
+          Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+        ),
+      )
+    } finally {
+      console.log = origLog
+    }
+
+    const parsed = JSON.parse(logs.join('')) as { status: string; count: number }
+    expect(parsed.status).toBe('success')
+    expect(parsed.count).toBe(3)
+  })
+
+  test('outputs XML', async () => {
+    const logs: string[] = []
+    const origLog = console.log
+    console.log = (...args: unknown[]) => logs.push(String(args[0]))
+
+    try {
+      await Effect.runPromise(
+        listCommand({ xml: true }).pipe(
+          Effect.provide(GerritApiServiceLive),
+          Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+        ),
+      )
+    } finally {
+      console.log = origLog
+    }
+
+    const output = logs.join('\n')
+    expect(output).toContain('<changes count="3">')
+    expect(output).toContain('<number>1</number>')
+  })
+
+  test('--detailed shows per-change info', async () => {
+    const logs: string[] = []
+    const origLog = console.log
+    console.log = (...args: unknown[]) => logs.push(String(args[0]))
+
+    try {
+      await Effect.runPromise(
+        listCommand({ detailed: true }).pipe(
+          Effect.provide(GerritApiServiceLive),
+          Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+        ),
+      )
+    } finally {
+      console.log = origLog
+    }
+
+    const output = logs.join('\n')
+    expect(output).toContain('Change:')
+    expect(output).toContain('Subject:')
+    expect(output).toContain('Project:')
+  })
+
+  test('--limit caps results', async () => {
+    const logs: string[] = []
+    const origLog = console.log
+    console.log = (...args: unknown[]) => logs.push(String(args[0]))
+
+    try {
+      await Effect.runPromise(
+        listCommand({ limit: 1, json: true }).pipe(
+          Effect.provide(GerritApiServiceLive),
+          Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+        ),
+      )
+    } finally {
+      console.log = origLog
+    }
+
+    const parsed = JSON.parse(logs.join('')) as { count: number }
+    expect(parsed.count).toBe(1)
+  })
+
+  test('--status passes query to API', async () => {
+    let capturedUrl = ''
+    server.use(
+      http.get('*/a/changes/', ({ request }) => {
+        capturedUrl = decodeURIComponent(request.url)
+        return HttpResponse.json([])
+      }),
+    )
+
+    await Effect.runPromise(
+      listCommand({ status: 'merged' }).pipe(
+        Effect.provide(GerritApiServiceLive),
+        Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+      ),
+    )
+
+    expect(capturedUrl).toContain('status:merged')
+  })
+
+  test('--reviewer uses reviewer query', async () => {
+    let capturedUrl = ''
+    server.use(
+      http.get('*/a/changes/', ({ request }) => {
+        capturedUrl = decodeURIComponent(request.url)
+        return HttpResponse.json([])
+      }),
+    )
+
+    await Effect.runPromise(
+      listCommand({ reviewer: true }).pipe(
+        Effect.provide(GerritApiServiceLive),
+        Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+      ),
+    )
+
+    expect(capturedUrl).toContain('reviewer:')
+  })
+
+  test('shows empty message when no changes', async () => {
+    server.use(http.get('*/a/changes/', () => HttpResponse.json([])))
+
+    const logs: string[] = []
+    const origLog = console.log
+    console.log = (...args: unknown[]) => logs.push(String(args[0]))
+
+    try {
+      await Effect.runPromise(
+        listCommand({}).pipe(
+          Effect.provide(GerritApiServiceLive),
+          Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+        ),
+      )
+    } finally {
+      console.log = origLog
+    }
+
+    expect(logs.join('\n')).toContain('No changes found')
+  })
+})

package/tests/retrigger.test.ts

@@ -0,0 +1,159 @@
+import { afterAll, afterEach, beforeAll, beforeEach, describe, expect, it, mock } from 'bun:test'
+import { Effect, Layer } from 'effect'
+import { HttpResponse, http } from 'msw'
+import { setupServer } from 'msw/node'
+import { GerritApiServiceLive } from '@/api/gerrit'
+import { ConfigService } from '@/services/config'
+import { retriggerCommand } from '@/cli/commands/retrigger'
+import { createMockConfigService } from './helpers/config-mock'
+
+// Mock @inquirer/prompts so tests don't block on stdin
+const mockInput = mock(async () => 'trigger-build')
+mock.module('@inquirer/prompts', () => ({ input: mockInput }))
+
+const server = setupServer(
+  http.get('*/a/accounts/self', () =>
+    HttpResponse.json({ _account_id: 1, name: 'User', email: 'u@example.com' }),
+  ),
+  http.post('*/a/changes/:changeId/revisions/current/review', () => HttpResponse.json({})),
+)
+
+beforeAll(() => server.listen({ onUnhandledRequest: 'bypass' }))
+afterAll(() => server.close())
+
+describe('retrigger command', () => {
+  let mockConsoleLog: ReturnType<typeof mock>
+  let mockConsoleError: ReturnType<typeof mock>
+
+  beforeEach(() => {
+    mockConsoleLog = mock()
+    mockConsoleError = mock()
+    console.log = mockConsoleLog
+    console.error = mockConsoleError
+    mockInput.mockReset()
+    mockInput.mockResolvedValue('trigger-build')
+    server.resetHandlers()
+  })
+
+  afterEach(() => {
+    server.resetHandlers()
+  })
+
+  it('posts the retrigger comment when change-id is explicit and comment is configured', async () => {
+    const mockConfig = createMockConfigService(undefined, undefined, '__TRIGGER__')
+
+    await Effect.runPromise(
+      retriggerCommand('12345', {}).pipe(
+        Effect.provide(GerritApiServiceLive),
+        Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+      ),
+    )
+
+    // Should print success
+    expect(mockConsoleLog).toHaveBeenCalledWith(expect.stringContaining('✓'))
+  })
+
+  it('posts to the given change-id', async () => {
+    const mockConfig = createMockConfigService(undefined, undefined, '__TRIGGER__')
+
+    let postedChangeId = ''
+    server.use(
+      http.post('*/a/changes/:changeId/revisions/current/review', ({ params }) => {
+        postedChangeId = params.changeId as string
+        return HttpResponse.json({})
+      }),
+    )
+
+    await Effect.runPromise(
+      retriggerCommand('67890', {}).pipe(
+        Effect.provide(GerritApiServiceLive),
+        Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+      ),
+    )
+
+    expect(postedChangeId).toBe('67890')
+  })
+
+  it('prompts for retrigger comment when not configured, then saves it', async () => {
+    let savedComment = ''
+    const mockConfig: ReturnType<typeof createMockConfigService> = {
+      ...createMockConfigService(),
+      getRetriggerComment: Effect.succeed(undefined),
+      saveRetriggerComment: (comment: string) => {
+        savedComment = comment
+        return Effect.succeed(undefined as void)
+      },
+    }
+
+    mockInput.mockResolvedValue('my-trigger-comment')
+
+    await Effect.runPromise(
+      retriggerCommand('12345', {}).pipe(
+        Effect.provide(GerritApiServiceLive),
+        Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+      ),
+    )
+
+    expect(mockInput).toHaveBeenCalled()
+    expect(savedComment).toBe('my-trigger-comment')
+  })
+
+  it('throws when prompted comment is empty', async () => {
+    const mockConfig: ReturnType<typeof createMockConfigService> = {
+      ...createMockConfigService(),
+      getRetriggerComment: Effect.succeed(undefined),
+      saveRetriggerComment: () => Effect.succeed(undefined as void),
+    }
+
+    mockInput.mockResolvedValue('   ')
+
+    let threw = false
+    try {
+      await Effect.runPromise(
+        retriggerCommand('12345', {}).pipe(
+          Effect.provide(GerritApiServiceLive),
+          Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+        ),
+      )
+    } catch (e) {
+      threw = true
+      expect(String(e)).toContain('cannot be empty')
+    }
+    expect(threw).toBe(true)
+  })
+
+  it('outputs JSON on success', async () => {
+    const mockConfig = createMockConfigService(undefined, undefined, '__TRIGGER__')
+
+    const logs: string[] = []
+    console.log = (msg: string) => logs.push(msg)
+
+    await Effect.runPromise(
+      retriggerCommand('12345', { json: true }).pipe(
+        Effect.provide(GerritApiServiceLive),
+        Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+      ),
+    )
+
+    const parsed = JSON.parse(logs[0]) as { status: string; change_id: string }
+    expect(parsed.status).toBe('success')
+    expect(parsed.change_id).toBe('12345')
+  })
+
+  it('outputs XML on success', async () => {
+    const mockConfig = createMockConfigService(undefined, undefined, '__TRIGGER__')
+
+    const logs: string[] = []
+    console.log = (msg: string) => logs.push(msg)
+
+    await Effect.runPromise(
+      retriggerCommand('12345', { xml: true }).pipe(
+        Effect.provide(GerritApiServiceLive),
+        Effect.provide(Layer.succeed(ConfigService, mockConfig)),
+      ),
+    )
+
+    expect(logs.join('\n')).toContain('<retrigger>')
+    expect(logs.join('\n')).toContain('<status>success</status>')
+  })
+})