Commit df63d37
committed
test: fix internet/test-dns
The `nodejs.org` domain has now two TXT records.
Do not verify the exact number of records returned (only their shape),
and check that one of them is the SPF.
PR-URL: #59660
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>1 parent 56ac9a2 commit df63d37
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -523,9 +523,9 @@ TEST(function test_resolveTlsa_failure(done) { | |||
| 523 | 523 | ||
| 524 | 524 | TEST(async function test_resolveTxt(done) { | |
| 525 | 525 | function validateResult(result) { | |
| 526 | - assert.ok(Array.isArray(result[0])); | ||
| 527 | - assert.strictEqual(result.length, 1); | ||
| 528 | - assert(result[0][0].startsWith('v=spf1')); | ||
| 526 | + assert.ok(result.length > 0); | ||
| 527 | + assert.ok(result.every((elem) => Array.isArray(elem) && elem.length === 1)); | ||
| 528 | + assert.ok(result.some((elem) => elem[0].startsWith('v=spf1'))); | ||
| 529 | 529 | } | |
| 530 | 530 | ||
| 531 | 531 | validateResult(await dnsPromises.resolveTxt(addresses.TXT_HOST)); | |
0 commit comments