summaryrefslogtreecommitdiff
path: root/test/sequential
diff options
context:
space:
mode:
authorRuben Bridgewater <ruben@bridgewater.de>2019-01-21 01:22:27 +0100
committerRuben Bridgewater <ruben@bridgewater.de>2019-02-28 18:31:10 +0100
commit9edce1e12a7b69e7986dd15fce18d6e46590161a (patch)
treec5d37d8016a27bab698520ecc51cda4f7ad99d40 /test/sequential
parent7b674697d8005c29391ebaaf562eb4d92ed9b129 (diff)
downloadnode-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.js8
-rw-r--r--test/sequential/test-cli-syntax-file-not-found.js4
-rw-r--r--test/sequential/test-cli-syntax-good.js6
-rw-r--r--test/sequential/test-dgram-bind-shared-ports.js2
-rw-r--r--test/sequential/test-http-econnrefused.js2
-rw-r--r--test/sequential/test-inspector.js6
-rw-r--r--test/sequential/test-module-loading.js6
-rw-r--r--test/sequential/test-net-listen-shared-ports.js2
-rw-r--r--test/sequential/test-stream2-stderr-sync.js2
-rw-r--r--test/sequential/test-timers-blocking-callback.js2
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();