diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2018-12-10 13:27:32 +0100 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2018-12-17 17:14:35 +0100 |
commit | 50dd555910ed0338c35f27ee57e947b9ec95724c (patch) | |
tree | 73a0d59eb5b1087afa4c42bc667bb268f1a23b72 /test/sequential | |
parent | be3ae339360c9833a77ebf5772786d75b7a8dd78 (diff) | |
download | android-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.js | 2 | ||||
-rw-r--r-- | test/sequential/test-fs-watch.js | 2 | ||||
-rw-r--r-- | test/sequential/test-http-max-http-headers.js | 2 | ||||
-rw-r--r-- | test/sequential/test-module-loading.js | 8 | ||||
-rw-r--r-- | test/sequential/test-next-tick-error-spin.js | 2 | ||||
-rw-r--r-- | test/sequential/test-timers-set-interval-excludes-callback-duration.js | 2 |
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) { |