Skip to content

Commit 0715294

Browse files
aredridelljharb
authored andcommitted
[Refactor] Avoid setting message property on primitives; use strict mode to catch this
1 parent e925bb1 commit 0715294

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

lib/test.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
"use strict";
12
var deepEqual = require('deep-equal');
23
var defined = require('defined');
34
var path = require('path');
@@ -543,7 +544,7 @@ Test.prototype['throws'] = function (fn, expected, msg, extra) {
543544
fn();
544545
} catch (err) {
545546
caught = { error: err };
546-
if (err != null && (!isEnumerable(err, 'message') || !has(err, 'message'))) {
547+
if (Object(err) === err && (!isEnumerable(err, 'message') || !has(err, 'message'))) {
547548
var message = err.message;
548549
delete err.message;
549550
err.message = message;

0 commit comments

Comments
 (0)