From ca3d131bd42afa0c68be6aeb27623d53eae547a1 Mon Sep 17 00:00:00 2001 From: John Maguire Date: Fri, 27 Jan 2017 15:09:36 -0500 Subject: [PATCH] test-console: streamline arrow fn and refine regex removed unneccessary curly braces and return statement from inspect arrow function updated `assert.throws` regex to look for exact match at start of string PR-URL: https://github.com/nodejs/node/pull/11039 Reviewed-By: Luigi Pinca Reviewed-By: Gibson Fahnestock Reviewed-By: Rich Trott Reviewed-By: James M Snell --- test/parallel/test-console.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-console.js b/test/parallel/test-console.js index 0eb7924cb6cbdc..7bf1a5215f89e8 100644 --- a/test/parallel/test-console.js +++ b/test/parallel/test-console.js @@ -22,7 +22,7 @@ assert.doesNotThrow(function() { }); // an Object with a custom .inspect() function -const custom_inspect = { foo: 'bar', inspect: () => { return 'inspect'; } }; +const custom_inspect = { foo: 'bar', inspect: () => 'inspect' }; const stdout_write = global.process.stdout.write; const stderr_write = global.process.stderr.write; @@ -130,7 +130,7 @@ assert.strictEqual(errStrings.length, 0); assert.throws(() => { console.assert(false, 'should throw'); -}, /should throw/); +}, /^AssertionError: should throw$/); assert.doesNotThrow(() => { console.assert(true, 'this should not throw');