From a52f4ebc11ccf5ff47fb450739c0cbb60c7a13d7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= Date: Sat, 6 Jan 2018 19:34:27 +0100 Subject: [PATCH] test: fix spelling in test case comments Backport-PR-URL: https://github.com/nodejs/node/pull/19118 PR-URL: https://github.com/nodejs/node/pull/18018 Reviewed-By: Jon Moss Reviewed-By: Anatoli Papirovski Reviewed-By: Weijia Wang Reviewed-By: Luigi Pinca Reviewed-By: James M Snell Reviewed-By: Colin Ihrig Reviewed-By: Yuta Hiroto --- test/abort/test-http-parser-consume.js | 4 ++-- test/async-hooks/hook-checks.js | 2 +- .../async-hooks/test-internal-nexttick-default-trigger.js | 2 +- test/async-hooks/test-pipeconnectwrap.js | 2 +- test/async-hooks/verify-graph.js | 2 +- test/common/index.js | 2 +- test/fixtures/net-fd-passing-receiver.js | 2 +- test/parallel/test-async-hooks-http-agent.js | 2 +- test/parallel/test-async-hooks-promise-enable-disable.js | 4 ++-- test/parallel/test-async-wrap-pop-id-during-load.js | 2 +- test/parallel/test-buffer-fill.js | 4 ++-- test/parallel/test-buffer-includes.js | 2 +- test/parallel/test-buffer-indexof.js | 2 +- test/parallel/test-buffer-read.js | 4 ++-- test/parallel/test-child-process-internal.js | 2 +- test/parallel/test-child-process-send-returns-boolean.js | 8 ++++---- test/parallel/test-crypto-deprecated.js | 2 +- test/parallel/test-crypto-fips.js | 2 +- ...ow-error-then-throw-from-uncaught-exception-handler.js | 2 +- test/parallel/test-event-emitter-remove-listeners.js | 4 ++-- test/parallel/test-fs-access.js | 2 +- test/parallel/test-http-agent-keepalive.js | 4 ++-- test/parallel/test-http-extra-response.js | 1 - test/parallel/test-http2-server-push-disabled.js | 2 +- test/parallel/test-https-agent-secure-protocol.js | 2 +- test/parallel/test-https-socket-options.js | 2 +- test/parallel/test-internal-fs-syncwritestream.js | 4 ++-- test/parallel/test-net-listen-error.js | 2 +- test/parallel/test-net-pipe-connect-errors.js | 2 +- test/parallel/test-net-write-after-close.js | 2 +- test/parallel/test-os.js | 2 +- test/parallel/test-promises-unhandled-rejections.js | 2 +- test/parallel/test-readline-interface.js | 2 +- test/parallel/test-repl.js | 2 +- test/parallel/test-require-symlink.js | 3 ++- test/parallel/test-stream-transform-final-sync.js | 2 +- test/parallel/test-stream-transform-final.js | 2 +- test/parallel/test-stream2-transform.js | 6 +++--- test/parallel/test-stream3-cork-uncork.js | 2 +- test/parallel/test-stringbytes-external.js | 2 +- test/parallel/test-tls-cnnic-whitelist.js | 2 +- test/parallel/test-tls-ecdh-disable.js | 2 +- test/parallel/test-tls-server-verify.js | 4 ++-- test/sequential/test-inspector-contexts.js | 7 ++++--- test/sequential/test-readline-interface.js | 2 +- 45 files changed, 61 insertions(+), 60 deletions(-) diff --git a/test/abort/test-http-parser-consume.js b/test/abort/test-http-parser-consume.js index 9115aba70dbf17..673e04cfa3a573 100644 --- a/test/abort/test-http-parser-consume.js +++ b/test/abort/test-http-parser-consume.js @@ -11,12 +11,12 @@ if (process.argv[2] === 'child') { const rr = get({ port: server.address().port }, common.mustCall(() => { // This bad input (0) should abort the parser and the process rr.parser.consume(0); - // This line should be unreachanble. + // This line should be unreachable. assert.fail('this should be unreachable'); })); })); } else { - // super-proces + // super-process const child = spawn(process.execPath, [__filename, 'child']); child.stdout.on('data', common.mustNotCall()); diff --git a/test/async-hooks/hook-checks.js b/test/async-hooks/hook-checks.js index 60f505a24a95de..2abed61555a158 100644 --- a/test/async-hooks/hook-checks.js +++ b/test/async-hooks/hook-checks.js @@ -11,7 +11,7 @@ require('../common'); * @param {Object} activity including timestamps for each life time event, * i.e. init, before ... * @param {Object} hooks the expected life time event invocations with a count - * indicating how oftn they should have been invoked, + * indicating how often they should have been invoked, * i.e. `{ init: 1, before: 2, after: 2 }` * @param {String} stage the name of the stage in the test at which we are * checking the invocations diff --git a/test/async-hooks/test-internal-nexttick-default-trigger.js b/test/async-hooks/test-internal-nexttick-default-trigger.js index ad352a8c147247..ed541868542206 100644 --- a/test/async-hooks/test-internal-nexttick-default-trigger.js +++ b/test/async-hooks/test-internal-nexttick-default-trigger.js @@ -3,7 +3,7 @@ const common = require('../common'); // This tests ensures that the triggerId of both the internal and external -// nexTick function sets the triggerAsyncId correctly. +// nextTick function sets the triggerAsyncId correctly. const assert = require('assert'); const async_hooks = require('async_hooks'); diff --git a/test/async-hooks/test-pipeconnectwrap.js b/test/async-hooks/test-pipeconnectwrap.js index 81a5abd42a3004..a993f0c8feedd3 100644 --- a/test/async-hooks/test-pipeconnectwrap.js +++ b/test/async-hooks/test-pipeconnectwrap.js @@ -53,7 +53,7 @@ function onlisten() { const awaitOnconnectCalls = new Set(['server', 'client']); function maybeOnconnect(source) { // both server and client must call onconnect. On most OS's waiting for - // the client is sufficient, but on CertOS 5 the sever needs to respond too. + // the client is sufficient, but on CentOS 5 the sever needs to respond too. assert.ok(awaitOnconnectCalls.size > 0); awaitOnconnectCalls.delete(source); if (awaitOnconnectCalls.size > 0) return; diff --git a/test/async-hooks/verify-graph.js b/test/async-hooks/verify-graph.js index 451550d33fc8e1..95c55c15958b2c 100644 --- a/test/async-hooks/verify-graph.js +++ b/test/async-hooks/verify-graph.js @@ -32,7 +32,7 @@ function pruneTickObjects(activities) { foundTickObject = true; // point all triggerAsyncIds that point to the tickObject - // to its triggerAsyncId and findally remove it from the activities + // to its triggerAsyncId and finally remove it from the activities const tickObject = activities[tickObjectIdx]; const newTriggerId = tickObject.triggerAsyncId; const oldTriggerId = tickObject.uid; diff --git a/test/common/index.js b/test/common/index.js index 5bebdcedff60ef..6b9f6d13b0ee38 100644 --- a/test/common/index.js +++ b/test/common/index.js @@ -82,7 +82,7 @@ if (process.env.NODE_TEST_WITH_ASYNC_HOOKS) { const async_wrap = process.binding('async_wrap'); process.on('exit', () => { - // itterate through handles to make sure nothing crashes + // iterate through handles to make sure nothing crashes for (const k in initHandles) util.inspect(initHandles[k]); }); diff --git a/test/fixtures/net-fd-passing-receiver.js b/test/fixtures/net-fd-passing-receiver.js index 8559f116c54956..7d328ac28c9029 100644 --- a/test/fixtures/net-fd-passing-receiver.js +++ b/test/fixtures/net-fd-passing-receiver.js @@ -45,7 +45,7 @@ receiver = net.createServer(function(socket) { }); }); -/* To signal the test runne we're up and listening */ +/* To signal the test runner we're up and listening */ receiver.on('listening', function() { console.log('ready'); }); diff --git a/test/parallel/test-async-hooks-http-agent.js b/test/parallel/test-async-hooks-http-agent.js index ff19d089a02099..e10820c3c202b6 100644 --- a/test/parallel/test-async-hooks-http-agent.js +++ b/test/parallel/test-async-hooks-http-agent.js @@ -8,7 +8,7 @@ const http = require('http'); // Checks that an http.Agent properly asyncReset()s a reused socket handle, and // re-assigns the fresh async id to the reused `net.Socket` instance. -// Make sure a single socket is transpartently reused for 2 requests. +// Make sure a single socket is transparently reused for 2 requests. const agent = new http.Agent({ keepAlive: true, keepAliveMsecs: Infinity, diff --git a/test/parallel/test-async-hooks-promise-enable-disable.js b/test/parallel/test-async-hooks-promise-enable-disable.js index 075b29e4e50a70..b7692c45cd9b2e 100644 --- a/test/parallel/test-async-hooks-promise-enable-disable.js +++ b/test/parallel/test-async-hooks-promise-enable-disable.js @@ -11,13 +11,13 @@ let p_inits = 0; common.crashOnUnhandledRejection(); // Not useful to place common.mustCall() around 'exit' event b/c it won't be -// able to check it anway. +// able to check it anyway. process.on('exit', (code) => { if (code !== 0) return; if (p_er !== null) throw p_er; - // Expecint exactly 2 PROMISE types to reach init. + // Expecting exactly 2 PROMISE types to reach init. assert.strictEqual(p_inits, EXPECTED_INITS); }); diff --git a/test/parallel/test-async-wrap-pop-id-during-load.js b/test/parallel/test-async-wrap-pop-id-during-load.js index 1017fc02a72b05..4f39a4fdf01b34 100644 --- a/test/parallel/test-async-wrap-pop-id-during-load.js +++ b/test/parallel/test-async-wrap-pop-id-during-load.js @@ -8,7 +8,7 @@ if (process.argv[2] === 'async') { throw new Error(); } (async function() { await fn(); })(); - // While the above should error, just in case it dosn't the script shouldn't + // While the above should error, just in case it doesn't the script shouldn't // fork itself indefinitely so return early. return; } diff --git a/test/parallel/test-buffer-fill.js b/test/parallel/test-buffer-fill.js index 2ca79170538a87..f80020752d6431 100644 --- a/test/parallel/test-buffer-fill.js +++ b/test/parallel/test-buffer-fill.js @@ -344,7 +344,7 @@ Buffer.alloc(8, ''); return 0; } else { elseWasLast = true; - // Once buffer.js calls the C++ implemenation of fill, return -1 + // Once buffer.js calls the C++ implementation of fill, return -1 return -1; } } @@ -377,7 +377,7 @@ assert.throws(() => { return 1; } else { elseWasLast = true; - // Once buffer.js calls the C++ implemenation of fill, return -1 + // Once buffer.js calls the C++ implementation of fill, return -1 return -1; } } diff --git a/test/parallel/test-buffer-includes.js b/test/parallel/test-buffer-includes.js index 2f3a5ba6b63b6b..59380f5403b295 100644 --- a/test/parallel/test-buffer-includes.js +++ b/test/parallel/test-buffer-includes.js @@ -137,7 +137,7 @@ assert.strictEqual( ); -// test usc2 encoding +// test ucs2 encoding let twoByteString = Buffer.from('\u039a\u0391\u03a3\u03a3\u0395', 'ucs2'); assert(twoByteString.includes('\u0395', 4, 'ucs2')); diff --git a/test/parallel/test-buffer-indexof.js b/test/parallel/test-buffer-indexof.js index 4da6f9fcddc5d2..52edda39fa0e54 100644 --- a/test/parallel/test-buffer-indexof.js +++ b/test/parallel/test-buffer-indexof.js @@ -553,7 +553,7 @@ assert.strictEqual(511, longBufferString.lastIndexOf(pattern, 1534)); // "yolo swag swag yolo swag yolo yolo swag" ..., goes on for about 5MB. // This is hard to search because it all looks similar, but never repeats. -// countBits returns the number of bits in the binary reprsentation of n. +// countBits returns the number of bits in the binary representation of n. function countBits(n) { let count; for (count = 0; n > 0; count++) { diff --git a/test/parallel/test-buffer-read.js b/test/parallel/test-buffer-read.js index 5eac575ff5ab81..e8f3bad383fd69 100644 --- a/test/parallel/test-buffer-read.js +++ b/test/parallel/test-buffer-read.js @@ -20,11 +20,11 @@ function read(buff, funx, args, expected) { } -// testing basic functionality of readDoubleBE() and readDOubleLE() +// testing basic functionality of readDoubleBE() and readDoubleLE() read(buf, 'readDoubleBE', [1], -3.1827727774563287e+295); read(buf, 'readDoubleLE', [1], -6.966010051009108e+144); -// testing basic functionality of readFLoatBE() and readFloatLE() +// testing basic functionality of readFloatBE() and readFloatLE() read(buf, 'readFloatBE', [1], -1.6691549692541768e+37); read(buf, 'readFloatLE', [1], -7861303808); diff --git a/test/parallel/test-child-process-internal.js b/test/parallel/test-child-process-internal.js index 4b22b4060307ed..c99010aeb8935a 100644 --- a/test/parallel/test-child-process-internal.js +++ b/test/parallel/test-child-process-internal.js @@ -32,7 +32,7 @@ if (process.argv[2] === 'child') { //send non-internal message containing PREFIX at a non prefix position process.send(normal); - //send inernal message + //send internal message process.send(internal); process.exit(0); diff --git a/test/parallel/test-child-process-send-returns-boolean.js b/test/parallel/test-child-process-send-returns-boolean.js index 2fbba1a4544ced..4c986e307e083a 100644 --- a/test/parallel/test-child-process-send-returns-boolean.js +++ b/test/parallel/test-child-process-send-returns-boolean.js @@ -14,7 +14,7 @@ const fixtures = require('../common/fixtures'); const subScript = fixtures.path('child-process-persistent.js'); { - // Test `send` return value on `fork` that opens and IPC by deafult. + // Test `send` return value on `fork` that opens and IPC by default. const n = fork(subScript); // `subprocess.send` should always return `true` for the first send. const rv = n.send({ h: 'w' }, (err) => { if (err) assert.fail(err); }); @@ -31,12 +31,12 @@ const subScript = fixtures.path('child-process-persistent.js'); const server = net.createServer(common.mustNotCall()).listen(0, () => { const handle = server._handle; - // Sending a handle and not giving the tickQueue time to acknoladge should + // Sending a handle and not giving the tickQueue time to acknowledge should // create the internal backlog, but leave it empty. const rv1 = s.send('one', handle, (err) => { if (err) assert.fail(err); }); assert.strictEqual(rv1, true); - // Since the first `send` included a handle (should be unackoladged), - // we can safly queue up only one more message. + // Since the first `send` included a handle (should be unacknowledged), + // we can safely queue up only one more message. const rv2 = s.send('two', (err) => { if (err) assert.fail(err); }); assert.strictEqual(rv2, true); // The backlog should now be indicate to backoff. diff --git a/test/parallel/test-crypto-deprecated.js b/test/parallel/test-crypto-deprecated.js index 84f25316d49b61..acdd71301fbed0 100644 --- a/test/parallel/test-crypto-deprecated.js +++ b/test/parallel/test-crypto-deprecated.js @@ -14,7 +14,7 @@ common.expectWarning('DeprecationWarning', [ // Accessing the deprecated function is enough to trigger the warning event. // It does not need to be called. So the assert serves the purpose of both -// triggering the warning event and confirming that the deprected function is +// triggering the warning event and confirming that the deprecated function is // mapped to the correct non-deprecated function. assert.strictEqual(crypto.Credentials, tls.SecureContext); assert.strictEqual(crypto.createCredentials, tls.createSecureContext); diff --git a/test/parallel/test-crypto-fips.js b/test/parallel/test-crypto-fips.js index 755c6e20c26b2d..ed2654ec5b639c 100644 --- a/test/parallel/test-crypto-fips.js +++ b/test/parallel/test-crypto-fips.js @@ -91,7 +91,7 @@ testHelper( // to try to call the fips setter, to try to detect this situation, as // that would throw an error: // ("Error: Cannot set FIPS mode in a non-FIPS build."). -// Due to this uncertanty the following tests are skipped when configured +// Due to this uncertainty the following tests are skipped when configured // with --shared-openssl. if (!sharedOpenSSL()) { // OpenSSL config file should be able to turn on FIPS mode diff --git a/test/parallel/test-domain-throw-error-then-throw-from-uncaught-exception-handler.js b/test/parallel/test-domain-throw-error-then-throw-from-uncaught-exception-handler.js index 089300bc481c10..a2afebd838f410 100644 --- a/test/parallel/test-domain-throw-error-then-throw-from-uncaught-exception-handler.js +++ b/test/parallel/test-domain-throw-error-then-throw-from-uncaught-exception-handler.js @@ -25,7 +25,7 @@ if (process.argv[2] === 'child') { // is not properly flushed in V8's Isolate::Throw right before the // process aborts due to an uncaught exception, and thus the error // message representing the error that was thrown cannot be read by the - // parent process. So instead of parsing the child process' stdandard + // parent process. So instead of parsing the child process' standard // error, the parent process will check that in the case // --abort-on-uncaught-exception was passed, the process did not exit // with exit code RAN_UNCAUGHT_EXCEPTION_HANDLER_EXIT_CODE. diff --git a/test/parallel/test-event-emitter-remove-listeners.js b/test/parallel/test-event-emitter-remove-listeners.js index d1c01e0bc3cfae..be7f68bcaecf0d 100644 --- a/test/parallel/test-event-emitter-remove-listeners.js +++ b/test/parallel/test-event-emitter-remove-listeners.js @@ -119,10 +119,10 @@ function listener2() {} // listener4 will still be called although it is removed by listener 3. ee.emit('hello'); - // This is so because the interal listener array at time of emit + // This is so because the internal listener array at time of emit // was [listener3,listener4] - // Interal listener array [listener3] + // Internal listener array [listener3] ee.emit('hello'); } diff --git a/test/parallel/test-fs-access.js b/test/parallel/test-fs-access.js index f378824cbcd950..166e3cac4bac66 100644 --- a/test/parallel/test-fs-access.js +++ b/test/parallel/test-fs-access.js @@ -28,7 +28,7 @@ createFileWithPerms(readWriteFile, 0o666); * The change of user id is done after creating the fixtures files for the same * reason: the test may be run as the superuser within a directory in which * only the superuser can create files, and thus it may need superuser - * priviledges to create them. + * privileges to create them. * * There's not really any point in resetting the process' user id to 0 after * changing it to 'nobody', since in the case that the test runs without diff --git a/test/parallel/test-http-agent-keepalive.js b/test/parallel/test-http-agent-keepalive.js index 2c8b6c1cc98d2d..8ac8d79df192b0 100644 --- a/test/parallel/test-http-agent-keepalive.js +++ b/test/parallel/test-http-agent-keepalive.js @@ -89,7 +89,7 @@ function remoteClose() { process.nextTick(common.mustCall(() => { assert.strictEqual(agent.sockets[name], undefined); assert.strictEqual(agent.freeSockets[name].length, 1); - // waitting remote server close the socket + // waiting remote server close the socket setTimeout(common.mustCall(() => { assert.strictEqual(agent.sockets[name], undefined); assert.strictEqual(agent.freeSockets[name], undefined, @@ -102,7 +102,7 @@ function remoteClose() { } function remoteError() { - // remove server will destroy ths socket + // remote server will destroy the socket const req = get('/error', common.mustNotCall()); req.on('error', common.mustCall((err) => { assert(err); diff --git a/test/parallel/test-http-extra-response.js b/test/parallel/test-http-extra-response.js index 7c9514232c7ba8..6d1a770487402d 100644 --- a/test/parallel/test-http-extra-response.js +++ b/test/parallel/test-http-extra-response.js @@ -51,7 +51,6 @@ const server = net.createServer(function(socket) { if (postBody.includes('\r\n')) { socket.write(fullResponse); - // omg, I wrote the response twice, what a terrible HTTP server I am. socket.end(fullResponse); } }); diff --git a/test/parallel/test-http2-server-push-disabled.js b/test/parallel/test-http2-server-push-disabled.js index c0148fe63b672e..cd1516f43ba252 100644 --- a/test/parallel/test-http2-server-push-disabled.js +++ b/test/parallel/test-http2-server-push-disabled.js @@ -42,7 +42,7 @@ server.listen(0, common.mustCall(() => { options); const req = client.request({ ':path': '/' }); - // Because push stream sre disabled, this must not be called. + // Because push streams are disabled, this must not be called. client.on('stream', common.mustNotCall()); req.resume(); diff --git a/test/parallel/test-https-agent-secure-protocol.js b/test/parallel/test-https-agent-secure-protocol.js index 4963b55febe345..82554952e8446b 100644 --- a/test/parallel/test-https-agent-secure-protocol.js +++ b/test/parallel/test-https-agent-secure-protocol.js @@ -42,7 +42,7 @@ server.listen(0, common.mustCall(function() { }, common.mustCall(function(res) { res.resume(); globalAgent.once('free', common.mustCall(function() { - // Verify that two keep-alived connections are created + // Verify that two keep-alive connections are created // due to the different secureProtocol settings: const keys = Object.keys(globalAgent.freeSockets); assert.strictEqual(keys.length, 2); diff --git a/test/parallel/test-https-socket-options.js b/test/parallel/test-https-socket-options.js index 2ba1b883684205..b41054d5aa0824 100644 --- a/test/parallel/test-https-socket-options.js +++ b/test/parallel/test-https-socket-options.js @@ -61,7 +61,7 @@ server_http.listen(0, function() { }); // Then try https server (requires functions to be -// mirroed in tls.js's CryptoStream) +// mirrored in tls.js's CryptoStream) const server_https = https.createServer(options, function(req, res) { console.log('got HTTPS request'); diff --git a/test/parallel/test-internal-fs-syncwritestream.js b/test/parallel/test-internal-fs-syncwritestream.js index 166692f4e6236e..9e0024df3481b2 100644 --- a/test/parallel/test-internal-fs-syncwritestream.js +++ b/test/parallel/test-internal-fs-syncwritestream.js @@ -31,7 +31,7 @@ const filename = path.join(common.tmpDir, 'sync-write-stream.txt'); assert.strictEqual(stream.listenerCount('end'), 1); } -// Verfiy that the file will be written synchronously. +// Verify that the file will be written synchronously. { const fd = fs.openSync(filename, 'w'); const stream = new SyncWriteStream(fd); @@ -54,7 +54,7 @@ const filename = path.join(common.tmpDir, 'sync-write-stream.txt'); assert.strictEqual(stream.destroySoon(), true); } -// Verfit that the 'end' event listener will also destroy the stream. +// Verify that the 'end' event listener will also destroy the stream. { const fd = fs.openSync(filename, 'w'); const stream = new SyncWriteStream(fd); diff --git a/test/parallel/test-net-listen-error.js b/test/parallel/test-net-listen-error.js index 26a74a72c38b85..05ca799d3e7351 100644 --- a/test/parallel/test-net-listen-error.js +++ b/test/parallel/test-net-listen-error.js @@ -25,5 +25,5 @@ const net = require('net'); const server = net.createServer(function(socket) { }); -server.listen(1, '1.1.1.1', common.mustNotCall()); // EACCESS or EADDRNOTAVAIL +server.listen(1, '1.1.1.1', common.mustNotCall()); // EACCES or EADDRNOTAVAIL server.on('error', common.mustCall()); diff --git a/test/parallel/test-net-pipe-connect-errors.js b/test/parallel/test-net-pipe-connect-errors.js index 91c4f7efd07966..119bca19fcc762 100644 --- a/test/parallel/test-net-pipe-connect-errors.js +++ b/test/parallel/test-net-pipe-connect-errors.js @@ -37,7 +37,7 @@ if (common.isWindows) { emptyTxt = fixtures.path('empty.txt'); } else { common.refreshTmpDir(); - // Keep the file name very short so tht we don't exceed the 108 char limit + // Keep the file name very short so that we don't exceed the 108 char limit // on CI for a POSIX socket. Even though this isn't actually a socket file, // the error will be different from the one we are expecting if we exceed the // limit. diff --git a/test/parallel/test-net-write-after-close.js b/test/parallel/test-net-write-after-close.js index 05669e476c7670..f17273f9417a2a 100644 --- a/test/parallel/test-net-write-after-close.js +++ b/test/parallel/test-net-write-after-close.js @@ -39,7 +39,7 @@ const server = net.createServer(common.mustCall(function(socket) { server.listen(0, function() { const client = net.connect(this.address().port, function() { - // cliend.end() will close both the readable and writable side + // client.end() will close both the readable and writable side // of the duplex because allowHalfOpen defaults to false. // Then 'end' will be emitted when it receives a FIN packet from // the other side. diff --git a/test/parallel/test-os.js b/test/parallel/test-os.js index f8e383e497a154..47d4209c36a20b 100644 --- a/test/parallel/test-os.js +++ b/test/parallel/test-os.js @@ -113,7 +113,7 @@ is.string(arch); assert.ok(arch.length > 0); if (!common.isSunOS) { - // not implemeneted yet + // not implemented yet assert.ok(os.loadavg().length > 0); assert.ok(os.freemem() > 0); assert.ok(os.totalmem() > 0); diff --git a/test/parallel/test-promises-unhandled-rejections.js b/test/parallel/test-promises-unhandled-rejections.js index 0d49ee796b7f70..099d762b671fed 100644 --- a/test/parallel/test-promises-unhandled-rejections.js +++ b/test/parallel/test-promises-unhandled-rejections.js @@ -293,7 +293,7 @@ asyncTest('While inside setImmediate, catching a rejected promise derived ' + }); }); -// State adapation tests +// State adaptation tests asyncTest('catching a promise which is asynchronously rejected (via ' + 'resolution to an asynchronously-rejected promise) prevents' + ' unhandledRejection', function(done) { diff --git a/test/parallel/test-readline-interface.js b/test/parallel/test-readline-interface.js index 03ea05dd2a44a5..f3fdd5044f4862 100644 --- a/test/parallel/test-readline-interface.js +++ b/test/parallel/test-readline-interface.js @@ -812,7 +812,7 @@ function isWarned(emitter) { assert.strictEqual(isWarned(process.stdout._events), false); } - // can create a new readline Interface with a null output arugument + // can create a new readline Interface with a null output argument { const fi = new FakeInput(); const rli = new readline.Interface( diff --git a/test/parallel/test-repl.js b/test/parallel/test-repl.js index 6d4f0d8c7a5862..11fb8b0a14450c 100644 --- a/test/parallel/test-repl.js +++ b/test/parallel/test-repl.js @@ -304,7 +304,7 @@ function error_test() { { client: client_unix, send: '/(.)(.)(.)(.)(.)(.)(.)(.)(.)/.test(\'123456789\')\n', expect: `true\n${prompt_unix}` }, - // the following test's result depends on the RegEx's match from the above + // the following test's result depends on the RegExp's match from the above { client: client_unix, send: 'RegExp.$1\nRegExp.$2\nRegExp.$3\nRegExp.$4\nRegExp.$5\n' + 'RegExp.$6\nRegExp.$7\nRegExp.$8\nRegExp.$9\n', diff --git a/test/parallel/test-require-symlink.js b/test/parallel/test-require-symlink.js index 7dde2a1a974087..60962156f2f286 100644 --- a/test/parallel/test-require-symlink.js +++ b/test/parallel/test-require-symlink.js @@ -38,7 +38,8 @@ function copyDir(source, target) { copyDir(fixtureSource, tmpDirTarget); // Move to tmp dir and do everything with relative paths there so that the test -// doesn't incorrectly fail due to a symlink somewhere else in the absolte path. +// doesn't incorrectly fail due to a symlink somewhere else in the absolute +// path. process.chdir(common.tmpDir); const linkDir = path.join(dirName, diff --git a/test/parallel/test-stream-transform-final-sync.js b/test/parallel/test-stream-transform-final-sync.js index de3f0904885bb9..7dbd06d60c3625 100644 --- a/test/parallel/test-stream-transform-final-sync.js +++ b/test/parallel/test-stream-transform-final-sync.js @@ -7,7 +7,7 @@ let state = 0; /* What you do -var stream = new tream.Transform({ +var stream = new stream.Transform({ transform: function transformCallback(chunk, _, next) { // part 1 this.push(chunk); diff --git a/test/parallel/test-stream-transform-final.js b/test/parallel/test-stream-transform-final.js index 56566152e69165..22128b4d9a9f7b 100644 --- a/test/parallel/test-stream-transform-final.js +++ b/test/parallel/test-stream-transform-final.js @@ -7,7 +7,7 @@ let state = 0; /* What you do -var stream = new tream.Transform({ +var stream = new stream.Transform({ transform: function transformCallback(chunk, _, next) { // part 1 this.push(chunk); diff --git a/test/parallel/test-stream2-transform.js b/test/parallel/test-stream2-transform.js index 819b088e2757f2..769684e3f4f907 100644 --- a/test/parallel/test-stream2-transform.js +++ b/test/parallel/test-stream2-transform.js @@ -175,7 +175,7 @@ const Transform = require('_stream_transform'); } { - // Verify assymetric transform (expand) + // Verify asymmetric transform (expand) const pt = new Transform(); // emit each chunk 2 times. @@ -207,7 +207,7 @@ const Transform = require('_stream_transform'); } { - // Verify assymetric trasform (compress) + // Verify asymmetric transform (compress) const pt = new Transform(); // each output is the first char of 3 consecutive chunks, @@ -262,7 +262,7 @@ const Transform = require('_stream_transform'); // this tests for a stall when data is written to a full stream // that has empty transforms. { - // Verify compex transform behavior + // Verify complex transform behavior let count = 0; let saved = null; const pt = new Transform({ highWaterMark: 3 }); diff --git a/test/parallel/test-stream3-cork-uncork.js b/test/parallel/test-stream3-cork-uncork.js index 2e8e86be1ef058..f8b411c84eded6 100644 --- a/test/parallel/test-stream3-cork-uncork.js +++ b/test/parallel/test-stream3-cork-uncork.js @@ -65,7 +65,7 @@ writeChunks(inputChunks, () => { // trigger writing out the buffer w.uncork(); - // buffered bytes shoud be seen in current tick + // buffered bytes should be seen in current tick assert.strictEqual(seenChunks.length, 4); // did the chunks match diff --git a/test/parallel/test-stringbytes-external.js b/test/parallel/test-stringbytes-external.js index 35d773da26f4d4..2a04159ce9db3c 100644 --- a/test/parallel/test-stringbytes-external.js +++ b/test/parallel/test-stringbytes-external.js @@ -82,7 +82,7 @@ assert.strictEqual(c_bin.toString('latin1'), ucs2_control); assert.strictEqual(c_ucs.toString('latin1'), ucs2_control); -// now let's test BASE64 and HEX ecoding/decoding +// now let's test BASE64 and HEX encoding/decoding const RADIOS = 2; const PRE_HALF_APEX = Math.ceil(EXTERN_APEX / 2) - RADIOS; const PRE_3OF4_APEX = Math.ceil((EXTERN_APEX / 4) * 3) - RADIOS; diff --git a/test/parallel/test-tls-cnnic-whitelist.js b/test/parallel/test-tls-cnnic-whitelist.js index c269835428e52d..d118567089d44d 100644 --- a/test/parallel/test-tls-cnnic-whitelist.js +++ b/test/parallel/test-tls-cnnic-whitelist.js @@ -31,7 +31,7 @@ const testCases = [ errorCode: 'CERT_REVOKED' }, // Test 1: for the fix of node#2061 - // agent6-cert.pem is signed by intermidate cert of ca3. + // agent6-cert.pem is signed by intermediate cert of ca3. // The server has a cert chain of agent6->ca3->ca1(root) but // tls.connect should be failed with an error of // UNABLE_TO_GET_ISSUER_CERT_LOCALLY since the root CA of ca1 is not diff --git a/test/parallel/test-tls-ecdh-disable.js b/test/parallel/test-tls-ecdh-disable.js index 72b51771c87280..835f6a7fa8cb4a 100644 --- a/test/parallel/test-tls-ecdh-disable.js +++ b/test/parallel/test-tls-ecdh-disable.js @@ -19,7 +19,7 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -// Test that the usage of eliptic curves are not permitted if disabled during +// Test that the usage of elliptic curves are not permitted if disabled during // server initialization. 'use strict'; diff --git a/test/parallel/test-tls-server-verify.js b/test/parallel/test-tls-server-verify.js index 92c4752a186317..eeea8b030da719 100644 --- a/test/parallel/test-tls-server-verify.js +++ b/test/parallel/test-tls-server-verify.js @@ -286,8 +286,8 @@ function runTest(port, testIndex) { let renegotiated = false; const server = tls.Server(serverOptions, function handleConnection(c) { c.on('error', function(e) { - // child.kill() leads ECONNRESET errro in the TLS connection of - // openssl s_client via spawn(). A Test result is already + // child.kill() leads ECONNRESET error in the TLS connection of + // openssl s_client via spawn(). A test result is already // checked by the data of client.stdout before child.kill() so // these tls errors can be ignored. }); diff --git a/test/sequential/test-inspector-contexts.js b/test/sequential/test-inspector-contexts.js index c7db962f2af006..79516b6447db97 100644 --- a/test/sequential/test-inspector-contexts.js +++ b/test/sequential/test-inspector-contexts.js @@ -27,9 +27,10 @@ async function testContextCreatedAndDestroyed() { const { name } = contextCreated.params.context; if (common.isSunOS || common.isWindows) { // uv_get_process_title() is unimplemented on Solaris-likes, it returns - // an empy string. On the Windows CI buildbots it returns "Administrator: - // Windows PowerShell[42]" because of a GetConsoleTitle() quirk. Not much - // we can do about either, just verify that it contains the PID. + // an empty string. On the Windows CI buildbots it returns + // "Administrator: Windows PowerShell[42]" because of a GetConsoleTitle() + // quirk. Not much we can do about either, just verify that it contains + // the PID. strictEqual(name.includes(`[${process.pid}]`), true); } else { strictEqual(`${process.argv0}[${process.pid}]`, name); diff --git a/test/sequential/test-readline-interface.js b/test/sequential/test-readline-interface.js index 5c1a0e08a13ae7..488dd8338da761 100644 --- a/test/sequential/test-readline-interface.js +++ b/test/sequential/test-readline-interface.js @@ -24,7 +24,7 @@ const common = require('../common'); // These test cases are in `sequential` rather than the analogous test file in -// `parallel` because they become unrelaible under load. The unreliability under +// `parallel` because they become unreliable under load. The unreliability under // load was determined empirically when the test cases were in `parallel` by // running: // tools/test.py -j 96 --repeat 192 test/parallel/test-readline-interface.js