test: add tests for per-test timing events and showTestResults flag

Author: codeman9

src/rendering/__tests__/text-render-parity.test.ts

@@ -2,6 +2,7 @@ import { afterEach, describe, expect, it, vi } from 'vitest';
 import type { PipelineEvent } from '../../types/pipeline-events.ts';
 import { renderEvents } from '../render.ts';
 import { createCliTextRenderer } from '../../utils/renderers/cli-text-renderer.ts';
+import { renderCliTextTranscript } from '../../utils/renderers/cli-text-renderer.ts';
 
 function captureCliText(events: readonly PipelineEvent[]): string {
   const stdoutWrite = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
@@ -168,4 +169,37 @@ describe('text render parity', () => {
     expect(output).toContain('xcodebuildmcp macos get-app-path --scheme "MCPTest"');
     expect(output).not.toContain('get_mac_app_path({');
   });
+
+  it('omits per-test results by default and includes them when showTestResults is true', () => {
+    const events: PipelineEvent[] = [
+      {
+        type: 'test-case-result',
+        timestamp: '2026-04-14T00:00:00.000Z',
+        operation: 'TEST',
+        suite: 'Suite',
+        test: 'testA',
+        status: 'passed',
+        durationMs: 100,
+      },
+      {
+        type: 'summary',
+        timestamp: '2026-04-14T00:00:01.000Z',
+        operation: 'TEST',
+        status: 'SUCCEEDED',
+        totalTests: 1,
+        passedTests: 1,
+        skippedTests: 0,
+        durationMs: 100,
+      },
+    ];
+
+    const withoutFlag = renderCliTextTranscript(events);
+    expect(withoutFlag).not.toContain('Test Results:');
+    expect(withoutFlag).toContain('1 test passed');
+
+    const withFlag = renderCliTextTranscript(events, { showTestResults: true });
+    expect(withFlag).toContain('Test Results:');
+    expect(withFlag).toContain('Suite/testA (0.100s)');
+    expect(withFlag).toContain('1 test passed');
+  });
 });

src/utils/__tests__/xcodebuild-event-parser.test.ts

@@ -304,4 +304,34 @@ describe('xcodebuild-event-parser', () => {
     const statusEvents = events.filter((e) => e.type === 'build-stage');
     expect(statusEvents.length).toBeLessThanOrEqual(1);
   });
+
+  it('emits test-case-result events with per-test timing for all statuses', () => {
+    const events = collectEvents('TEST', [
+      { source: 'stdout', text: "Test Case '-[Suite testA]' passed (0.001 seconds)\n" },
+      { source: 'stdout', text: "Test Case '-[Suite testB]' failed (0.002 seconds)\n" },
+      { source: 'stdout', text: "Test Case '-[Suite testC]' skipped (0.000 seconds)\n" },
+    ]);
+
+    const results = events.filter((e) => e.type === 'test-case-result');
+    expect(results).toHaveLength(3);
+    expect(results[0]).toMatchObject({
+      type: 'test-case-result',
+      operation: 'TEST',
+      suite: 'Suite',
+      test: 'testA',
+      status: 'passed',
+      durationMs: 1,
+    });
+    expect(results[1]).toMatchObject({ test: 'testB', status: 'failed', durationMs: 2 });
+    expect(results[2]).toMatchObject({ test: 'testC', status: 'skipped', durationMs: 0 });
+  });
+
+  it('does not emit test-case-result for BUILD operations', () => {
+    const events = collectEvents('BUILD', [
+      { source: 'stdout', text: "Test Case '-[Suite testA]' passed (0.001 seconds)\n" },
+    ]);
+
+    const results = events.filter((e) => e.type === 'test-case-result');
+    expect(results).toHaveLength(0);
+  });
 });

src/utils/__tests__/xcodebuild-run-state.test.ts

@@ -404,4 +404,32 @@ describe('xcodebuild-run-state', () => {
     expect(forwarded[0].type).toBe('header');
     expect(forwarded[1].type).toBe('next-steps');
   });
+
+  it('collects test-case-result events', () => {
+    const state = createXcodebuildRunState({ operation: 'TEST' });
+
+    state.push({
+      type: 'test-case-result',
+      timestamp: ts(),
+      operation: 'TEST',
+      suite: 'Suite',
+      test: 'testA',
+      status: 'passed',
+      durationMs: 100,
+    });
+    state.push({
+      type: 'test-case-result',
+      timestamp: ts(),
+      operation: 'TEST',
+      suite: 'Suite',
+      test: 'testB',
+      status: 'failed',
+      durationMs: 200,
+    });
+
+    const snap = state.snapshot();
+    expect(snap.testCaseResults).toHaveLength(2);
+    expect(snap.testCaseResults[0]).toMatchObject({ test: 'testA', status: 'passed' });
+    expect(snap.testCaseResults[1]).toMatchObject({ test: 'testB', status: 'failed' });
+  });
 });

src/utils/renderers/__tests__/event-formatting.test.ts

@@ -12,6 +12,7 @@ import {
   formatStatusLineEvent,
   formatDetailTreeEvent,
   formatTransientStatusLineEvent,
+  formatTestCaseResults,
 } from '../event-formatting.ts';
 
 describe('event formatting', () => {
@@ -277,4 +278,34 @@ describe('event formatting', () => {
     expect(rendered).toContain('      - XCTAssertEqual failed');
     expect(rendered).toContain('      - Expected 4, got 5');
   });
+
+  it('formats per-test case results with status icons and durations', () => {
+    const rendered = formatTestCaseResults([
+      {
+        type: 'test-case-result',
+        timestamp: '',
+        operation: 'TEST',
+        suite: 'MathTests',
+        test: 'testAdd',
+        status: 'passed',
+        durationMs: 1234,
+      },
+      {
+        type: 'test-case-result',
+        timestamp: '',
+        operation: 'TEST',
+        test: 'testOrphan',
+        status: 'skipped',
+      },
+    ]);
+
+    expect(rendered).toContain('Test Results:');
+    expect(rendered).toContain('\u{2705} MathTests/testAdd (1.234s)');
+    expect(rendered).toContain('\u{23ED}\u{FE0F} testOrphan');
+    expect(rendered).not.toContain('testOrphan (');
+  });
+
+  it('returns empty string for no test case results', () => {
+    expect(formatTestCaseResults([])).toBe('');
+  });
 });