1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: corrected assertion arguments order

Browse Source 
Fixed the order of expected arguments for assert.strictEqual()

PR-URL: https://github.com/nodejs/node/pull/23569
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
This commit is contained in:
francois 2018-10-12 10:57:07 -04:00 committed by Ruben Bridgewater
parent 552fe56b65
commit 01d40473be
No known key found for this signature in database
GPG Key ID: F07496B3EB3C1762
1 changed files with 73 additions and 66 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

139
test/parallel/test-util.js
View File

@ -28,75 +28,75 @@ const errors = require('internal/errors');
const context = require('vm').runInNewContext; const context = require('vm').runInNewContext;
// isArray // isArray
assert.strictEqual(true, util.isArray([])); assert.strictEqual(util.isArray([]), true);
assert.strictEqual(true, util.isArray(Array())); assert.strictEqual(util.isArray(Array()), true);
assert.strictEqual(true, util.isArray(new Array())); assert.strictEqual(util.isArray(new Array()), true);
assert.strictEqual(true, util.isArray(new Array(5))); assert.strictEqual(util.isArray(new Array(5)), true);
assert.strictEqual(true, util.isArray(new Array('with', 'some', 'entries'))); assert.strictEqual(util.isArray(new Array('with', 'some', 'entries')), true);
assert.strictEqual(true, util.isArray(context('Array')())); assert.strictEqual(util.isArray(context('Array')()), true);
assert.strictEqual(false, util.isArray({})); assert.strictEqual(util.isArray({}), false);
assert.strictEqual(false, util.isArray({ push: function() {} })); assert.strictEqual(util.isArray({ push: function() {} }), false);
assert.strictEqual(false, util.isArray(/regexp/)); assert.strictEqual(util.isArray(/regexp/), false);
assert.strictEqual(false, util.isArray(new Error())); assert.strictEqual(util.isArray(new Error()), false);
assert.strictEqual(false, util.isArray(Object.create(Array.prototype))); assert.strictEqual(util.isArray(Object.create(Array.prototype)), false);
// isRegExp // isRegExp
assert.strictEqual(true, util.isRegExp(/regexp/)); assert.strictEqual(util.isRegExp(/regexp/), true);
assert.strictEqual(true, util.isRegExp(RegExp(), 'foo')); assert.strictEqual(util.isRegExp(RegExp(), 'foo'), true);
assert.strictEqual(true, util.isRegExp(new RegExp())); assert.strictEqual(util.isRegExp(new RegExp()), true);
assert.strictEqual(true, util.isRegExp(context('RegExp')())); assert.strictEqual(util.isRegExp(context('RegExp')()), true);
assert.strictEqual(false, util.isRegExp({})); assert.strictEqual(util.isRegExp({}), false);
assert.strictEqual(false, util.isRegExp([])); assert.strictEqual(util.isRegExp([]), false);
assert.strictEqual(false, util.isRegExp(new Date())); assert.strictEqual(util.isRegExp(new Date()), false);
assert.strictEqual(false, util.isRegExp(Object.create(RegExp.prototype))); assert.strictEqual(util.isRegExp(Object.create(RegExp.prototype)), false);
// isDate // isDate
assert.strictEqual(true, util.isDate(new Date())); assert.strictEqual(util.isDate(new Date()), true);
assert.strictEqual(true, util.isDate(new Date(0), 'foo')); assert.strictEqual(util.isDate(new Date(0), 'foo'), true);
assert.strictEqual(true, util.isDate(new (context('Date'))())); assert.strictEqual(util.isDate(new (context('Date'))()), true);
assert.strictEqual(false, util.isDate(Date())); assert.strictEqual(util.isDate(Date()), false);
assert.strictEqual(false, util.isDate({})); assert.strictEqual(util.isDate({}), false);
assert.strictEqual(false, util.isDate([])); assert.strictEqual(util.isDate([]), false);
assert.strictEqual(false, util.isDate(new Error())); assert.strictEqual(util.isDate(new Error()), false);
assert.strictEqual(false, util.isDate(Object.create(Date.prototype))); assert.strictEqual(util.isDate(Object.create(Date.prototype)), false);
// isError // isError
assert.strictEqual(true, util.isError(new Error())); assert.strictEqual(util.isError(new Error()), true);
assert.strictEqual(true, util.isError(new TypeError())); assert.strictEqual(util.isError(new TypeError()), true);
assert.strictEqual(true, util.isError(new SyntaxError())); assert.strictEqual(util.isError(new SyntaxError()), true);
assert.strictEqual(true, util.isError(new (context('Error'))())); assert.strictEqual(util.isError(new (context('Error'))()), true);
assert.strictEqual(true, util.isError(new (context('TypeError'))())); assert.strictEqual(util.isError(new (context('TypeError'))()), true);
assert.strictEqual(true, util.isError(new (context('SyntaxError'))())); assert.strictEqual(util.isError(new (context('SyntaxError'))()), true);
assert.strictEqual(false, util.isError({})); assert.strictEqual(util.isError({}), false);
assert.strictEqual(false, util.isError({ name: 'Error', message: '' })); assert.strictEqual(util.isError({ name: 'Error', message: '' }), false);
assert.strictEqual(false, util.isError([])); assert.strictEqual(util.isError([]), false);
assert.strictEqual(true, util.isError(Object.create(Error.prototype))); assert.strictEqual(util.isError(Object.create(Error.prototype)), true);
// isObject // isObject
assert.ok(util.isObject({}) === true); assert.ok(util.isObject({}) === true);
// isPrimitive // isPrimitive
assert.strictEqual(false, util.isPrimitive({})); assert.strictEqual(util.isPrimitive({}), false);
assert.strictEqual(false, util.isPrimitive(new Error())); assert.strictEqual(util.isPrimitive(new Error()), false);
assert.strictEqual(false, util.isPrimitive(new Date())); assert.strictEqual(util.isPrimitive(new Date()), false);
assert.strictEqual(false, util.isPrimitive([])); assert.strictEqual(util.isPrimitive([]), false);
assert.strictEqual(false, util.isPrimitive(/regexp/)); assert.strictEqual(util.isPrimitive(/regexp/), false);
assert.strictEqual(false, util.isPrimitive(function() {})); assert.strictEqual(util.isPrimitive(function() {}), false);
assert.strictEqual(false, util.isPrimitive(new Number(1))); assert.strictEqual(util.isPrimitive(new Number(1)), false);
assert.strictEqual(false, util.isPrimitive(new String('bla'))); assert.strictEqual(util.isPrimitive(new String('bla')), false);
assert.strictEqual(false, util.isPrimitive(new Boolean(true))); assert.strictEqual(util.isPrimitive(new Boolean(true)), false);
assert.strictEqual(true, util.isPrimitive(1)); assert.strictEqual(util.isPrimitive(1), true);
assert.strictEqual(true, util.isPrimitive('bla')); assert.strictEqual(util.isPrimitive('bla'), true);
assert.strictEqual(true, util.isPrimitive(true)); assert.strictEqual(util.isPrimitive(true), true);
assert.strictEqual(true, util.isPrimitive(undefined)); assert.strictEqual(util.isPrimitive(undefined), true);
assert.strictEqual(true, util.isPrimitive(null)); assert.strictEqual(util.isPrimitive(null), true);
assert.strictEqual(true, util.isPrimitive(Infinity)); assert.strictEqual(util.isPrimitive(Infinity), true);
assert.strictEqual(true, util.isPrimitive(NaN)); assert.strictEqual(util.isPrimitive(NaN), true);
assert.strictEqual(true, util.isPrimitive(Symbol('symbol'))); assert.strictEqual(util.isPrimitive(Symbol('symbol')), true);
// isBuffer // isBuffer
assert.strictEqual(false, util.isBuffer('foo')); assert.strictEqual(util.isBuffer('foo'), false);
assert.strictEqual(true, util.isBuffer(Buffer.from('foo'))); assert.strictEqual(util.isBuffer(Buffer.from('foo')), true);
// _extend // _extend
assert.deepStrictEqual(util._extend({ a: 1 }), { a: 1 }); 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 Error()), true);
assert.strictEqual(util.types.isNativeError(new TypeError()), true); assert.strictEqual(util.types.isNativeError(new TypeError()), true);
assert.strictEqual(util.types.isNativeError(new SyntaxError()), true); assert.strictEqual(util.types.isNativeError(new SyntaxError()), true);
assert.strictEqual(util.types.isNativeError(new (context('Error'))()), assert.strictEqual(util.types.isNativeError(new (context('Error'))()), true);
true); assert.strictEqual(
assert.strictEqual(util.types.isNativeError(new (context('TypeError'))()), util.types.isNativeError(new (context('TypeError'))()),
true); true
assert.strictEqual(util.types.isNativeError(new (context('SyntaxError'))()), );
true); assert.strictEqual(
util.types.isNativeError(new (context('SyntaxError'))()),
true
);
assert.strictEqual(util.types.isNativeError({}), false); assert.strictEqual(util.types.isNativeError({}), false);
assert.strictEqual(util.types.isNativeError({ name: 'Error', message: '' }), assert.strictEqual(
false); util.types.isNativeError({ name: 'Error', message: '' }),
false
);
assert.strictEqual(util.types.isNativeError([]), false); assert.strictEqual(util.types.isNativeError([]), false);
assert.strictEqual(util.types.isNativeError(Object.create(Error.prototype)), assert.strictEqual(
false); util.types.isNativeError(Object.create(Error.prototype)),
false
);
assert.strictEqual( assert.strictEqual(
util.types.isNativeError(new errors.codes.ERR_IPC_CHANNEL_CLOSED()), util.types.isNativeError(new errors.codes.ERR_IPC_CHANNEL_CLOSED()),
true true