@travetto/test 8.0.0-alpha.2 → 8.0.0-alpha.21

package/src/consumer/types/cumulative.ts

@@ -1,15 +1,15 @@
-import type { TestConsumerShape } from '../types.ts';
+import type { SuiteCore } from '../../model/common.ts';
 import type { TestEvent, TestRemoveEvent } from '../../model/event.ts';
+import type { SuiteConfig, SuiteResult } from '../../model/suite.ts';
 import type { TestConfig, TestDiffSource, TestResult } from '../../model/test.ts';
-import type { Counts, SuiteConfig, SuiteResult } from '../../model/suite.ts';
-import { DelegatingConsumer } from './delegating.ts';
-import type { SuiteCore } from '../../model/common.ts';
 import { TestModelUtil } from '../../model/util.ts';
+import type { TestConsumerShape } from '../types.ts';
+import { DelegatingConsumer } from './delegating.ts';
 
 type ClassId = string;
 type ImportName = string;
 
-type CumulativeTestResult = Pick<TestResult, 'sourceHash' | 'status' | 'duration'>;
+type CumulativeTestResult = Pick<TestResult, 'sourceHash' | 'status' | 'duration' | 'selfDuration'>;
 type CumulativeSuiteResult = Pick<SuiteCore, 'import' | 'classId' | 'sourceHash'> & {
   tests: Record<string, CumulativeTestResult>;
 };
@@ -32,12 +32,12 @@ export class CumulativeSummaryConsumer extends DelegatingConsumer {
   }
 
   getOrCreateSuite({ tests: _, ...core }: SuiteConfig | SuiteResult): CumulativeSuiteResult {
-    return (this.#state[core.import] ??= {})[core.classId] ??= { ...core, tests: {} };
+    return ((this.#state[core.import] ??= {})[core.classId] ??= { ...core, tests: {} });
   }
 
   onTestBefore(config: TestConfig): TestConfig {
     const suite = this.getSuite(config);
-    suite.tests[config.methodName] = { sourceHash: config.sourceHash, status: 'unknown', duration: 0 };
+    suite.tests[config.methodName] = { sourceHash: config.sourceHash, status: 'unknown', duration: 0, selfDuration: 0 };
     return config;
   }
 
@@ -56,21 +56,9 @@ export class CumulativeSummaryConsumer extends DelegatingConsumer {
   onSuiteAfter(result: SuiteResult): SuiteResult {
     // Reset counts
     const suite = this.getSuite(result);
-    const totals: Counts & { duration: number } = {
-      passed: 0,
-      failed: 0,
-      skipped: 0,
-      errored: 0,
-      unknown: 0,
-      total: 0,
-      duration: 0
-    };
-    for (const test of Object.values(suite.tests)) {
-      totals[test.status] += 1;
-      totals.total += 1;
-      totals.duration += test.duration ?? 0;
-    }
-    return { ...result, ...totals, status: TestModelUtil.countsToTestStatus(totals) };
+    const results = TestModelUtil.buildSummary();
+    TestModelUtil.countTestResult(results, Object.values(suite.tests));
+    return { ...result, ...results, status: TestModelUtil.computeTestStatus(results) };
   }
 
   removeTest(importName: string, classId?: string, methodName?: string): void {
@@ -126,4 +114,4 @@ export class CumulativeSummaryConsumer extends DelegatingConsumer {
     }
     return output;
   }
-}
+}

package/src/consumer/types/delegating.ts

@@ -1,5 +1,5 @@
-import type { SuitesSummary, TestConsumerShape, TestRunState } from '../types.ts';
 import type { TestEvent, TestRemoveEvent } from '../../model/event.ts';
+import type { SuitesSummary, TestConsumerShape, TestRunState } from '../types.ts';
 
 /**
  * Delegating event consumer
@@ -20,6 +20,12 @@ export abstract class DelegatingConsumer implements TestConsumerShape {
     }
   }
 
+  async onTestRunState(state: TestRunState): Promise<void> {
+    for (const consumer of this.#consumers) {
+      await consumer.onTestRunState?.(state);
+    }
+  }
+
   onRemoveEvent(event: TestRemoveEvent): void {
     let result = event;
     if (this.transformRemove) {
@@ -58,4 +64,4 @@ export abstract class DelegatingConsumer implements TestConsumerShape {
 
   transform?(event: TestEvent): TestEvent | undefined;
   transformRemove?(event: TestRemoveEvent): TestRemoveEvent | undefined;
-}
+}

package/src/consumer/types/event.ts

@@ -3,8 +3,8 @@ import type { Writable } from 'node:stream';
 import { JSONUtil } from '@travetto/runtime';
 
 import type { TestEvent, TestRemoveEvent } from '../../model/event.ts';
-import type { TestConsumerShape } from '../types.ts';
 import { TestConsumer } from '../decorator.ts';
+import type { TestConsumerShape } from '../types.ts';
 
 /**
  * Streams all test events a JSON payload, in an nd-json format
@@ -28,4 +28,4 @@ export class EventStreamer implements TestConsumerShape {
   onRemoveEvent(event: TestRemoveEvent): void {
     this.sendPayload(event);
   }
-}
+}

package/src/consumer/types/exec.ts

@@ -1,16 +1,15 @@
-import { IpcChannel } from '@travetto/worker';
 import { JSONUtil } from '@travetto/runtime';
+import { IpcChannel } from '@travetto/worker';
 
 import type { TestEvent, TestRemoveEvent } from '../../model/event.ts';
-import type { TestConsumerShape } from '../types.ts';
 import { TestConsumer } from '../decorator.ts';
+import type { TestConsumerShape } from '../types.ts';
 
 /**
  * Triggers each event as an IPC command to a parent process
  */
 @TestConsumer()
 export class ExecutionEmitter extends IpcChannel<TestEvent> implements TestConsumerShape {
-
   sendPayload(payload: unknown & { type: string }): void {
     this.send(payload.type, JSONUtil.cloneForTransmit(payload));
   }
@@ -22,4 +21,4 @@ export class ExecutionEmitter extends IpcChannel<TestEvent> implements TestConsu
   onRemoveEvent(event: TestRemoveEvent): void {
     this.sendPayload(event);
   }
-}
+}

package/src/consumer/types/json.ts

@@ -2,24 +2,23 @@ import type { Writable } from 'node:stream';
 
 import { JSONUtil } from '@travetto/runtime';
 
-import type { SuitesSummary } from '../types.ts';
 import { TestConsumer } from '../decorator.ts';
+import type { SuitesSummary } from '../types.ts';
 
 /**
  * Returns the entire result set as a single JSON document
  */
 @TestConsumer()
 export class JSONEmitter {
-
   #stream: Writable;
 
   constructor(stream: Writable = process.stdout) {
     this.#stream = stream;
   }
 
-  onEvent(): void { }
+  onEvent(): void {}
 
   onSummary(summary: SuitesSummary): void {
     this.#stream.write(JSONUtil.toUTF8Pretty(summary));
   }
-}
+}

package/src/consumer/types/noop.ts

@@ -1,10 +1,10 @@
-import type { TestConsumerShape } from '../types.ts';
 import { TestConsumer } from '../decorator.ts';
+import type { TestConsumerShape } from '../types.ts';
 
 /**
  * Does nothing consumer
  */
 @TestConsumer()
 export class NoopConsumer implements TestConsumerShape {
-  onEvent(): void { }
-}
+  onEvent(): void {}
+}

package/src/consumer/types/runnable.ts

@@ -1,13 +1,12 @@
-import type { TestConsumerShape } from '../types.ts';
-import { TestResultsSummarizer } from './summarizer.ts';
 import type { TestEvent } from '../../model/event.ts';
+import type { TestConsumerShape } from '../types.ts';
 import { DelegatingConsumer } from './delegating.ts';
+import { TestResultsSummarizer } from './summarizer.ts';
 
 /**
  * Test consumer with support for multiple nested consumers, and summarization
  */
 export class RunnableTestConsumer extends DelegatingConsumer {
-
   #results?: TestResultsSummarizer;
 
   constructor(...consumers: TestConsumerShape[]) {
@@ -22,6 +21,6 @@ export class RunnableTestConsumer extends DelegatingConsumer {
 
   async summarizeAsBoolean(): Promise<boolean> {
     await this.summarize(this.#results?.summary);
-    return (this.#results?.summary.failed ?? 0) <= 0;
+    return (this.#results?.summary.failed ?? 0) <= 0 && (this.#results?.summary.errored ?? 0) <= 0;
   }
-}
+}

package/src/consumer/types/summarizer.ts

@@ -1,33 +1,20 @@
-import type { SuiteResult } from '../../model/suite.ts';
 import type { TestEvent } from '../../model/event.ts';
+import type { SuiteResult } from '../../model/suite.ts';
+import { TestModelUtil } from '../../model/util.ts';
 import type { SuitesSummary, TestConsumerShape } from '../types.ts';
 
 /**
  * Test Result Collector, combines all results into a single Suite Result
  */
 export class TestResultsSummarizer implements TestConsumerShape {
-
   summary: SuitesSummary = {
-    passed: 0,
-    failed: 0,
-    errored: 0,
-    skipped: 0,
-    unknown: 0,
-    total: 0,
-    duration: 0,
-    suites: [],
-    errors: []
+    ...TestModelUtil.buildSummary(),
+    suites: []
   };
 
   #merge(result: SuiteResult): void {
+    TestModelUtil.countTestResult(this.summary, Object.values(result.tests));
     this.summary.suites.push(result);
-    this.summary.failed += result.failed;
-    this.summary.errored += result.errored;
-    this.summary.passed += result.passed;
-    this.summary.unknown += result.unknown;
-    this.summary.skipped += result.skipped;
-    this.summary.duration += result.duration;
-    this.summary.total += result.total;
   }
 
   /**

package/src/consumer/types/tap-summary.ts

@@ -1,15 +1,14 @@
-import { Util, AsyncQueue } from '@travetto/runtime';
-import { StyleUtil, Terminal, TerminalUtil } from '@travetto/terminal';
+import { AsyncQueue, Util } from '@travetto/runtime';
+import { Terminal, TerminalUtil } from '@travetto/terminal';
 
 import type { TestEvent } from '../../model/event.ts';
-import type { TestResult, TestStatus } from '../../model/test.ts';
-
-import type { SuitesSummary, TestConsumerShape, TestRunState } from '../types.ts';
+import type { SuiteResult } from '../../model/suite.ts';
+import type { TestResult } from '../../model/test.ts';
+import { TestModelUtil } from '../../model/util.ts';
 import { TestConsumer } from '../decorator.ts';
-
-import { TapEmitter } from './tap.ts';
 import { CONSOLE_ENHANCER, type TestResultsEnhancer } from '../enhancer.ts';
-import type { SuiteResult } from '../../model/suite.ts';
+import type { SuitesSummary, TestConsumerShape, TestRunState } from '../types.ts';
+import { TapEmitter } from './tap.ts';
 
 type Result = {
   key: string;
@@ -22,7 +21,6 @@ type Result = {
  */
 @TestConsumer()
 export class TapSummaryEmitter implements TestConsumerShape {
-
   #timings = new Map<'test' | 'module' | 'file' | 'suite', Map<string, Result>>();
 
   #terminal: Terminal;
@@ -31,6 +29,7 @@ export class TapSummaryEmitter implements TestConsumerShape {
   #consumer: TapEmitter;
   #enhancer: TestResultsEnhancer;
   #options?: Record<string, unknown>;
+  #state: TestRunState = {};
 
   constructor(terminal: Terminal = new Terminal(process.stderr)) {
     this.#terminal = terminal;
@@ -51,20 +50,16 @@ export class TapSummaryEmitter implements TestConsumerShape {
     foundModule.duration += suite.duration;
     foundModule.tests += testCount;
 
-    const foundFile = this.#timings
-      .getOrInsert('file', new Map<string, Result>())
-      .getOrInsert(file, { key: file, duration: 0, tests: 0 });
+    const foundFile = this.#timings.getOrInsert('file', new Map<string, Result>()).getOrInsert(file, { key: file, duration: 0, tests: 0 });
 
     foundFile.duration += suite.duration;
     foundFile.tests += testCount;
 
-    this.#timings
-      .getOrInsert('suite', new Map<string, Result>())
-      .set(suite.classId, {
-        key: suite.classId,
-        duration: suite.duration,
-        tests: testCount
-      });
+    this.#timings.getOrInsert('suite', new Map<string, Result>()).set(suite.classId, {
+      key: suite.classId,
+      duration: suite.duration,
+      tests: testCount
+    });
   }
 
   #renderTimings(): void {
@@ -75,7 +70,9 @@ export class TapSummaryEmitter implements TestConsumerShape {
       const top10 = [...results.values()].toSorted((a, b) => b.duration - a.duration).slice(0, count);
 
       for (const result of top10) {
-        console.log(`  * ${this.#enhancer.testName(result.key)} - ${this.#enhancer.total(result.duration)}ms / ${this.#enhancer.total(result.tests)} tests`);
+        console.log(
+          `  * ${this.#enhancer.testName(result.key)} - ${this.#enhancer.total(result.duration)}ms / ${this.#enhancer.total(result.tests)} tests`
+        );
       }
       this.#consumer.log('');
     }
@@ -87,22 +84,34 @@ export class TapSummaryEmitter implements TestConsumerShape {
     this.#consumer.setOptions(options);
   }
 
+  onTestRunState(state: TestRunState): void {
+    Object.assign(this.#state, state);
+  }
+
   async onStart(state: TestRunState): Promise<void> {
     this.#consumer.onStart();
-    const total: Record<TestStatus | 'count', number> = { errored: 0, failed: 0, passed: 0, skipped: 0, unknown: 0, count: 0 };
-    const success = StyleUtil.getStyle({ text: '#e5e5e5', background: '#026020' }); // White on dark green
-    const fail = StyleUtil.getStyle({ text: '#e5e5e5', background: '#8b0000' }); // White on dark red
+    this.onTestRunState(state);
+
+    const total = TestModelUtil.buildSummary();
     this.#progress = this.#terminal.streamToBottom(
       Util.mapAsyncIterable(
         this.#results,
-        (value) => {
-          total[value.status] += 1;
-          total.count += 1;
+        value => {
+          TestModelUtil.countTestResult(total, [value]);
           const statusLine = `${total.failed} failed, ${total.errored} errored, ${total.skipped} skipped`;
-          return { value: `Tests %idx/%total [${statusLine}] -- ${value.classId}`, total: state.testCount, idx: total.passed };
-
+          return {
+            value: `Tests %completed/%total [${statusLine}] -- ${value.classId}`,
+            total: this.#state.testCount,
+            completed: total.passed,
+            failed: total.failed + total.errored
+          };
         },
-        TerminalUtil.progressBarUpdater(this.#terminal, { style: () => ({ complete: (total.failed || total.errored) ? fail : success }) })
+        TerminalUtil.progressBarUpdater(this.#terminal, {
+          style: {
+            complete: { text: '#e5e5e5', background: '#026020' },
+            failed: { text: '#e5e5e5', background: '#8b0000' }
+          }
+        })
       ),
       { minDelay: 100 }
     );
@@ -112,7 +121,7 @@ export class TapSummaryEmitter implements TestConsumerShape {
     if (event.type === 'test' && event.phase === 'after') {
       const { test } = event;
       this.#results.add(test);
-      if (test.status === 'failed') {
+      if (test.status !== 'passed' && test.status !== 'skipped') {
         this.#consumer.onEvent(event);
       }
       const tests = this.#timings.getOrInsert('test', new Map<string, Result>());

package/src/consumer/types/tap.ts

@@ -1,13 +1,17 @@
 import path from 'node:path';
+
 import { stringify } from 'yaml';
 
-import { Terminal, StyleUtil } from '@travetto/terminal';
-import { TimeUtil, RuntimeIndex, hasToJSON, JSONUtil } from '@travetto/runtime';
+import { RuntimeIndex, TimeUtil } from '@travetto/runtime';
+import { StyleUtil, Terminal } from '@travetto/terminal';
 
 import type { TestEvent } from '../../model/event.ts';
-import type { SuitesSummary, TestConsumerShape } from '../types.ts';
 import { TestConsumer } from '../decorator.ts';
-import { type TestResultsEnhancer, CONSOLE_ENHANCER } from '../enhancer.ts';
+import { CONSOLE_ENHANCER, type TestResultsEnhancer } from '../enhancer.ts';
+import type { SuitesSummary, TestConsumerShape } from '../types.ts';
+import { TestConsumerUtil } from './util.ts';
+
+const SPACE = ' ';
 
 /**
  * TAP Format consumer
@@ -17,13 +21,10 @@ export class TapEmitter implements TestConsumerShape {
   #count = 0;
   #enhancer: TestResultsEnhancer;
   #terminal: Terminal;
-  #start: number;
   #options?: Record<string, unknown>;
+  #start: number = 0;
 
-  constructor(
-    terminal = new Terminal(),
-    enhancer: TestResultsEnhancer = CONSOLE_ENHANCER
-  ) {
+  constructor(terminal = new Terminal(), enhancer: TestResultsEnhancer = CONSOLE_ENHANCER) {
     this.#terminal = terminal;
     this.#enhancer = enhancer;
   }
@@ -50,28 +51,13 @@ export class TapEmitter implements TestConsumerShape {
   logMeta(metadata: Record<string, unknown>): void {
     const lineLength = this.#terminal.width - 5;
     let body = stringify(metadata, { lineWidth: lineLength, indent: 2 });
-    body = body.split('\n').map(line => `  ${line}`).join('\n');
+    body = body
+      .split('\n')
+      .map(line => `  ${line}`)
+      .join('\n');
     this.log(`---\n${this.#enhancer.objectInspect(body)}\n...`);
   }
 
-  /**
-   * Error to string
-   * @param error
-   */
-  errorToString(error?: Error): string | undefined {
-    if (error && error.name !== 'AssertionError') {
-      if (error instanceof Error) {
-        let out = JSONUtil.toUTF8(hasToJSON(error) ? error.toJSON() : error, { indent: 2 });
-        if (this.#options?.verbose && error.stack) {
-          out = `${out}\n${error.stack}`;
-        }
-        return out;
-      } else {
-        return `${error}`;
-      }
-    }
-  }
-
   /**
    * Listen for each event
    */
@@ -86,7 +72,7 @@ export class TapEmitter implements TestConsumerShape {
         StyleUtil.link(suiteId, `file://${suiteSourceFile}#${test.suiteLineStart ?? 1}`),
         ' - ',
         StyleUtil.link(this.#enhancer.testName(test.methodName), `file://${testSourceFile}#${test.lineStart}`),
-        ...test.description ? [`: ${this.#enhancer.testDescription(test.description)}`] : []
+        ...(test.description ? [`: ${this.#enhancer.testDescription(test.description)}`] : [])
       ].join('');
 
       this.log(`# ${header}`);
@@ -125,9 +111,17 @@ export class TapEmitter implements TestConsumerShape {
       // Track test result
       let status = `${this.#enhancer.testNumber(++this.#count)} `;
       switch (test.status) {
-        case 'skipped': status += ' # SKIP'; break;
-        case 'failed': status = `${this.#enhancer.failure('not ok')} ${status}`; break;
-        default: status = `${this.#enhancer.success('ok')} ${status}`;
+        case 'passed':
+          `${this.#enhancer.success('ok')} ${status}`;
+          break;
+        case 'skipped':
+          status += ' # SKIP';
+          break;
+        case 'unknown':
+          break;
+        default:
+          status = `${this.#enhancer.failure('not ok')} ${status}`;
+          break;
       }
       status += header;
 
@@ -138,9 +132,9 @@ export class TapEmitter implements TestConsumerShape {
         case 'errored':
         case 'failed': {
           if (test.error) {
-            const msg = this.errorToString(test.error);
-            if (msg) {
-              this.logMeta({ error: msg });
+            const message = TestConsumerUtil.errorToString(test.error, !!this.#options?.verbose);
+            if (message) {
+              this.logMeta({ error: message });
             }
           }
           break;
@@ -168,28 +162,42 @@ export class TapEmitter implements TestConsumerShape {
   onSummary(summary: SuitesSummary): void {
     this.log(`${this.#enhancer.testNumber(1)}..${this.#enhancer.testNumber(summary.total)}`);
 
-    if (summary.errors.length) {
-      this.log('---\n');
-      for (const error of summary.errors) {
-        const msg = this.errorToString(error);
-        if (msg) {
-          this.log(this.#enhancer.failure(msg));
-        }
-      }
-    }
-
     const allPassed = !summary.failed && !summary.errored;
 
-    this.log([
-      this.#enhancer[allPassed ? 'success' : 'failure']('Results'),
-      `${this.#enhancer.total(summary.passed)}/${this.#enhancer.total(summary.total)},`,
-      allPassed ? 'failed' : this.#enhancer.failure('failed'),
-      `${this.#enhancer.total(summary.failed)}`,
-      allPassed ? 'errored' : this.#enhancer.failure('errored'),
-      `${this.#enhancer.total(summary.errored)}`,
-      'skipped',
-      this.#enhancer.total(summary.skipped),
-      `# (Total Test Time: ${TimeUtil.asClock(summary.duration)}, Total Run Time: ${TimeUtil.asClock(Date.now() - this.#start)})`
-    ].join(' '));
+    this.log(
+      [
+        this.#enhancer[allPassed ? 'success' : 'failure']('Results'),
+        SPACE,
+        `${this.#enhancer.total(summary.passed)}/${this.#enhancer.total(summary.total)},`,
+        SPACE,
+        allPassed ? 'failed' : this.#enhancer.failure('failed'),
+        SPACE,
+        `${this.#enhancer.total(summary.failed)}`,
+        SPACE,
+        allPassed ? 'errored' : this.#enhancer.failure('errored'),
+        SPACE,
+        `${this.#enhancer.total(summary.errored)}`,
+        SPACE,
+        'skipped',
+        SPACE,
+        this.#enhancer.total(summary.skipped),
+        SPACE,
+        '#',
+        SPACE,
+        '(Timings:',
+        SPACE,
+        'Self=',
+        TimeUtil.asClock(summary.selfDuration),
+        ',',
+        SPACE,
+        'Total=',
+        TimeUtil.asClock(summary.duration),
+        ',',
+        SPACE,
+        'Clock=',
+        TimeUtil.asClock(Date.now() - this.#start),
+        ')'
+      ].join('')
+    );
   }
 }

package/src/consumer/types/util.ts

@@ -0,0 +1,31 @@
+import { AssertionError } from 'node:assert';
+import util from 'node:util';
+
+import { TypedObject } from '@travetto/runtime';
+
+export class TestConsumerUtil {
+  /**
+   * Convert error to string
+   */
+  static errorToString(error?: Error, verbose?: boolean): string | undefined {
+    if (error instanceof AssertionError) {
+      return;
+    } else if (error instanceof Error) {
+      const stack = error.stack
+        ? error.stack
+            .split(/\n/)
+            .slice(0, verbose ? -1 : 5)
+            .join('\n')
+        : error.message;
+      const subObject: Record<string, unknown> = {};
+      for (const key of TypedObject.keys(error)) {
+        if (key !== 'stack' && key !== 'message' && key !== 'name') {
+          subObject[key] = error[key];
+        }
+      }
+      return `${stack}${Object.keys(subObject).length ? `\n${util.inspect(subObject)}` : ''}`;
+    } else {
+      return `${error}`;
+    }
+  }
+}

package/src/consumer/types/xunit.ts

@@ -5,8 +5,9 @@ import { stringify } from 'yaml';
 import { RuntimeIndex } from '@travetto/runtime';
 
 import type { TestEvent } from '../../model/event.ts';
-import type { SuitesSummary, TestConsumerShape } from '../types.ts';
 import { TestConsumer } from '../decorator.ts';
+import type { SuitesSummary, TestConsumerShape } from '../types.ts';
+import { TestConsumerUtil } from './util.ts';
 
 /**
  * Xunit consumer, compatible with JUnit formatters
@@ -36,7 +37,10 @@ export class XunitEmitter implements TestConsumerShape {
     }
     if (Object.keys(metadata).length) {
       let body = stringify(metadata);
-      body = body.split('\n').map(line => `  ${line}`).join('\n');
+      body = body
+        .split('\n')
+        .map(line => `  ${line}`)
+        .join('\n');
       return `<![CDATA[\n${body}\n]]>`;
     } else {
       return '';
@@ -48,7 +52,6 @@ export class XunitEmitter implements TestConsumerShape {
    */
   onEvent(event: TestEvent): void {
     if (event.type === 'test' && event.phase === 'after') {
-
       const { test } = event;
 
       let name = `${test.methodName}`;
@@ -59,9 +62,9 @@ export class XunitEmitter implements TestConsumerShape {
       let body = '';
 
       if (test.error) {
-        const assertion = test.assertions.find(item => !!item.error)!;
+        const errorMessage = TestConsumerUtil.errorToString(test.error);
         const node = test.status === 'failed' ? 'failure' : 'error';
-        body = `<${node} type="${assertion.text}" message="${encodeURIComponent(assertion.message!)}"><![CDATA[${assertion.error!.stack}]]></${node}>`;
+        body = `<${node} type="${test.error.constructor.name}" message="${encodeURIComponent(test.error.message)}"><![CDATA[${errorMessage}]]></${node}>`;
       }
 
       const groupedByLevel: Record<string, string[]> = {};
@@ -78,8 +81,7 @@ export class XunitEmitter implements TestConsumerShape {
       ${body}
       <system-out>${this.buildMeta({ log: groupedByLevel.log, info: groupedByLevel.info, debug: groupedByLevel.debug })}</system-out>
       <system-err>${this.buildMeta({ error: groupedByLevel.error, warn: groupedByLevel.warn })}</system-err>
-    </testcase>`
-      );
+    </testcase>`);
     } else if (event.type === 'suite' && event.phase === 'after') {
       const { suite } = event;
       const testBodies = this.#tests.slice(0);
@@ -120,4 +122,4 @@ export class XunitEmitter implements TestConsumerShape {
 </testsuites>
 `);
   }
-}
+}

package/src/consumer/types.ts

@@ -1,24 +1,16 @@
 import type { Class } from '@travetto/runtime';
 
 import type { TestEvent, TestRemoveEvent } from '../model/event.ts';
-import type { Counts, SuiteResult } from '../model/suite.ts';
+import type { ResultsSummary, SuiteResult } from '../model/suite.ts';
 
 /**
  * All suite results
  */
-export interface SuitesSummary extends Counts {
+export interface SuitesSummary extends ResultsSummary {
   /**
    * List of all suites
    */
   suites: SuiteResult[];
-  /**
-   * List of all errors
-   */
-  errors: Error[];
-  /**
-   * Total duration
-   */
-  duration: number;
 }
 
 export type TestRunState = {
@@ -44,6 +36,10 @@ export interface TestConsumerShape {
    * Set options
    */
   setOptions?(options?: Record<string, unknown>): Promise<void> | void;
+  /**
+   * Can directly update the known test run state as needed
+   */
+  onTestRunState?(state: TestRunState): Promise<void> | void;
   /**
    * Listen for start of the test run
    */