fix: include test name in dedup key and capture parameterized case count

- Test failure dedup key now includes the test name when a location is
  present, preventing distinct failures from different tests sharing
  the same assertion line from being collapsed. Suite name is excluded
  from the location-present key because it disagrees between xcresult
  and live parsing sources.

- ParsedTestCase gains a caseCount field populated from the 'with N
  test cases' suffix in Swift Testing parameterized results. Event
  parsers increment progress by caseCount instead of 1.

Author: cameroncooke

src/utils/__tests__/swift-testing-line-parsers.test.ts

@@ -83,17 +83,6 @@ describe('Swift Testing line parsers', () => {
       });
     });
 
-    it('should parse a failed parameterized test', () => {
-      const result = parseSwiftTestingResultLine(
-        '✘ Test "Parameterized failure" with 3 test cases failed after 0.001 seconds with 1 issue.',
-      );
-      expect(result).toEqual({
-        status: 'failed',
-        rawName: 'Parameterized failure',
-        testName: 'Parameterized failure',
-        durationText: '0.001s',
-      });
-    });
 
     it('should parse a skipped test (arrow format)', () => {
       const result = parseSwiftTestingResultLine('➜ Test disabledTest() skipped: "Not ready yet"');

src/utils/swift-testing-event-parser.ts

@@ -100,8 +100,9 @@ export function createSwiftTestingEventParser(
         durationMs,
       });
       lastIssueDiagnostic = null;
-      completedCount += 1;
-      failedCount += 1;
+      const increment = stResult.caseCount ?? 1;
+      completedCount += increment;
+      failedCount += increment;
       emitTestProgress();
       return;
     }
@@ -122,12 +123,13 @@ export function createSwiftTestingEventParser(
 
     // Swift Testing result line: ✔/✘/◇ Test "Name" passed/failed/skipped (non-failure or no pending issue)
     if (stResult) {
-      completedCount += 1;
+      const increment = stResult.caseCount ?? 1;
+      completedCount += increment;
       if (stResult.status === 'failed') {
-        failedCount += 1;
+        failedCount += increment;
       }
       if (stResult.status === 'skipped') {
-        skippedCount += 1;
+        skippedCount += increment;
       }
       emitTestProgress();
       return;

src/utils/swift-testing-line-parsers.ts

@@ -8,7 +8,7 @@ import {
 // Optional verbose suffix: (aka 'funcName()')
 // Optional parameterized suffix: with N test cases
 const OPTIONAL_AKA = `(?:\\s*\\(aka '[^']*'\\))?`;
-const OPTIONAL_PARAMETERIZED = `(?:\\s+with \\d+ test cases?)?`;
+const OPTIONAL_PARAMETERIZED = `(?:\\s+with (\\d+) test cases?)?`;
 
 /**
  * Parse a Swift Testing result line (passed/failed/skipped).
@@ -29,14 +29,16 @@ export function parseSwiftTestingResultLine(line: string): ParsedTestCase | null
   );
   const passedMatch = line.match(passedRegex);
   if (passedMatch) {
-    const [, name, duration] = passedMatch;
+    const [, name, caseCountStr, duration] = passedMatch;
     const { suiteName, testName } = parseRawTestName(name);
+    const caseCount = caseCountStr ? Number(caseCountStr) : undefined;
     return {
       status: 'passed',
       rawName: name,
       suiteName,
       testName,
       durationText: `${duration}s`,
+      ...(caseCount !== undefined && { caseCount }),
     };
   }
 
@@ -46,14 +48,16 @@ export function parseSwiftTestingResultLine(line: string): ParsedTestCase | null
   );
   const failedMatch = line.match(failedRegex);
   if (failedMatch) {
-    const [, name, duration] = failedMatch;
+    const [, name, caseCountStr, duration] = failedMatch;
     const { suiteName, testName } = parseRawTestName(name);
+    const caseCount = caseCountStr ? Number(caseCountStr) : undefined;
     return {
       status: 'failed',
       rawName: name,
       suiteName,
       testName,
       durationText: `${duration}s`,
+      ...(caseCount !== undefined && { caseCount }),
     };
   }
 

src/utils/xcodebuild-line-parsers.ts

@@ -26,6 +26,7 @@ export interface ParsedTestCase {
   suiteName?: string;
   testName: string;
   durationText?: string;
+  caseCount?: number;
 }
 
 export interface ParsedTotals {

src/utils/xcodebuild-run-state.ts

@@ -50,12 +50,16 @@ function normalizeTestFailureLocation(location: string | undefined): string | nu
 function normalizeTestFailureKey(event: TestFailureEvent): string {
   const normalizedLocation = normalizeTestFailureLocation(event.location);
   const normalizedMessage = event.message.trim().toLowerCase();
+  const suite = normalizeTestIdentifier(event.suite);
+  const test = normalizeTestIdentifier(event.test);
 
   if (normalizedLocation) {
-    return `${normalizedLocation}|${normalizedMessage}`;
+    // Include test name but NOT suite -- suite naming disagrees between xcresult
+    // and live parsing (e.g. 'Module.Suite' vs absent). Test name is consistent.
+    return `${test}|${normalizedLocation}|${normalizedMessage}`;
   }
 
-  return `${normalizeTestIdentifier(event.suite)}|${normalizeTestIdentifier(event.test)}|${normalizedMessage}`;
+  return `${suite}|${test}|${normalizedMessage}`;
 }
 
 export interface FinalizeOptions {