Skip to content

Commit

Permalink
test: make test-require-json engine agnostic
Browse files Browse the repository at this point in the history
Remove reliance on V8-specific error message in test-require-json.

PR-URL: #16272
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
  • Loading branch information
Trott authored and evanlucas committed Nov 13, 2017
1 parent 835ca63 commit 0be7f8c
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions test/parallel/test-require-json.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ try {
require(fixtures.path('invalid.json'));
} catch (err) {
assert.ok(
/test[/\\]fixtures[/\\]invalid\.json: Unexpected string/.test(err.message),
`require() json error should include path - currently: ${err.message}`
);
/test[/\\]fixtures[/\\]invalid\.json: /.test(err.message),
`require() json error should include path: ${err.message}`);
}

0 comments on commit 0be7f8c

Please sign in to comment.