fix(version): better http-api and interface-ipfs-core compliant

Author: dignifiedquire

src/cli/commands/version.js

@@ -37,11 +37,7 @@ module.exports = {
           throw err
         }
 
-        if (typeof version === 'object') { // js-ipfs-api output
-          version = version.Version
-        }
-
-        console.log(`js-ipfs version: ${version}`)
+        console.log(`js-ipfs version: ${version.version}`)
       })
     })
   }

src/http-api/resources/version.js

@@ -5,11 +5,17 @@ const boom = require('boom')
 exports = module.exports
 
 exports.get = (request, reply) => {
-  request.server.app.ipfs.version((err, version) => {
+  const ipfs = request.server.app.ipfs
+
+  ipfs.version((err, version) => {
     if (err) {
       return reply(boom.badRequest(err))
     }
 
-    reply(version)
+    reply({
+      Version: version.version,
+      Commit: version.commit,
+      Repo: version.repo
+    })
   })
 }

test/cli/test-version.js

@@ -12,7 +12,7 @@ describe('version', () => {
   const env = _.clone(process.env)
   env.IPFS_PATH = repoPath
 
-  describe.skip('api offline', () => {
+  describe('api offline', () => {
     it('get the version', (done) => {
       nexpect.spawn('node', [process.cwd() + '/src/cli/bin.js', 'version'], {env})
         .run((err, stdout, exitcode) => {
@@ -24,7 +24,7 @@ describe('version', () => {
     })
   })
 
-  describe.skip('api running', () => {
+  describe('api running', () => {
     let httpAPI
 
     before((done) => {

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

@@ -17,9 +17,9 @@ module.exports = (http) => {
         method: 'GET',
         url: '/api/v0/version'
       }, (res) => {
-        expect(res.result.version).to.equal(pkgversion)
-        expect(res.result).to.have.a.property('commit')
-        expect(res.result).to.have.a.property('repo')
+        expect(res.result).to.have.a.property('Version', pkgversion)
+        expect(res.result).to.have.a.property('Commit')
+        expect(res.result).to.have.a.property('Repo')
         done()
       })
     })