Deno/deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-22 04:51:22 +00:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

chore: disable PTR_HOST dns test for Node compat (#21735)

Browse Source 
These tests started failing on CI on Dec 28th, 2023 returning
ENOTFOUND. It's unclear what's going on, since `dig -x
8.8.8.8.in-addr.arpa`.

It needs a deeper investigation, but I want to unblock main branch
in the meantime.
This commit is contained in:
Bartek Iwańczuk 2023-12-29 22:15:22 +01:00 committed by GitHub
parent 1dd1aba244
commit fa3a12a805
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 36 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
cli/tests/node_compat/test/internet/test-dns-any.js
View File

@ -172,21 +172,23 @@ TEST(async function test_google_for_cname_and_srv(done) {
checkWrap(req);
});
TEST(async function test_ptr(done) {
function validateResult(res) {
const types = processResult(res);
assert.ok(types.PTR);
}
// TODO(bartlomieju): this test started failing on CI on Dec 28th, 2023 returning
// ENOTFOUND. It's unclear what's going on, since `dig -x 8.8.8.8.in-addr.arpa`
// TEST(async function test_ptr(done) {
// function validateResult(res) {
// const types = processResult(res);
// assert.ok(types.PTR);
// }
validateResult(await dnsPromises.resolve('8.8.8.8.in-addr.arpa', 'ANY'));
// validateResult(await dnsPromises.resolve('8.8.8.8.in-addr.arpa', 'ANY'));
const req = dns.resolve(
'8.8.8.8.in-addr.arpa',
'ANY',
common.mustSucceed((ret) => {
validateResult(ret);
done();
}));
// const req = dns.resolve(
// '8.8.8.8.in-addr.arpa',
// 'ANY',
// common.mustSucceed((ret) => {
// validateResult(ret);
// done();
// }));
checkWrap(req);
});
// checkWrap(req);
// });

35
cli/tests/node_compat/test/internet/test-dns.js
View File

@ -281,26 +281,29 @@ TEST(async function test_resolveSrv(done) {
// checkWrap(req);
// });
TEST(async function test_resolvePtr(done) {
function validateResult(result) {
assert.ok(result.length > 0);
// TODO(bartlomieju): this test started failing on CI on Dec 28th, 2023 returning
// ENOTFOUND. It's unclear what's going on, since `dig -x 8.8.8.8.in-addr.arpa`
// returns correct PTR record.
// TEST(async function test_resolvePtr(done) {
// function validateResult(result) {
// assert.ok(result.length > 0);
for (const item of result) {
assert.ok(item);
assert.strictEqual(typeof item, 'string');
}
}
// for (const item of result) {
// assert.ok(item);
// assert.strictEqual(typeof item, 'string');
// }
// }
validateResult(await dnsPromises.resolvePtr(addresses.PTR_HOST));
// validateResult(await dnsPromises.resolvePtr(addresses.PTR_HOST));
const req = dns.resolvePtr(addresses.PTR_HOST, function(err, result) {
assert.ifError(err);
validateResult(result);
done();
});
// const req = dns.resolvePtr(addresses.PTR_HOST, function(err, result) {
// assert.ifError(err);
// validateResult(result);
// done();
// });
checkWrap(req);
});
// checkWrap(req);
// });
// TODO(bartlomieju): this test became very flaky on CI, returning `UNKNOWN`
// instead of `ENOTFOUND`.