Skip to content

Commit df63d37

Browse files
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 <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]>
1 parent 56ac9a2 commit df63d37

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/internet/test-dns.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -523,9 +523,9 @@ TEST(function test_resolveTlsa_failure(done) {
523523

524524
TEST(async function test_resolveTxt(done) {
525525
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')));
529529
}
530530

531531
validateResult(await dnsPromises.resolveTxt(addresses.TXT_HOST));

0 commit comments

Comments
 (0)