From 01d40473be791c301b596272feef15e7611689cb Mon Sep 17 00:00:00 2001 From: francois Date: Fri, 12 Oct 2018 10:57:07 -0400 Subject: [PATCH] test: corrected assertion arguments order Fixed the order of expected arguments for assert.strictEqual() PR-URL: https://github.com/nodejs/node/pull/23569 Reviewed-By: Joyee Cheung Reviewed-By: James M Snell Reviewed-By: Ruben Bridgewater Reviewed-By: Trivikram Kamat Reviewed-By: Colin Ihrig Reviewed-By: Gireesh Punathil --- test/parallel/test-util.js | 139 +++++++++++++++++++------------------ 1 file changed, 73 insertions(+), 66 deletions(-) diff --git a/test/parallel/test-util.js b/test/parallel/test-util.js index 2f91417d106148..f0e2f642ea25d4 100644 --- a/test/parallel/test-util.js +++ b/test/parallel/test-util.js @@ -28,75 +28,75 @@ const errors = require('internal/errors'); const context = require('vm').runInNewContext; // isArray -assert.strictEqual(true, util.isArray([])); -assert.strictEqual(true, util.isArray(Array())); -assert.strictEqual(true, util.isArray(new Array())); -assert.strictEqual(true, util.isArray(new Array(5))); -assert.strictEqual(true, util.isArray(new Array('with', 'some', 'entries'))); -assert.strictEqual(true, util.isArray(context('Array')())); -assert.strictEqual(false, util.isArray({})); -assert.strictEqual(false, util.isArray({ push: function() {} })); -assert.strictEqual(false, util.isArray(/regexp/)); -assert.strictEqual(false, util.isArray(new Error())); -assert.strictEqual(false, util.isArray(Object.create(Array.prototype))); +assert.strictEqual(util.isArray([]), true); +assert.strictEqual(util.isArray(Array()), true); +assert.strictEqual(util.isArray(new Array()), true); +assert.strictEqual(util.isArray(new Array(5)), true); +assert.strictEqual(util.isArray(new Array('with', 'some', 'entries')), true); +assert.strictEqual(util.isArray(context('Array')()), true); +assert.strictEqual(util.isArray({}), false); +assert.strictEqual(util.isArray({ push: function() {} }), false); +assert.strictEqual(util.isArray(/regexp/), false); +assert.strictEqual(util.isArray(new Error()), false); +assert.strictEqual(util.isArray(Object.create(Array.prototype)), false); // isRegExp -assert.strictEqual(true, util.isRegExp(/regexp/)); -assert.strictEqual(true, util.isRegExp(RegExp(), 'foo')); -assert.strictEqual(true, util.isRegExp(new RegExp())); -assert.strictEqual(true, util.isRegExp(context('RegExp')())); -assert.strictEqual(false, util.isRegExp({})); -assert.strictEqual(false, util.isRegExp([])); -assert.strictEqual(false, util.isRegExp(new Date())); -assert.strictEqual(false, util.isRegExp(Object.create(RegExp.prototype))); +assert.strictEqual(util.isRegExp(/regexp/), true); +assert.strictEqual(util.isRegExp(RegExp(), 'foo'), true); +assert.strictEqual(util.isRegExp(new RegExp()), true); +assert.strictEqual(util.isRegExp(context('RegExp')()), true); +assert.strictEqual(util.isRegExp({}), false); +assert.strictEqual(util.isRegExp([]), false); +assert.strictEqual(util.isRegExp(new Date()), false); +assert.strictEqual(util.isRegExp(Object.create(RegExp.prototype)), false); // isDate -assert.strictEqual(true, util.isDate(new Date())); -assert.strictEqual(true, util.isDate(new Date(0), 'foo')); -assert.strictEqual(true, util.isDate(new (context('Date'))())); -assert.strictEqual(false, util.isDate(Date())); -assert.strictEqual(false, util.isDate({})); -assert.strictEqual(false, util.isDate([])); -assert.strictEqual(false, util.isDate(new Error())); -assert.strictEqual(false, util.isDate(Object.create(Date.prototype))); +assert.strictEqual(util.isDate(new Date()), true); +assert.strictEqual(util.isDate(new Date(0), 'foo'), true); +assert.strictEqual(util.isDate(new (context('Date'))()), true); +assert.strictEqual(util.isDate(Date()), false); +assert.strictEqual(util.isDate({}), false); +assert.strictEqual(util.isDate([]), false); +assert.strictEqual(util.isDate(new Error()), false); +assert.strictEqual(util.isDate(Object.create(Date.prototype)), false); // isError -assert.strictEqual(true, util.isError(new Error())); -assert.strictEqual(true, util.isError(new TypeError())); -assert.strictEqual(true, util.isError(new SyntaxError())); -assert.strictEqual(true, util.isError(new (context('Error'))())); -assert.strictEqual(true, util.isError(new (context('TypeError'))())); -assert.strictEqual(true, util.isError(new (context('SyntaxError'))())); -assert.strictEqual(false, util.isError({})); -assert.strictEqual(false, util.isError({ name: 'Error', message: '' })); -assert.strictEqual(false, util.isError([])); -assert.strictEqual(true, util.isError(Object.create(Error.prototype))); +assert.strictEqual(util.isError(new Error()), true); +assert.strictEqual(util.isError(new TypeError()), true); +assert.strictEqual(util.isError(new SyntaxError()), true); +assert.strictEqual(util.isError(new (context('Error'))()), true); +assert.strictEqual(util.isError(new (context('TypeError'))()), true); +assert.strictEqual(util.isError(new (context('SyntaxError'))()), true); +assert.strictEqual(util.isError({}), false); +assert.strictEqual(util.isError({ name: 'Error', message: '' }), false); +assert.strictEqual(util.isError([]), false); +assert.strictEqual(util.isError(Object.create(Error.prototype)), true); // isObject assert.ok(util.isObject({}) === true); // isPrimitive -assert.strictEqual(false, util.isPrimitive({})); -assert.strictEqual(false, util.isPrimitive(new Error())); -assert.strictEqual(false, util.isPrimitive(new Date())); -assert.strictEqual(false, util.isPrimitive([])); -assert.strictEqual(false, util.isPrimitive(/regexp/)); -assert.strictEqual(false, util.isPrimitive(function() {})); -assert.strictEqual(false, util.isPrimitive(new Number(1))); -assert.strictEqual(false, util.isPrimitive(new String('bla'))); -assert.strictEqual(false, util.isPrimitive(new Boolean(true))); -assert.strictEqual(true, util.isPrimitive(1)); -assert.strictEqual(true, util.isPrimitive('bla')); -assert.strictEqual(true, util.isPrimitive(true)); -assert.strictEqual(true, util.isPrimitive(undefined)); -assert.strictEqual(true, util.isPrimitive(null)); -assert.strictEqual(true, util.isPrimitive(Infinity)); -assert.strictEqual(true, util.isPrimitive(NaN)); -assert.strictEqual(true, util.isPrimitive(Symbol('symbol'))); +assert.strictEqual(util.isPrimitive({}), false); +assert.strictEqual(util.isPrimitive(new Error()), false); +assert.strictEqual(util.isPrimitive(new Date()), false); +assert.strictEqual(util.isPrimitive([]), false); +assert.strictEqual(util.isPrimitive(/regexp/), false); +assert.strictEqual(util.isPrimitive(function() {}), false); +assert.strictEqual(util.isPrimitive(new Number(1)), false); +assert.strictEqual(util.isPrimitive(new String('bla')), false); +assert.strictEqual(util.isPrimitive(new Boolean(true)), false); +assert.strictEqual(util.isPrimitive(1), true); +assert.strictEqual(util.isPrimitive('bla'), true); +assert.strictEqual(util.isPrimitive(true), true); +assert.strictEqual(util.isPrimitive(undefined), true); +assert.strictEqual(util.isPrimitive(null), true); +assert.strictEqual(util.isPrimitive(Infinity), true); +assert.strictEqual(util.isPrimitive(NaN), true); +assert.strictEqual(util.isPrimitive(Symbol('symbol')), true); // isBuffer -assert.strictEqual(false, util.isBuffer('foo')); -assert.strictEqual(true, util.isBuffer(Buffer.from('foo'))); +assert.strictEqual(util.isBuffer('foo'), false); +assert.strictEqual(util.isBuffer(Buffer.from('foo')), true); // _extend assert.deepStrictEqual(util._extend({ a: 1 }), { a: 1 }); @@ -160,18 +160,25 @@ util.error('test'); assert.strictEqual(util.types.isNativeError(new Error()), true); assert.strictEqual(util.types.isNativeError(new TypeError()), true); assert.strictEqual(util.types.isNativeError(new SyntaxError()), true); - assert.strictEqual(util.types.isNativeError(new (context('Error'))()), - true); - assert.strictEqual(util.types.isNativeError(new (context('TypeError'))()), - true); - assert.strictEqual(util.types.isNativeError(new (context('SyntaxError'))()), - true); + assert.strictEqual(util.types.isNativeError(new (context('Error'))()), true); + assert.strictEqual( + util.types.isNativeError(new (context('TypeError'))()), + true + ); + assert.strictEqual( + util.types.isNativeError(new (context('SyntaxError'))()), + true + ); assert.strictEqual(util.types.isNativeError({}), false); - assert.strictEqual(util.types.isNativeError({ name: 'Error', message: '' }), - false); + assert.strictEqual( + util.types.isNativeError({ name: 'Error', message: '' }), + false + ); assert.strictEqual(util.types.isNativeError([]), false); - assert.strictEqual(util.types.isNativeError(Object.create(Error.prototype)), - false); + assert.strictEqual( + util.types.isNativeError(Object.create(Error.prototype)), + false + ); assert.strictEqual( util.types.isNativeError(new errors.codes.ERR_IPC_CHANNEL_CLOSED()), true