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 <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
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

+8-8
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

+2-2
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)