chore: update ipfs-http-client peer dep

package.json

@@ -22,10 +22,10 @@
     "coverage": "aegir coverage"
   },
   "devDependencies": {
-    "aegir": "^26.0.0",
-    "go-ipfs": "0.6.0",
-    "ipfs-http-client": "^46.0.0",
-    "ipfs-utils": "^3.0.0",
+    "aegir": "^28.0.2",
+    "go-ipfs": "^0.7.0",
+    "ipfs-http-client": "^47.0.1",
+    "ipfs-utils": "^4.0.0",
     "ipfsd-ctl": "^7.0.0",
     "it-all": "^1.0.2"
   },
@@ -37,7 +37,7 @@
     "peer-id": "^0.14.0"
   },
   "peerDependencies": {
-    "ipfs-http-client": "^46.0.0"
+    "ipfs-http-client": "^47.0.1"
   },
   "browser": {
     "go-ipfs": false

src/index.js

@@ -45,7 +45,7 @@ class DelegatedPeerRouting {
    *
    * @param {PeerID} id
    * @param {object} options
-   * @param {number} options.timeout How long the query can take. Defaults to 30 seconds
+   * @param {number} options.timeout - How long the query can take. Defaults to 30 seconds
    * @returns {Promise<{ id: PeerId, multiaddrs: Multiaddr[] }>}
    */
   async findPeer (id, options = {}) {
@@ -82,9 +82,10 @@ class DelegatedPeerRouting {
 
   /**
    * Attempt to find the closest peers on the network to the given key
-   * @param {Uint8Array} key A CID like key
+   *
+   * @param {Uint8Array} key - A CID like key
    * @param {object} [options]
-   * @param {number} [options.timeout=30e3] How long the query can take.
+   * @param {number} [options.timeout=30e3] - How long the query can take.
    * @returns {AsyncIterable<{ id: PeerId, multiaddrs: Multiaddr[] }>}
    */
   async * getClosestPeers (key, options = {}) {

test/index.spec.js

@@ -14,20 +14,16 @@ const factory = createFactory({
   ipfsHttpModule: require('ipfs-http-client'),
   ipfsBin: isNode ? require('go-ipfs').path() : undefined,
   test: true,
+  disposable: true,
   endpoint: 'http://localhost:57483'
 })
 
-async function spawnNode (boostrap = []) {
+async function spawnNode (bootstrap = []) {
   const node = await factory.spawn({
     // Lock down the nodes so testing can be deterministic
     ipfsOptions: {
       config: {
-        Bootstrap: boostrap,
-        Discovery: {
-          MDNS: {
-            Enabled: false
-          }
-        }
+        Bootstrap: bootstrap
       }
     }
   })
@@ -107,7 +103,10 @@ describe('DelegatedPeerRouting', function () {
         host: opts.host
       }))
 
-      const { id, multiaddrs } = await router.findPeer(peerIdToFind.id)
+      const peer = await router.findPeer(peerIdToFind.id)
+      expect(peer).to.be.ok()
+
+      const { id, multiaddrs } = peer
       expect(id).to.exist()
       expect(multiaddrs).to.exist()
       expect(id).to.eql(peerIdToFind.id)
@@ -121,7 +120,10 @@ describe('DelegatedPeerRouting', function () {
         host: opts.host
       }))
 
-      const { id, multiaddrs } = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id))
+      const peer = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id))
+      expect(peer).to.be.ok()
+
+      const { id, multiaddrs } = peer
       expect(id).to.exist()
       expect(multiaddrs).to.exist()
 
@@ -136,7 +138,10 @@ describe('DelegatedPeerRouting', function () {
         host: opts.host
       }))
 
-      const { id, multiaddrs } = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id), { timeout: 2000 })
+      const peer = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id), { timeout: 2000 })
+      expect(peer).to.be.ok()
+
+      const { id, multiaddrs } = peer
       expect(id).to.exist()
       expect(multiaddrs).to.exist()