feat(bitswap.unwant) expose bitswap.unwant to cli and http api

Author: wraithgar

src/cli/commands/bitswap/unwant.js

@@ -1,11 +1,21 @@
 'use strict'
 
+const print = require('../../utils').print
+
 module.exports = {
   command: 'unwant <key>',
 
-  describe: 'Remove a given block from your wantlist.',
+  describe: 'Removes a given block from your wantlist.',
 
+  builder: {
+    key: {
+      alias: 'k',
+      describe: 'Key to remove from your wantlist',
+      type: 'string'
+    }
+  },
   handler (argv) {
-    throw new Error('Not implemented yet')
+    argv.ipfs.bitswap.unwant(argv.key);
+    print(`Key ${argv.key} removed from wantlist`)
   }
 }

src/core/components/bitswap.js

@@ -40,12 +40,15 @@ module.exports = function bitswap (self) {
       })
     }),
 
+    //unwant: promisify((key, callback) => {
     unwant: (key) => {
       if (!self.isOnline()) {
         throw new Error(OFFLINE_ERROR)
+        //return setImmediate(() => callback(new Error(OFFLINE_ERROR)))
       }
 
-      // TODO: implement when https://github.com/ipfs/js-ipfs-bitswap/pull/10 is merged
+      self._bitswap.unwant(key)
     }
+    //})
   }
 }

src/http/api/resources/bitswap.js

@@ -46,10 +46,19 @@ exports.stat = (request, reply) => {
 }
 
 exports.unwant = {
-  // uses common parseKey method that returns a `key`
+  // uses common parseKey method that assigns a `key` to request.pre.args
   parseArgs: parseKey,
 
+  // main route handler which is called after the above `parseArgs`, but only if the args were valid
   handler: (request, reply) => {
-    reply(boom.badRequest(new Error('Not implemented yet')))
+    const key = request.pre.args.key
+    const ipfs = request.server.app.ipfs
+    try {
+      ipfs.bitswap.unwant(key)
+    } catch (err) {
+      return reply(boom.badRequest(err))
+    }
+
+    reply({ Key: key })
   }
 }

test/cli/bitswap.js

@@ -23,8 +23,7 @@ describe('bitswap', () => runOn((thing) => {
     })
   })
 
-  // TODO @hacdias fix this with https://github.com/ipfs/js-ipfs/pull/1198
-  it.skip('stat', function () {
+  it('stat', function () {
     this.timeout(20 * 1000)
 
     return ipfs('bitswap stat').then((out) => {
@@ -40,4 +39,10 @@ describe('bitswap', () => runOn((thing) => {
       ].join('\n') + '\n')
     })
   })
+
+  it('unwant', function () {
+    return ipfs('bitswap unwant ' + key).then((out) => {
+      expect(out).to.eql(`Key ${key} removed from wantlist\n`)
+    })
+  })
 }))

test/core/bitswap.spec.js

@@ -283,7 +283,7 @@ skipOnWindows('bitswap', function () {
         })
       })
 
-      it('throws if offline', () => {
+      it('.unwant throws if offline', () => {
         expect(() => node.bitswap.unwant('my key')).to.throw(/online/)
       })
     })