FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test,tools: adjust function argument alignment · nodejs/node@6d1606e · GitHub

/ node Public

Commit 6d1606e

Browse files
Trott
authored and
Fishrock123
committed
test,tools: adjust function argument alignment
In preparation for a lint rule enforcing function argument alignment, adjust function arguments to be aligned. PR-URL: #6390 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Imran Iqbal <imran@imraniqbal.org> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Ryan Graham <r.m.graham@gmail.com>
1 parent fdde369 commit 6d1606e

29 files changed

Lines changed: 111 additions & 79 deletions

‎test/internet/test-dgram-multicast-multi-process.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ function launchChildProcess(index) {
9191
worker.pid, count);
9292

9393
assert.strictEqual(count, messages.length,
94-
'A worker received an invalid multicast message');
94+
'A worker received an invalid multicast message');
9595
});
9696

9797
clearTimeout(timer);

‎test/parallel/test-assert.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -462,7 +462,7 @@ try {
462462
} catch (e) {
463463
assert.equal(e.toString().split('\n')[0], 'AssertionError: oh no');
464464
assert.equal(e.generatedMessage, false,
465-
'Message incorrectly marked as generated');
465+
'Message incorrectly marked as generated');
466466
}
467467

468468
// Verify that throws() and doesNotThrow() throw on non-function block

‎test/parallel/test-buffer-alloc.js‎

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -645,18 +645,30 @@ assert.equal(Buffer.from('KioqKioqKioqKioqKioqKioqKio', 'base64').toString(),
645645
'********************');
646646

647647
// handle padding graciously, multiple-of-4 or not
648-
assert.equal(Buffer.from('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw==',
649-
'base64').length, 32);
650-
assert.equal(Buffer.from('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw=',
651-
'base64').length, 32);
652-
assert.equal(Buffer.from('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw',
653-
'base64').length, 32);
654-
assert.equal(Buffer.from('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg==',
655-
'base64').length, 31);
656-
assert.equal(Buffer.from('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg=',
657-
'base64').length, 31);
658-
assert.equal(Buffer.from('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg',
659-
'base64').length, 31);
648+
assert.equal(
649+
Buffer.from('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw==', 'base64').length,
650+
32
651+
);
652+
assert.equal(
653+
Buffer.from('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw=', 'base64').length,
654+
32
655+
);
656+
assert.equal(
657+
Buffer.from('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw', 'base64').length,
658+
32
659+
);
660+
assert.equal(
661+
Buffer.from('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg==', 'base64').length,
662+
31
663+
);
664+
assert.equal(
665+
Buffer.from('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg=', 'base64').length,
666+
31
667+
);
668+
assert.equal(
669+
Buffer.from('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg', 'base64').length,
670+
31
671+
);
660672

661673
// This string encodes single '.' character in UTF-16
662674
var dot = Buffer.from('//4uAA==', 'base64');

‎test/parallel/test-buffer.js‎

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -643,18 +643,30 @@ assert.equal(new Buffer('KioqKioqKioqKioqKioqKioqKio', 'base64').toString(),
643643
'********************');
644644

645645
// handle padding graciously, multiple-of-4 or not
646-
assert.equal(new Buffer('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw==',
647-
'base64').length, 32);
648-
assert.equal(new Buffer('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw=',
649-
'base64').length, 32);
650-
assert.equal(new Buffer('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw',
651-
'base64').length, 32);
652-
assert.equal(new Buffer('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg==',
653-
'base64').length, 31);
654-
assert.equal(new Buffer('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg=',
655-
'base64').length, 31);
656-
assert.equal(new Buffer('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg',
657-
'base64').length, 31);
646+
assert.equal(
647+
new Buffer('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw==', 'base64').length,
648+
32
649+
);
650+
assert.equal(
651+
new Buffer('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw=', 'base64').length,
652+
32
653+
);
654+
assert.equal(
655+
new Buffer('72INjkR5fchcxk9+VgdGPFJDxUBFR5/rMFsghgxADiw', 'base64').length,
656+
32
657+
);
658+
assert.equal(
659+
new Buffer('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg==', 'base64').length,
660+
31
661+
);
662+
assert.equal(
663+
new Buffer('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg=', 'base64').length,
664+
31
665+
);
666+
assert.equal(
667+
new Buffer('w69jACy6BgZmaFvv96HG6MYksWytuZu3T1FvGnulPg', 'base64').length,
668+
31
669+
);
658670

659671
// This string encodes single '.' character in UTF-16
660672
var dot = new Buffer('//4uAA==', 'base64');

‎test/parallel/test-cluster-basic.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ var assert = require('assert');
44
var cluster = require('cluster');
55

66
assert.equal('NODE_UNIQUE_ID' in process.env, false,
7-
'NODE_UNIQUE_ID should be removed on startup');
7+
'NODE_UNIQUE_ID should be removed on startup');
88

99
function forEach(obj, fn) {
1010
Object.keys(obj).forEach(function(name, index) {
@@ -93,7 +93,7 @@ else if (cluster.isMaster) {
9393
worker = cluster.fork();
9494
assert.equal(worker.id, 1);
9595
assert.ok(worker instanceof cluster.Worker,
96-
'the worker is not a instance of the Worker constructor');
96+
'the worker is not a instance of the Worker constructor');
9797

9898
//Check event
9999
forEach(checks.worker.events, function(bool, name, index) {

‎test/parallel/test-crypto-binary-default.js‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -347,8 +347,11 @@ var a4 = crypto.createHash('sha1').update('Test123').digest('buffer');
347347

348348
if (!common.hasFipsCrypto) {
349349
var a0 = crypto.createHash('md5').update('Test123').digest('binary');
350-
assert.equal(a0, 'h\u00ea\u00cb\u0097\u00d8o\fF!\u00fa+\u000e\u0017\u00ca' +
351-
'\u00bd\u008c', 'Test MD5 as binary');
350+
assert.equal(
351+
a0,
352+
'h\u00ea\u00cb\u0097\u00d8o\fF!\u00fa+\u000e\u0017\u00ca\u00bd\u008c',
353+
'Test MD5 as binary'
354+
);
352355
}
353356

354357
assert.equal(a1, '8308651804facb7b9af8ffc53a33a22d6a1c8ac2', 'Test SHA1');

‎test/parallel/test-crypto-hash.js‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,11 @@ a8 = a8.read();
3939

4040
if (!common.hasFipsCrypto) {
4141
var a0 = crypto.createHash('md5').update('Test123').digest('binary');
42-
assert.equal(a0, 'h\u00ea\u00cb\u0097\u00d8o\fF!\u00fa+\u000e\u0017\u00ca' +
43-
'\u00bd\u008c', 'Test MD5 as binary');
42+
assert.equal(
43+
a0,
44+
'h\u00ea\u00cb\u0097\u00d8o\fF!\u00fa+\u000e\u0017\u00ca\u00bd\u008c',
45+
'Test MD5 as binary'
46+
);
4447
}
4548
assert.equal(a1, '8308651804facb7b9af8ffc53a33a22d6a1c8ac2', 'Test SHA1');
4649
assert.equal(a2, '2bX1jws4GYKTlxhloUB09Z66PoJZW+y+hq5R8dnx9l4=',

‎test/parallel/test-domain-exit-dispose-again.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ setTimeout(function firstTimer() {
3838
d.dispose();
3939
console.error(err);
4040
console.error('in domain error handler',
41-
process.domain, process.domain === d);
41+
process.domain, process.domain === d);
4242
});
4343

4444
d.run(function() {

‎test/parallel/test-domain-stack-empty-in-process-uncaughtexception.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,12 @@ const d = domain.create();
88

99
process.on('uncaughtException', common.mustCall(function onUncaught() {
1010
assert.equal(process.domain, null,
11-
'domains stack should be empty in uncaughtException handler');
11+
'domains stack should be empty in uncaughtException handler');
1212
}));
1313

1414
process.on('beforeExit', common.mustCall(function onBeforeExit() {
1515
assert.equal(process.domain, null,
16-
'domains stack should be empty in beforeExit handler');
16+
'domains stack should be empty in beforeExit handler');
1717
}));
1818

1919
d.run(function() {

‎test/parallel/test-domain-throw-error-then-throw-from-uncaught-exception-handler.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,8 @@ function runTestWithoutAbortOnUncaughtException() {
6262
'include domain\'s error\'s message');
6363

6464
assert.notEqual(err.code, 0,
65-
'child process should have exited with a non-zero exit code, ' +
66-
'but did not');
65+
'child process should have exited with a non-zero ' +
66+
'exit code, but did not');
6767
});
6868
}
6969

@@ -72,8 +72,8 @@ function runTestWithAbortOnUncaughtException() {
7272
withAbortOnUncaughtException: true
7373
}), function onTestDone(err, stdout, stderr) {
7474
assert.notEqual(err.code, RAN_UNCAUGHT_EXCEPTION_HANDLER_EXIT_CODE,
75-
'child process should not have run its uncaughtException event ' +
76-
'handler');
75+
'child process should not have run its uncaughtException ' +
76+
'event handler');
7777
assert(common.nodeProcessAborted(err.code, err.signal),
7878
'process should have aborted, but did not');
7979
});

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL