test: fix typos in various files

nodejs · brandon93s · Oct 7, 2018 · Oct 7, 2018 · Oct 7, 2018 · Oct 7, 2018
commit 0cf3c8e84e8a1d24babae091d3ea03dd553cde00
diff --git a/test/async-hooks/test-embedder.api.async-resource.improper-unwind.js b/test/async-hooks/test-embedder.api.async-resource.improper-unwind.js
@@ -19,7 +19,7 @@ if (process.argv[2] === 'child') {
   // Therefore the 'after' of event2 needs to occur before the
   // 'after' of event 1.
   // The first test of the two below follows that rule,
-  // the second one doesnt.
+  // the second one doesn't.
 
   const event1 = new AsyncResource('event1', async_hooks.executionAsyncId());
   const event2 = new AsyncResource('event2', async_hooks.executionAsyncId());

diff --git a/test/internet/test-dgram-broadcast-multi-process.js b/test/internet/test-dgram-broadcast-multi-process.js
@@ -146,7 +146,7 @@ if (process.argv[2] !== 'child') {
                 }
               });
 
-              console.error('[PARENT] %d received %d matching messges.',
+              console.error('[PARENT] %d received %d matching messages.',
                             worker.pid,
                             count);
 

diff --git a/test/parallel/test-cluster-fork-windowsHide.js b/test/parallel/test-cluster-fork-windowsHide.js
@@ -45,7 +45,7 @@ if (!process.argv[2]) {
 
 } else if (cluster.isMaster) {
   cluster.setupMaster({
-    silient: true,
+    silent: true,
     windowsHide: true
   });
 

diff --git a/test/parallel/test-cluster-setup-master-cumulative.js b/test/parallel/test-cluster-setup-master-cumulative.js
@@ -40,7 +40,7 @@ console.log('ok sets defaults');
 
 cluster.setupMaster({ exec: 'overridden' });
 assert.strictEqual(cluster.settings.exec, 'overridden');
-console.log('ok overrids defaults');
+console.log('ok overrides defaults');
 
 cluster.setupMaster({ args: ['foo', 'bar'] });
 assert.strictEqual(cluster.settings.exec, 'overridden');

diff --git a/test/parallel/test-crypto-stream.js b/test/parallel/test-crypto-stream.js
@@ -40,7 +40,7 @@ if (!common.hasFipsCrypto) {
       });
     }
 
-    _write(data, encodeing, done) {
+    _write(data, encoding, done) {
       this._buffers.push(data);
       return done(null);
     }

diff --git a/test/parallel/test-gc-tls-external-memory.js b/test/parallel/test-gc-tls-external-memory.js
@@ -31,7 +31,7 @@ function connect() {
   const externalMemoryUsage = process.memoryUsage().external;
   assert(externalMemoryUsage >= 0, `${externalMemoryUsage} < 0`);
   if (runs++ === 512) {
-    // Make sure at least half the TLS sockets have been gargbage collected
+    // Make sure at least half the TLS sockets have been garbage collected
     // (so that this test can actually check what it's testing):
     assert(gced >= 256, `${gced} < 256`);
     return;

diff --git a/test/parallel/test-http-eof-on-connect.js b/test/parallel/test-http-eof-on-connect.js
@@ -27,7 +27,7 @@ const http = require('http');
 
 // This is a regression test for https://github.com/joyent/node/issues/44
 // It is separate from test-http-malformed-request.js because it is only
-// reproduceable on the first packet on the first connection to a server.
+// reproducible on the first packet on the first connection to a server.
 
 const server = http.createServer(common.mustNotCall());
 server.listen(0);

diff --git a/test/parallel/test-http-full-response.js b/test/parallel/test-http-full-response.js
@@ -56,10 +56,10 @@ function runAb(opts, callback) {
     const completeRequests = parseInt(m[1]);
 
     m = /HTML transferred:\s*(\d+) bytes/i.exec(stdout);
-    const htmlTransfered = parseInt(m[1]);
+    const htmlTransferred = parseInt(m[1]);
 
     assert.strictEqual(bodyLength, documentLength);
-    assert.strictEqual(completeRequests * documentLength, htmlTransfered);
+    assert.strictEqual(completeRequests * documentLength, htmlTransferred);
 
     if (callback) callback();
   });

diff --git a/test/parallel/test-http-pipeline-flood.js b/test/parallel/test-http-pipeline-flood.js
@@ -9,7 +9,7 @@ const common = require('../common');
 // processed).
 
 // Normally when the writable stream emits a 'drain' event, the server then
-// uncorks the readable stream, although we arent testing that part here.
+// uncorks the readable stream, although we aren't testing that part here.
 
 // The issue being tested exists in Node.js 0.10.20 and is resolved in 0.10.21
 // and newer.

diff --git a/test/parallel/test-http2-compat-serverresponse-end-after-statuses-without-body.js b/test/parallel/test-http2-compat-serverresponse-end-after-statuses-without-body.js
@@ -15,15 +15,15 @@ const {
   HTTP_STATUS_NOT_MODIFIED
 } = h2.constants;
 
-const statusWithouBody = [
+const statusWithoutBody = [
   HTTP_STATUS_NO_CONTENT,
   HTTP_STATUS_RESET_CONTENT,
   HTTP_STATUS_NOT_MODIFIED,
 ];
-const STATUS_CODES_COUNT = statusWithouBody.length;
+const STATUS_CODES_COUNT = statusWithoutBody.length;
 
 const server = h2.createServer(common.mustCall(function(req, res) {
-  res.writeHead(statusWithouBody.pop());
+  res.writeHead(statusWithoutBody.pop());
   res.end();
 }, STATUS_CODES_COUNT));
 

diff --git a/test/parallel/test-https-agent-additional-options.js b/test/parallel/test-https-agent-additional-options.js
@@ -24,7 +24,7 @@ function getBaseOptions(port) {
     path: '/',
     port: port,
     ca: options.ca,
-    rejectUnautorized: true,
+    rejectUnauthorized: true,
     servername: 'agent1',
   };
 }

diff --git a/test/parallel/test-module-readonly.js b/test/parallel/test-module-readonly.js
@@ -38,7 +38,7 @@ try {
   except = err;
 }
 
-// Remove the expliclty granted rights, and reenable inheritance
+// Remove the explicitly granted rights, and re-enable inheritance
 cp.execSync(
   `icacls.exe "${readOnlyModFullPath}" /remove "%USERNAME%" /inheritance:e`);
 

diff --git a/test/parallel/test-querystring-maxKeys-non-finite.js b/test/parallel/test-querystring-maxKeys-non-finite.js
@@ -34,7 +34,7 @@ const originalMaxLength = 1000;
 const params = createManyParams(count);
 
 // thealphanerd
-// 27def4f introduced a change to parse that would cause Inifity
+// 27def4f introduced a change to parse that would cause Infinity
 // to be passed to String.prototype.split as an argument for limit
 // In this instance split will always return an empty array
 // this test confirms that the output of parse is the expected length

diff --git a/test/parallel/test-repl-editor.js b/test/parallel/test-repl-editor.js
@@ -74,7 +74,7 @@ const tests = [
 tests.forEach(run);
 
 // Auto code alignment for .editor mode
-function testCodeAligment({ input, cursor = 0, line = '' }) {
+function testCodeAlignment({ input, cursor = 0, line = '' }) {
   const stream = new ArrayStream();
   const outputStream = new ArrayStream();
 
@@ -121,4 +121,4 @@ const codeAlignmentTests = [
   }
 ];
 
-codeAlignmentTests.forEach(testCodeAligment);
+codeAlignmentTests.forEach(testCodeAlignment);
diff --git a/test/parallel/test-stdio-pipe-access.js b/test/parallel/test-stdio-pipe-access.js
@@ -3,7 +3,7 @@ const common = require('../common');
 if (!common.isMainThread)
   common.skip("Workers don't have process-like stdio");
 
-// Test if Node handles acessing process.stdin if it is a redirected
+// Test if Node handles accessing process.stdin if it is a redirected
 // pipe without deadlocking
 const { spawn, spawnSync } = require('child_process');
 

diff --git a/test/parallel/test-stream-writable-write-writev-finish.js b/test/parallel/test-stream-writable-write-writev-finish.js
@@ -174,7 +174,7 @@ const stream = require('stream');
   };
   w.on('error', common.mustCall());
   w.on('finish', () => {
-    w.write("should't write in finish listener");
+    w.write("shouldn't write in finish listener");
   });
   w.end();
 }
diff --git a/test/parallel/test-timers-unrefd-interval-still-fires.js b/test/parallel/test-timers-unrefd-interval-still-fires.js
@@ -16,7 +16,7 @@ const timer = setInterval(common.mustCall(() => {
   if (--N === 0) {
     clearInterval(timer);
     timer._onTimeout =
-      common.mustNotCall('Unrefd interal fired after being cleared');
+      common.mustNotCall('Unrefd interval fired after being cleared');
     clearTimeout(keepOpen);
   }
 }, N), 1);

diff --git a/test/parallel/test-worker-debug.js b/test/parallel/test-worker-debug.js
@@ -130,11 +130,11 @@ class WorkerSession extends EventEmitter {
 
 async function testBasicWorkerDebug(session, post) {
   /*
-    1. Do 'enble' with waitForDebuggerOnStart = true
+    1. Do 'enable' with waitForDebuggerOnStart = true
     2. Run worker. It should break on start.
     3. Enable Runtime (to get console message) and Debugger. Resume.
     4. Breaks on the 'debugger' statement. Resume.
-    5. Console message recieved, worker runs to a completion.
+    5. Console message received, worker runs to a completion.
     6. contextCreated/contextDestroyed had been properly dispatched
   */
   console.log('Test basic debug scenario');
-Original file line number
+Diff line change
@@ Expand Up / @@ -146,7 +146,7 @@ if (process.argv[2] !== 'child') { @@
                     }
                   });
-                  console.error('[PARENT] %d received %d matching messges.',
+                  console.error('[PARENT] %d received %d matching messages.',
                                 worker.pid,
                                 count);
@@ Expand Down @@