Treat duration_ms as a time directive

Fix: #895
tapjs · Sep 19, 2023 · b03f951 · b03f951
1 parent e42ea1a
commit b03f951
Show file tree

Hide file tree

Showing 5 changed files with 1,190 additions and 0 deletions.
diff --git a/src/parser/src/index.ts b/src/parser/src/index.ts
@@ -358,6 +358,14 @@ export class Parser
       return
     }
 
+    if (
+      typeof diags.duration_ms === 'number' &&
+      this.#current.time === null
+    ) {
+      this.#current.time = diags.duration_ms
+      delete diags.duration_ms
+    }
+
     this.#current.diag = diags
     // we still don't emit the result here yet, to support diags
     // that come ahead of buffered subtests.

diff --git a/src/parser/src/parse-directive.ts b/src/parser/src/parse-directive.ts
@@ -19,6 +19,7 @@ export const parseDirective = (
   if (!line.trim()) return false
 
   line = line.replace(OPEN_BRACE_EOL, '').trim()
+    .replace(/^duration_ms ([0-9.]+)$/, 'time=$1ms')
   const time = line.match(
     /^time=((?:[1-9][0-9]*|0)(?:\.[0-9]+)?)(ms|s)$/i
   )