Add api running test for ipfs version & fix ipfs#70

Author: fbaiodias

src/cli/commands/version.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const Command = require('ronin').Command
-const IPFS = require('../../ipfs-core')
+const utils = require('../utils')
 const debug = require('debug')
 const log = debug('cli:version')
 log.error = debug('cli:version:error')
@@ -26,11 +26,16 @@ module.exports = Command.extend({
   },
 
   run: (name) => {
-    var node = new IPFS()
-    node.version((err, version) => {
+    var ipfs = utils.getIPFS()
+    ipfs.version((err, version) => {
       if (err) { return log.error(err) }
 
-      console.log(version)
+      if (typeof version === 'object') { // js-ipfs-api output
+        console.log('ipfs version', version.Version)
+        return
+      }
+
+      console.log('ipfs version', version)
     })
   }
 })

src/http-api/resources/version.js

@@ -18,8 +18,7 @@ exports.get = (request, reply) => {
         Version: ipfsVersion,
         Commit: '',
         Repo: repoVersion
-      }).header('Transfer-Encoding', 'chunked')
-        .type('application/json')
+      })
     })
   })
 }

tests/test-cli/test-version.js

@@ -2,21 +2,44 @@
 
 const expect = require('chai').expect
 const nexpect = require('nexpect')
+const httpAPI = require('../../src/http-api')
 
 describe('version', () => {
   describe('api offline', () => {
     it('get the version', (done) => {
       nexpect.spawn('node', [process.cwd() + '/src/cli/bin.js', 'version'])
-        .expect('0.4.0-dev')
         .run((err, stdout, exitcode) => {
           expect(err).to.not.exist
+          expect(stdout[0]).to.equal('ipfs version 0.4.0-dev')
           expect(exitcode).to.equal(0)
           done()
         })
     })
   })
 
   describe('api running', () => {
-    // TODO
+    before((done) => {
+      httpAPI.start((err) => {
+        expect(err).to.not.exist
+        done()
+      })
+    })
+
+    after((done) => {
+      httpAPI.stop((err) => {
+        expect(err).to.not.exist
+        done()
+      })
+    })
+
+    it('get the version', (done) => {
+      nexpect.spawn('node', [process.cwd() + '/src/cli/bin.js', 'version'])
+        .run((err, stdout, exitcode) => {
+          expect(err).to.not.exist
+          expect(exitcode).to.equal(0)
+          expect(stdout[0]).to.equal('ipfs version 0.4.0-dev')
+          done()
+        })
+    })
   })
 })

tests/test-http-api/test-version.js

@@ -35,8 +35,7 @@ describe('version', () => {
       done()
     })
 
-    // TODO fix: only fails on travis
-    it.skip('get the version', (done) => {
+    it('get the version', (done) => {
       ctl.version((err, result) => {
         expect(err).to.not.exist
         expect(result).to.have.a.property('Version')