diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2019-01-21 01:22:27 +0100 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2019-02-28 18:31:10 +0100 |
commit | 9edce1e12a7b69e7986dd15fce18d6e46590161a (patch) | |
tree | c5d37d8016a27bab698520ecc51cda4f7ad99d40 /test/sequential | |
parent | 7b674697d8005c29391ebaaf562eb4d92ed9b129 (diff) | |
download | node-new-9edce1e12a7b69e7986dd15fce18d6e46590161a.tar.gz |
benchmark,doc,lib,test: capitalize comments
This updates a lot of comments.
PR-URL: https://github.com/nodejs/node/pull/26223
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Diffstat (limited to 'test/sequential')
-rw-r--r-- | test/sequential/test-cli-syntax-bad.js | 8 | ||||
-rw-r--r-- | test/sequential/test-cli-syntax-file-not-found.js | 4 | ||||
-rw-r--r-- | test/sequential/test-cli-syntax-good.js | 6 | ||||
-rw-r--r-- | test/sequential/test-dgram-bind-shared-ports.js | 2 | ||||
-rw-r--r-- | test/sequential/test-http-econnrefused.js | 2 | ||||
-rw-r--r-- | test/sequential/test-inspector.js | 6 | ||||
-rw-r--r-- | test/sequential/test-module-loading.js | 6 | ||||
-rw-r--r-- | test/sequential/test-net-listen-shared-ports.js | 2 | ||||
-rw-r--r-- | test/sequential/test-stream2-stderr-sync.js | 2 | ||||
-rw-r--r-- | test/sequential/test-timers-blocking-callback.js | 2 |
10 files changed, 20 insertions, 20 deletions
diff --git a/test/sequential/test-cli-syntax-bad.js b/test/sequential/test-cli-syntax-bad.js index 7c4c9c70d9..1512d9df4f 100644 --- a/test/sequential/test-cli-syntax-bad.js +++ b/test/sequential/test-cli-syntax-bad.js @@ -7,7 +7,7 @@ const fixtures = require('../common/fixtures'); const node = process.execPath; -// test both sets of arguments that check syntax +// Test both sets of arguments that check syntax const syntaxArgs = [ ['-c'], ['--check'] @@ -17,7 +17,7 @@ const syntaxArgs = [ // depending on the JavaScript engine. const syntaxErrorRE = /^SyntaxError: \b/m; -// test bad syntax with and without shebang +// Test bad syntax with and without shebang [ 'syntax/bad_syntax.js', 'syntax/bad_syntax', @@ -26,7 +26,7 @@ const syntaxErrorRE = /^SyntaxError: \b/m; ].forEach(function(file) { file = fixtures.path(file); - // loop each possible option, `-c` or `--check` + // Loop each possible option, `-c` or `--check` syntaxArgs.forEach(function(args) { const _args = args.concat(file); const cmd = [node, ..._args].join(' '); @@ -38,7 +38,7 @@ const syntaxErrorRE = /^SyntaxError: \b/m; // no stdout should be produced assert.strictEqual(stdout, ''); - // stderr should have a syntax error message + // Stderr should have a syntax error message assert(syntaxErrorRE.test(stderr), `${syntaxErrorRE} === ${stderr}`); // stderr should include the filename diff --git a/test/sequential/test-cli-syntax-file-not-found.js b/test/sequential/test-cli-syntax-file-not-found.js index 2bfb0e38c7..b3bb9723e2 100644 --- a/test/sequential/test-cli-syntax-file-not-found.js +++ b/test/sequential/test-cli-syntax-file-not-found.js @@ -7,7 +7,7 @@ const fixtures = require('../common/fixtures'); const node = process.execPath; -// test both sets of arguments that check syntax +// Test both sets of arguments that check syntax const syntaxArgs = [ ['-c'], ['--check'] @@ -22,7 +22,7 @@ const notFoundRE = /^Error: Cannot find module/m; ].forEach(function(file) { file = fixtures.path(file); - // loop each possible option, `-c` or `--check` + // Loop each possible option, `-c` or `--check` syntaxArgs.forEach(function(args) { const _args = args.concat(file); const cmd = [node, ..._args].join(' '); diff --git a/test/sequential/test-cli-syntax-good.js b/test/sequential/test-cli-syntax-good.js index 1c2b3cbe55..48581e7733 100644 --- a/test/sequential/test-cli-syntax-good.js +++ b/test/sequential/test-cli-syntax-good.js @@ -7,13 +7,13 @@ const fixtures = require('../common/fixtures'); const node = process.execPath; -// test both sets of arguments that check syntax +// Test both sets of arguments that check syntax const syntaxArgs = [ ['-c'], ['--check'] ]; -// test good syntax with and without shebang +// Test good syntax with and without shebang [ 'syntax/good_syntax.js', 'syntax/good_syntax', @@ -23,7 +23,7 @@ const syntaxArgs = [ ].forEach(function(file) { file = fixtures.path(file); - // loop each possible option, `-c` or `--check` + // Loop each possible option, `-c` or `--check` syntaxArgs.forEach(function(args) { const _args = args.concat(file); diff --git a/test/sequential/test-dgram-bind-shared-ports.js b/test/sequential/test-dgram-bind-shared-ports.js index 9f7e23ba0b..cc12d93787 100644 --- a/test/sequential/test-dgram-bind-shared-ports.js +++ b/test/sequential/test-dgram-bind-shared-ports.js @@ -89,7 +89,7 @@ if (cluster.isMaster) { assert.strictEqual(typeof port3, 'number'); process.send('success'); }); - // an error is expected only in the second worker + // An error is expected only in the second worker const socket3OnError = !isSecondWorker ? common.mustNotCall() : diff --git a/test/sequential/test-http-econnrefused.js b/test/sequential/test-http-econnrefused.js index ff4508963b..5650e74308 100644 --- a/test/sequential/test-http-econnrefused.js +++ b/test/sequential/test-http-econnrefused.js @@ -95,7 +95,7 @@ function afterPing(result) { assert.ok(successRE.test(responses[6])); assert.ok(successRE.test(responses[7])); server.close(); - // we should go to process.on('exit') from here. + // We should go to process.on('exit') from here. break; } } diff --git a/test/sequential/test-inspector.js b/test/sequential/test-inspector.js index 237b65193f..9aaabdc6c9 100644 --- a/test/sequential/test-inspector.js +++ b/test/sequential/test-inspector.js @@ -212,7 +212,7 @@ async function testCommandLineAPI(session) { }); checkException(result); assert.strictEqual(result.result.value, true); - // after require the module appears in require.cache + // After require the module appears in require.cache result = await session.send( { 'method': 'Runtime.evaluate', 'params': { @@ -235,7 +235,7 @@ async function testCommandLineAPI(session) { }); checkException(result); assert.strictEqual(result.result.value, true); - // require again, should get fresh (empty) exports + // Require again, should get fresh (empty) exports result = await session.send( { 'method': 'Runtime.evaluate', 'params': { @@ -255,7 +255,7 @@ async function testCommandLineAPI(session) { }); checkException(result); assert.deepStrictEqual(JSON.parse(result.result.value), {}); - // both modules end up with the same module.parent + // Both modules end up with the same module.parent result = await session.send( { 'method': 'Runtime.evaluate', 'params': { diff --git a/test/sequential/test-module-loading.js b/test/sequential/test-module-loading.js index 90ab1d4f4f..9db97a2769 100644 --- a/test/sequential/test-module-loading.js +++ b/test/sequential/test-module-loading.js @@ -120,7 +120,7 @@ assert.throws( } console.error('test node_modules folders'); -// asserts are in the fixtures files themselves, +// Asserts are in the fixtures files themselves, // since they depend on the folder structure. require('../fixtures/node_modules/foo'); @@ -129,7 +129,7 @@ require('../fixtures/node_modules/foo'); // This one exists and should import the local module const my_path = require('../fixtures/path'); assert.ok(my_path.path_func instanceof Function); - // this one does not exist and should throw + // This one does not exist and should throw assert.throws(function() { require('./utils'); }, /^Error: Cannot find module '\.\/utils'/); } @@ -216,7 +216,7 @@ assert.throws( } { - // make sure that module.require() is the same as + // Make sure that module.require() is the same as // doing require() inside of that module. const parent = require('../fixtures/module-require/parent/'); const child = require('../fixtures/module-require/child/'); diff --git a/test/sequential/test-net-listen-shared-ports.js b/test/sequential/test-net-listen-shared-ports.js index be33821bf6..78d94b4ae1 100644 --- a/test/sequential/test-net-listen-shared-ports.js +++ b/test/sequential/test-net-listen-shared-ports.js @@ -48,7 +48,7 @@ if (cluster.isMaster) { }); server2.on('error', function(err) { - // an error is expected on the second worker + // An error is expected on the second worker process.send(`server2:${err.code}`); }); diff --git a/test/sequential/test-stream2-stderr-sync.js b/test/sequential/test-stream2-stderr-sync.js index 40995394ac..68d35dbdbc 100644 --- a/test/sequential/test-stream2-stderr-sync.js +++ b/test/sequential/test-stream2-stderr-sync.js @@ -89,6 +89,6 @@ if (!process.argv[2]) { parent(); } else { children[process.argv[2]](); - // immediate process.exit to kill any waiting stuff. + // Immediate process.exit to kill any waiting stuff. process.exit(); } diff --git a/test/sequential/test-timers-blocking-callback.js b/test/sequential/test-timers-blocking-callback.js index 3d05a538ea..053bc767b8 100644 --- a/test/sequential/test-timers-blocking-callback.js +++ b/test/sequential/test-timers-blocking-callback.js @@ -64,7 +64,7 @@ function blockingCallback(retry, callback) { if (callback) return callback(); } else { - // block by busy-looping to trigger the issue + // Block by busy-looping to trigger the issue common.busyLoop(TIMEOUT); timeCallbackScheduled = Date.now(); |