Skip to content

Commit 467385a

Browse files
committed
test: use invalid host according to RFC2606
PR-URL: #14863 Refs: #14781 Refs: https://tools.ietf.org/html/rfc2606#section-2 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
1 parent fd8cf79 commit 467385a

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

test/internet/test-dns.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -388,12 +388,12 @@ TEST(function test_resolveTxt_failure(done) {
388388

389389

390390
TEST(function test_lookup_failure(done) {
391-
const req = dns.lookup('does.not.exist', 4, function(err, ip, family) {
391+
const req = dns.lookup('this.hostname.is.invalid', 4, (err, ip, family) => {
392392
assert.ok(err instanceof Error);
393393
assert.strictEqual(err.errno, dns.NOTFOUND);
394394
assert.strictEqual(err.errno, 'ENOTFOUND');
395395
assert.ok(!/ENOENT/.test(err.message));
396-
assert.ok(/does\.not\.exist/.test(err.message));
396+
assert.ok(err.message.includes('this.hostname.is.invalid'));
397397

398398
done();
399399
});
@@ -511,11 +511,11 @@ TEST(function test_reverse_failure(done) {
511511

512512

513513
TEST(function test_lookup_failure(done) {
514-
const req = dns.lookup('nosuchhostimsure', function(err) {
514+
const req = dns.lookup('this.hostname.is.invalid', (err) => {
515515
assert(err instanceof Error);
516516
assert.strictEqual(err.code, 'ENOTFOUND'); // Silly error code...
517-
assert.strictEqual(err.hostname, 'nosuchhostimsure');
518-
assert.ok(/nosuchhostimsure/.test(err.message));
517+
assert.strictEqual(err.hostname, 'this.hostname.is.invalid');
518+
assert.ok(err.message.includes('this.hostname.is.invalid'));
519519

520520
done();
521521
});
@@ -525,7 +525,7 @@ TEST(function test_lookup_failure(done) {
525525

526526

527527
TEST(function test_resolve_failure(done) {
528-
const req = dns.resolve4('nosuchhostimsure', function(err) {
528+
const req = dns.resolve4('this.hostname.is.invalid', (err) => {
529529
assert(err instanceof Error);
530530

531531
switch (err.code) {
@@ -537,8 +537,8 @@ TEST(function test_resolve_failure(done) {
537537
break;
538538
}
539539

540-
assert.strictEqual(err.hostname, 'nosuchhostimsure');
541-
assert.ok(/nosuchhostimsure/.test(err.message));
540+
assert.strictEqual(err.hostname, 'this.hostname.is.invalid');
541+
assert.ok(err.message.includes('this.hostname.is.invalid'));
542542

543543
done();
544544
});

test/parallel/test-http-client-req-error-dont-double-fire.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ const assert = require('assert');
33
const http = require('http');
44
const common = require('../common');
55

6-
// not exists host
7-
const host = '*'.repeat(256);
6+
// Invalid hostname as per https://tools.ietf.org/html/rfc2606#section-2
7+
const host = 'this.hostname.is.invalid';
88
const req = http.get({ host });
99
const err = new Error('mock unexpected code error');
1010
req.on('error', common.mustCall(() => {

0 commit comments

Comments
 (0)