GeezerVison is showing github.com — test: fix dns test case failures after c-ares update to 1.21.0+ · nodejs/node@95d7a75
File treeExpand file treeCollapse file tree Open diff view settings Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: test/parallel/test-dns-resolveany-bad-ancount.js +3-2Lines changed: 3 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -30,13 +30,14 @@ server.bind(0, common.mustCall(async () => { | 30 | 30 | dnsPromises.resolveAny('example.org') | 31 | 31 | .then(common.mustNotCall()) | 32 | 32 | .catch(common.expectsError({ | 33 | | - code: 'EBADRESP', | | 33 | + // May return EBADRESP or ETIMEOUT | | 34 | + code: /^(?:EBADRESP|ETIMEOUT)$/, | 34 | 35 | syscall: 'queryAny', | 35 | 36 | hostname: 'example.org' | 36 | 37 | })); | 37 | 38 |
| 38 | 39 | dns.resolveAny('example.org', common.mustCall((err) => { | 39 | | - assert.strictEqual(err.code, 'EBADRESP'); | | 40 | + assert.notStrictEqual(err.code, 'SUCCESS'); | 40 | 41 | assert.strictEqual(err.syscall, 'queryAny'); | 41 | 42 | assert.strictEqual(err.hostname, 'example.org'); | 42 | 43 | const descriptor = Object.getOwnPropertyDescriptor(err, 'message'); | |
Collapse file Copy file name to clipboardExpand all lines: test/parallel/test-dns-resolveany.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -11,7 +11,7 @@ const answers = [ | 11 | 11 | { type: 'AAAA', address: '::42', ttl: 123 }, | 12 | 12 | { type: 'MX', priority: 42, exchange: 'foobar.com', ttl: 124 }, | 13 | 13 | { type: 'NS', value: 'foobar.org', ttl: 457 }, | 14 | | - { type: 'TXT', entries: [ 'v=spf1 ~all', 'xyz\0foo' ] }, | | 14 | + { type: 'TXT', entries: [ 'v=spf1 ~all xyz\0foo' ] }, | 15 | 15 | { type: 'PTR', value: 'baz.org', ttl: 987 }, | 16 | 16 | { | 17 | 17 | type: 'SOA', | |
|