Report error when parsing for line number selection

This makes the failure a little less mysterious.
avajs · Jul 10, 2022 · 5899923 · 5899923
1 parent ac0d75d
commit 5899923
Show file tree

Hide file tree

Showing 5 changed files with 55 additions and 4 deletions.
diff --git a/lib/reporters/default.js b/lib/reporters/default.js
@@ -220,6 +220,9 @@ export default class Reporter {
 				this.lineNumberErrors.push(event);
 
 				this.write(colors.information(`${figures.warning} Could not parse ${this.relativeFile(event.testFile)} for line number selection`));
+				this.lineWriter.writeLine();
+				this.lineWriter.writeLine(colors.errorStack(event.err.stack));
+				this.lineWriter.writeLine();
 				break;
 			}
 

diff --git a/test-tap/reporters/default.edgecases.v12.log b/test-tap/reporters/default.edgecases.v12.log
@@ -1,8 +1,20 @@
 
 ---tty-stream-chunk-separator
   [35m⚠ Could not parse ast-syntax-error.cjs for line number selection[39m
----tty-stream-chunk-separator
 
+  [90mSyntaxError: Unexpected token (3:11)[39m
+  [90m    at Parser.pp$4.raise (~/node_modules/acorn/dist/acorn.js:3462:15)[39m
+  [90m    at Parser.pp$9.unexpected (~/node_modules/acorn/dist/acorn.js:756:10)[39m
+  [90m    at Parser.pp$5.parseExprAtom (~/node_modules/acorn/dist/acorn.js:2837:12)[39m
+  [90m    at Parser.pp$5.parseExprSubscripts (~/node_modules/acorn/dist/acorn.js:2629:21)[39m
+  [90m    at Parser.pp$5.parseMaybeUnary (~/node_modules/acorn/dist/acorn.js:2595:19)[39m
+  [90m    at Parser.pp$5.parseExprOps (~/node_modules/acorn/dist/acorn.js:2522:21)[39m
+  [90m    at Parser.pp$5.parseMaybeConditional (~/node_modules/acorn/dist/acorn.js:2505:21)[39m
+  [90m    at Parser.pp$5.parseMaybeAssign (~/node_modules/acorn/dist/acorn.js:2472:21)[39m
+  [90m    at Parser.pp$8.parseVar (~/node_modules/acorn/dist/acorn.js:1299:26)[39m
+  [90m    at Parser.pp$8.parseVarStatement (~/node_modules/acorn/dist/acorn.js:1163:10)[39m
+
+---tty-stream-chunk-separator
   [1mUncaught exception in ast-syntax-error.cjs[22m
 
   ~/test-tap/fixture/report/edgecases/ast-syntax-error.cjs:3

diff --git a/test-tap/reporters/default.edgecases.v14.log b/test-tap/reporters/default.edgecases.v14.log
@@ -1,8 +1,20 @@
 
 ---tty-stream-chunk-separator
   [35m⚠ Could not parse ast-syntax-error.cjs for line number selection[39m
----tty-stream-chunk-separator
 
+  [90mSyntaxError: Unexpected token (3:11)[39m
+  [90m    at Parser.pp$4.raise (~/node_modules/acorn/dist/acorn.js:3462:15)[39m
+  [90m    at Parser.pp$9.unexpected (~/node_modules/acorn/dist/acorn.js:756:10)[39m
+  [90m    at Parser.pp$5.parseExprAtom (~/node_modules/acorn/dist/acorn.js:2837:12)[39m
+  [90m    at Parser.pp$5.parseExprSubscripts (~/node_modules/acorn/dist/acorn.js:2629:21)[39m
+  [90m    at Parser.pp$5.parseMaybeUnary (~/node_modules/acorn/dist/acorn.js:2595:19)[39m
+  [90m    at Parser.pp$5.parseExprOps (~/node_modules/acorn/dist/acorn.js:2522:21)[39m
+  [90m    at Parser.pp$5.parseMaybeConditional (~/node_modules/acorn/dist/acorn.js:2505:21)[39m
+  [90m    at Parser.pp$5.parseMaybeAssign (~/node_modules/acorn/dist/acorn.js:2472:21)[39m
+  [90m    at Parser.pp$8.parseVar (~/node_modules/acorn/dist/acorn.js:1299:26)[39m
+  [90m    at Parser.pp$8.parseVarStatement (~/node_modules/acorn/dist/acorn.js:1163:10)[39m
+
+---tty-stream-chunk-separator
   [1mUncaught exception in ast-syntax-error.cjs[22m
 
   SyntaxError: Unexpected token 'do'

diff --git a/test-tap/reporters/default.edgecases.v16.log b/test-tap/reporters/default.edgecases.v16.log
@@ -1,8 +1,20 @@
 
 ---tty-stream-chunk-separator
   [35m⚠ Could not parse ast-syntax-error.cjs for line number selection[39m
----tty-stream-chunk-separator
 
+  [90mSyntaxError: Unexpected token (3:11)[39m
+  [90m    at Parser.pp$4.raise (~/node_modules/acorn/dist/acorn.js:3462:15)[39m
+  [90m    at Parser.pp$9.unexpected (~/node_modules/acorn/dist/acorn.js:756:10)[39m
+  [90m    at Parser.pp$5.parseExprAtom (~/node_modules/acorn/dist/acorn.js:2837:12)[39m
+  [90m    at Parser.pp$5.parseExprSubscripts (~/node_modules/acorn/dist/acorn.js:2629:21)[39m
+  [90m    at Parser.pp$5.parseMaybeUnary (~/node_modules/acorn/dist/acorn.js:2595:19)[39m
+  [90m    at Parser.pp$5.parseExprOps (~/node_modules/acorn/dist/acorn.js:2522:21)[39m
+  [90m    at Parser.pp$5.parseMaybeConditional (~/node_modules/acorn/dist/acorn.js:2505:21)[39m
+  [90m    at Parser.pp$5.parseMaybeAssign (~/node_modules/acorn/dist/acorn.js:2472:21)[39m
+  [90m    at Parser.pp$8.parseVar (~/node_modules/acorn/dist/acorn.js:1299:26)[39m
+  [90m    at Parser.pp$8.parseVarStatement (~/node_modules/acorn/dist/acorn.js:1163:10)[39m
+
+---tty-stream-chunk-separator
   [1mUncaught exception in ast-syntax-error.cjs[22m
 
   SyntaxError: Unexpected token 'do'

diff --git a/test-tap/reporters/default.edgecases.v18.log b/test-tap/reporters/default.edgecases.v18.log
@@ -1,8 +1,20 @@
 
 ---tty-stream-chunk-separator
   [35m⚠ Could not parse ast-syntax-error.cjs for line number selection[39m
----tty-stream-chunk-separator
 
+  [90mSyntaxError: Unexpected token (3:11)[39m
+  [90m    at pp$4.raise (~/node_modules/acorn/dist/acorn.js:3462:15)[39m
+  [90m    at pp$9.unexpected (~/node_modules/acorn/dist/acorn.js:756:10)[39m
+  [90m    at pp$5.parseExprAtom (~/node_modules/acorn/dist/acorn.js:2837:12)[39m
+  [90m    at pp$5.parseExprSubscripts (~/node_modules/acorn/dist/acorn.js:2629:21)[39m
+  [90m    at pp$5.parseMaybeUnary (~/node_modules/acorn/dist/acorn.js:2595:19)[39m
+  [90m    at pp$5.parseExprOps (~/node_modules/acorn/dist/acorn.js:2522:21)[39m
+  [90m    at pp$5.parseMaybeConditional (~/node_modules/acorn/dist/acorn.js:2505:21)[39m
+  [90m    at pp$5.parseMaybeAssign (~/node_modules/acorn/dist/acorn.js:2472:21)[39m
+  [90m    at pp$8.parseVar (~/node_modules/acorn/dist/acorn.js:1299:26)[39m
+  [90m    at pp$8.parseVarStatement (~/node_modules/acorn/dist/acorn.js:1163:10)[39m
+
+---tty-stream-chunk-separator
   [1mUncaught exception in ast-syntax-error.cjs[22m
 
   SyntaxError: Unexpected token 'do'