Fix things

Author: dignifiedquire

src/request-api.js

@@ -29,12 +29,18 @@ function onRes (buffer, cb) {
         cb(error)
       })
     }
-    console.log(res.headers)
+
     if (stream && !buffer) return cb(null, res)
 
     if (chunkedObjects) {
       const parsed = []
-      res.on('data', chunk => parsed.push(JSON.parse(chunk)))
+      res.on('data', chunk => {
+        try {
+          parsed.push(JSON.parse(chunk))
+        } catch (err) {
+          // Browser quirks emit more than needed sometimes
+        }
+      })
       res.on('end', () => cb(null, parsed))
       return
     }

test/tests.js

@@ -193,13 +193,6 @@ describe('IPFS Node.js API wrapper tests', () => {
           throw err
         }
 
-        if (!res.on) {
-          // Just  a string
-          assert.equal(res.toString(), testfile)
-          done()
-          return
-        }
-
         let buf = ''
         res
           .on('error', err => { throw err })
@@ -391,13 +384,6 @@ describe('IPFS Node.js API wrapper tests', () => {
       apiClients['a'].block.get(blorbKey, (err, res) => {
         if (err) throw err
 
-        if (!res.on) {
-          // Just  a string
-          assert.equal(res.toString(), 'blorb')
-          done()
-          return
-        }
-
         let buf = ''
         res
           .on('data', function (data) { buf += data })
@@ -442,13 +428,6 @@ describe('IPFS Node.js API wrapper tests', () => {
       apiClients['a'].object.data(testObjectHash, (err, res) => {
         if (err) throw err
 
-        if (!res.on) {
-          // Just  a string
-          assert.equal(res.toString(), 'testdata')
-          done()
-          return
-        }
-
         let buf = ''
         res
           .on('error', err => { throw err })
@@ -729,6 +708,8 @@ describe('IPFS Node.js API wrapper tests', () => {
           throw err
         }
 
+        assert.equal(typeof res, 'object')
+
         return done()
 
         // non ipns or pk hashes fail to fetch, known bug
@@ -749,7 +730,8 @@ describe('IPFS Node.js API wrapper tests', () => {
         if (err) {
           throw err
         }
-        console.log(res)
+
+        assert.equal(typeof res, 'object')
         assert(res)
         done()
       })