Skip to content

Commit

Permalink
test: Updated test/integration/core/dns reverse test to remove flaki… (
Browse files Browse the repository at this point in the history
  • Loading branch information
jsumners-nr authored Apr 3, 2024
1 parent 88bdace commit c32cb27
Showing 1 changed file with 10 additions and 17 deletions.
27 changes: 10 additions & 17 deletions test/integration/core/dns.tap.js
Original file line number Diff line number Diff line change
Expand Up @@ -128,26 +128,19 @@ test('resolveSrv', function (t) {
})

test('reverse', function (t) {
const reverse = dns.reverse
dns.reverse = (addr, cb) => {
cb(undefined, ['localhost'])
}
t.teardown(() => {
dns.reverse = reverse
})

const agent = setupAgent(t)
helper.runInTransaction(agent, function () {
dns.reverse('127.0.0.1', function (err, names) {
t.notOk(err, 'should not error')
let expected = []
if (names.length > 0) {
if (process.env.DOCKERIZED) {
if (names.length === 2) {
expected = ['127.0.0.1', 'localhost']
} else {
expected = ['localhost']
}
} else {
expected = ['nettuno', 'travis', 'vagrant']
}
}

expected.forEach((name) => {
t.not(names.indexOf(name), -1, 'should have expected name')
})
t.error(err, 'should not error')
t.not(names.indexOf('localhost'), -1, 'should have expected name')
verifySegments(t, agent, 'dns.reverse')
})
})
Expand Down

0 comments on commit c32cb27

Please sign in to comment.