From b6e174b4f5c4b3e62274ba542640ddc6bc7ced7a Mon Sep 17 00:00:00 2001
From: Alejandro Nanez <alejonanez@gmail.com>
Date: Fri, 21 Jun 2019 16:58:01 -0500
Subject: [PATCH] test: use assert.throws() in test-require-json.js

Use assert.throws() instead of try/catch.

PR-URL: https://github.com/nodejs/node/pull/28358
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
---
 test/parallel/test-require-json.js | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/test/parallel/test-require-json.js b/test/parallel/test-require-json.js
index e8c7b05bab8a57..9da2b15b27a7af 100644
--- a/test/parallel/test-require-json.js
+++ b/test/parallel/test-require-json.js
@@ -24,10 +24,9 @@ require('../common');
 const assert = require('assert');
 const fixtures = require('../common/fixtures');
 
-try {
+assert.throws(function() {
   require(fixtures.path('invalid.json'));
-} catch (err) {
-  assert.ok(
-    /test[/\\]fixtures[/\\]invalid\.json: /.test(err.message),
-    `require() json error should include path: ${err.message}`);
-}
+}, {
+  name: 'SyntaxError',
+  message: /test[/\\]fixtures[/\\]invalid\.json: /,
+});