Skip to content

Commit

Permalink
test: fix comments in test files
Browse files Browse the repository at this point in the history
This commit fixes typos found in test files.

PR-URL: #42536
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
daeyeon authored Mar 31, 2022
1 parent f2a22ef commit 1600869
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion test/common/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -998,7 +998,7 @@ an `emitReceived()` API for actin as if data has been received on it.
`makeUDPPair` returns an object `{ clientSide, serverSide }` where each side
is an `FakeUDPWrap` connected to the other side.

There is no difference between cient or server side beyond their names.
There is no difference between client or server side beyond their names.

## WPT Module

Expand Down
2 changes: 1 addition & 1 deletion test/common/internet.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ const addresses = {
NAPTR_HOST: 'sip2sip.info',
// A host with SOA records registered
SOA_HOST: 'nodejs.org',
// A host with CAA record registred
// A host with CAA record registered
CAA_HOST: 'google.com',
// A host with CNAME records registered
CNAME_HOST: 'blog.nodejs.org',
Expand Down
2 changes: 1 addition & 1 deletion test/es-module/test-esm-loader-resolve-type.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,6 @@ await import(`${moduleName}`).finally(() => {
const { importedESM: importedESMAfter,
importedCJS: importedCJSAfter } = global.getModuleTypeStats();

// Dynamic import above should incriment ESM counter but not CJS counter
// Dynamic import above should increment ESM counter but not CJS counter
assert.strictEqual(importedESMBefore + 1, importedESMAfter);
assert.strictEqual(importedCJSBefore, importedCJSAfter);
2 changes: 1 addition & 1 deletion test/parallel/test-crypto-keygen.js
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ const sec1EncExp = (cipher) => getRegExpForPEM('EC PRIVATE KEY', cipher);

{
// Test sync key generation with key objects with a non-standard
// publicExpononent
// publicExponent
const { publicKey, privateKey } = generateKeyPairSync('rsa', {
publicExponent: 3,
modulusLength: 512
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-domain-thrown-error-handler-stack.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
const common = require('../common');
const domain = require('domain');

// Make sure that when an erorr is thrown from a nested domain, its error
// Make sure that when an error is thrown from a nested domain, its error
// handler runs outside of that domain, but within the context of any parent
// domain.

Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-http-keep-alive-max-requests.js
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ server.listen(0, common.mustCall((res) => {

anotherSocket.on('ready', common.mustCall(() => {
// Do another 2 requests with another socket
// enusre that this will not affect the first socket
// ensure that this will not affect the first socket
initialRequests(anotherSocket, 2, common.mustCall(() => {
let buffer = '';

Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-http2-server-stream-session-destroy.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ server.on('stream', common.mustCall((stream) => {
name: 'Error'
}
);
// When session is detroyed all streams are destroyed and no further
// When session is destroyed all streams are destroyed and no further
// error should be emitted.
stream.on('error', common.mustNotCall());
assert.strictEqual(stream.write('data', common.expectsError({
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-stream-pipeline-listeners.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ process.on('uncaughtException', common.mustCall((err) => {
assert.strictEqual(err.message, 'no way');
}, 2));

// Ensure that listeners is removed if last stream is readble
// Ensure that listeners is removed if last stream is readable
// And other stream's listeners unchanged
const a = new PassThrough();
a.end('foobar');
Expand Down

0 comments on commit 1600869

Please sign in to comment.