fix: Always return the underlying request object

Author: dignifiedquire

src/api/add.js

@@ -8,15 +8,13 @@ module.exports = send => {
     }
 
     if (typeof files === 'string' && files.startsWith('http')) {
-      Wreck.request('GET', files, null, (err, res) => {
+      return Wreck.request('GET', files, null, (err, res) => {
         if (err) return cb(err)
 
         send('add', null, opts, res, cb)
       })
-
-      return
     }
 
-    send('add', null, opts, files, cb)
+    return send('add', null, opts, files, cb)
   }
 }

src/api/log.js

@@ -5,7 +5,7 @@ const ndjson = require('ndjson')
 module.exports = send => {
   return {
     tail (cb) {
-      send('log/tail', null, {}, null, false, (err, res) => {
+      return send('log/tail', null, {}, null, false, (err, res) => {
         if (err) return cb(err)
         cb(null, res.pipe(ndjson.parse()))
       })

src/api/pin.js

@@ -8,15 +8,15 @@ module.exports = send => {
         opts = null
       }
 
-      send('pin/add', hash, opts, null, cb)
+      return send('pin/add', hash, opts, null, cb)
     },
     remove (hash, opts, cb) {
       if (typeof opts === 'function') {
         cb = opts
         opts = null
       }
 
-      send('pin/rm', hash, opts, null, cb)
+      return send('pin/rm', hash, opts, null, cb)
     },
     list (type, cb) {
       if (typeof type === 'function') {

src/request-api.js

@@ -110,7 +110,7 @@ function requestAPI (config, path, args, qs, files, buffer, cb) {
     opts.payload = stream
   }
 
-  Wreck.request(opts.method, opts.uri, opts, onRes(buffer, cb))
+  return Wreck.request(opts.method, opts.uri, opts, onRes(buffer, cb))
 }
 
 // -- Interface

test/api/log.spec.js

@@ -2,9 +2,11 @@
 
 describe('.log', () => {
   it('.log.tail', done => {
-    apiClients['a'].log.tail((err, res) => {
+    const req = apiClients['a'].log.tail((err, res) => {
       expect(err).to.not.exist
 
+      expect(req).to.exist
+
       res.once('data', obj => {
         expect(obj).to.be.an('object')
         done()