Report error when parsing for line number selection

novemberborn · novemberborn · commit d57c76c47687 · 2022-07-10T17:54:20.000+02:00
This makes the failure a little less mysterious.
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'

Original file line number	Diff line number	Diff line change
`@@ -220,6 +220,9 @@ export default class Reporter {`
`220`	`220`	`this.lineNumberErrors.push(event);`
`221`	`221`
`222`	`222`	this.write(colors.information(`${figures.warning} Could not parse ${this.relativeFile(event.testFile)} for line number selection`));
	`223`	`+ this.lineWriter.writeLine();`
	`224`	`+ this.lineWriter.writeLine(colors.errorStack(event.err.stack));`
	`225`	`+ this.lineWriter.writeLine();`
`223`	`226`	`break;`
`224`	`227`	`}`
`225`	`228`
-Original file line number
+Diff line change
@@ @@ -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