summaryrefslogtreecommitdiff
path: root/test/sequential
diff options
context:
space:
mode:
authorRuben Bridgewater <ruben@bridgewater.de>2018-12-10 13:27:32 +0100
committerRuben Bridgewater <ruben@bridgewater.de>2018-12-17 17:14:35 +0100
commit50dd555910ed0338c35f27ee57e947b9ec95724c (patch)
tree73a0d59eb5b1087afa4c42bc667bb268f1a23b72 /test/sequential
parentbe3ae339360c9833a77ebf5772786d75b7a8dd78 (diff)
downloadandroid-node-v8-50dd555910ed0338c35f27ee57e947b9ec95724c.tar.gz
android-node-v8-50dd555910ed0338c35f27ee57e947b9ec95724c.tar.bz2
android-node-v8-50dd555910ed0338c35f27ee57e947b9ec95724c.zip
doc,lib,test: capitalize comment sentences
This activates the eslint capitalize comment rule for comments above 50 characters. PR-URL: https://github.com/nodejs/node/pull/24996 Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'test/sequential')
-rw-r--r--test/sequential/test-cli-syntax-file-not-found.js2
-rw-r--r--test/sequential/test-fs-watch.js2
-rw-r--r--test/sequential/test-http-max-http-headers.js2
-rw-r--r--test/sequential/test-module-loading.js8
-rw-r--r--test/sequential/test-next-tick-error-spin.js2
-rw-r--r--test/sequential/test-timers-set-interval-excludes-callback-duration.js2
6 files changed, 9 insertions, 9 deletions
diff --git a/test/sequential/test-cli-syntax-file-not-found.js b/test/sequential/test-cli-syntax-file-not-found.js
index b90033a396..2bfb0e38c7 100644
--- a/test/sequential/test-cli-syntax-file-not-found.js
+++ b/test/sequential/test-cli-syntax-file-not-found.js
@@ -30,7 +30,7 @@ const notFoundRE = /^Error: Cannot find module/m;
// no stdout should be produced
assert.strictEqual(stdout, '');
- // stderr should have a module not found error message
+ // `stderr` should have a module not found error message.
assert(notFoundRE.test(stderr), `${notFoundRE} === ${stderr}`);
assert.strictEqual(err.code, 1,
diff --git a/test/sequential/test-fs-watch.js b/test/sequential/test-fs-watch.js
index 9755cc8bb1..a67de8c6d1 100644
--- a/test/sequential/test-fs-watch.js
+++ b/test/sequential/test-fs-watch.js
@@ -111,7 +111,7 @@ tmpdir.refresh();
fs.watch(__filename, { persistent: false }, common.mustNotCall());
}
-// whitebox test to ensure that wrapped FSEvent is safe
+// Whitebox test to ensure that wrapped FSEvent is safe
// https://github.com/joyent/node/issues/6690
{
let oldhandle;
diff --git a/test/sequential/test-http-max-http-headers.js b/test/sequential/test-http-max-http-headers.js
index 64358e8140..f1be923e22 100644
--- a/test/sequential/test-http-max-http-headers.js
+++ b/test/sequential/test-http-max-http-headers.js
@@ -49,7 +49,7 @@ const timeout = common.platformTimeout(10);
function writeHeaders(socket, headers) {
const array = [];
- // this is off from 1024 so that \r\n does not get split
+ // This is off from 1024 so that \r\n does not get split
const chunkSize = 1000;
let last = 0;
diff --git a/test/sequential/test-module-loading.js b/test/sequential/test-module-loading.js
index 0e8aafe51b..6cfa4da04e 100644
--- a/test/sequential/test-module-loading.js
+++ b/test/sequential/test-module-loading.js
@@ -35,11 +35,11 @@ assert.strictEqual(require.main.id, '.');
assert.strictEqual(require.main, module);
assert.strictEqual(process.mainModule, module);
-// assert that it's *not* the main module in the required module.
+// Assert that it's *not* the main module in the required module.
require('../fixtures/not-main-module.js');
{
- // require a file with a request that includes the extension
+ // Require a file with a request that includes the extension
const a_js = require('../fixtures/a.js');
assert.strictEqual(a_js.number, 42);
}
@@ -126,7 +126,7 @@ require('../fixtures/node_modules/foo');
{
console.error('test name clashes');
- // this one exists and should import the local module
+ // 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
@@ -235,7 +235,7 @@ try {
{
- // now verify that module.children contains all the different
+ // Now verify that module.children contains all the different
// modules that we've required, and that all of them contain
// the appropriate children, and so on.
diff --git a/test/sequential/test-next-tick-error-spin.js b/test/sequential/test-next-tick-error-spin.js
index 70d924cfa3..071a017877 100644
--- a/test/sequential/test-next-tick-error-spin.js
+++ b/test/sequential/test-next-tick-error-spin.js
@@ -40,7 +40,7 @@ if (process.argv[2] !== 'child') {
const domain = require('domain');
const d = domain.create();
- // in the error handler, we trigger several MakeCallback events
+ // In the error handler, we trigger several MakeCallback events
d.on('error', function() {
console.log('a');
console.log('b');
diff --git a/test/sequential/test-timers-set-interval-excludes-callback-duration.js b/test/sequential/test-timers-set-interval-excludes-callback-duration.js
index be9f491b92..90eb16b0e2 100644
--- a/test/sequential/test-timers-set-interval-excludes-callback-duration.js
+++ b/test/sequential/test-timers-set-interval-excludes-callback-duration.js
@@ -8,7 +8,7 @@ const t = setInterval(() => {
cntr++;
if (cntr === 1) {
common.busyLoop(100);
- // ensure that the event loop passes before the second interval
+ // Ensure that the event loop passes before the second interval
setImmediate(() => assert.strictEqual(cntr, 1));
first = Date.now();
} else if (cntr === 2) {