Skip to content
This repository was archived by the owner on Mar 10, 2020. It is now read-only.

Commit 6be0f63

Browse files
committed
fix: cleanup dns tests
'should resolve a DNS link' is a duplicate plus the `r: true` isn't documented 'should non-recursively resolve ipfs.io' doesn't need to be skiped anymore
1 parent cee7e9c commit 6be0f63

File tree

1 file changed

+4
-15
lines changed

1 file changed

+4
-15
lines changed

src/miscellaneous/dns.js

Lines changed: 4 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ module.exports = (createCommon, options) => {
88
const it = getIt(options)
99
const common = createCommon()
1010

11-
describe('.dns', () => {
11+
describe('.dns', function () {
12+
this.timeout(10 * 1000)
13+
this.retries(3)
1214
let ipfs
1315

1416
before(function (done) {
@@ -30,20 +32,7 @@ module.exports = (createCommon, options) => {
3032
common.teardown(done)
3133
})
3234

33-
it('should resolve a DNS link', function (done) {
34-
this.timeout(20 * 1000)
35-
this.retries(3)
36-
37-
ipfs.dns('ipfs.io', { r: true }, (err, path) => {
38-
expect(err).to.not.exist()
39-
expect(path).to.exist()
40-
done()
41-
})
42-
})
43-
44-
// skipping because there is an error in https://ipfs.io/api/v0/dns?arg=ipfs.io
45-
// unskip when it is resolved and the new version released: https://github.com/ipfs/go-ipfs/issues/6086
46-
it.skip('should non-recursively resolve ipfs.io', () => {
35+
it('should non-recursively resolve ipfs.io', () => {
4736
return ipfs.dns('ipfs.io', { recursive: false }).then(res => {
4837
// matches pattern /ipns/<ipnsaddress>
4938
expect(res).to.match(/\/ipns\/.+$/)

0 commit comments

Comments
 (0)