Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: fix comments in test files #42536

Merged
merged 1 commit into from
Mar 31, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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