add cli:id to use the daemon if on + tests

src/cli/commands/id.js

@@ -1,8 +1,9 @@
 const Command = require('ronin').Command
 const IPFS = require('../../ipfs-core')
 const debug = require('debug')
-const log = debug('cli:id')
-log.error = debug('cli:id:error')
+const utils = require('../utils')
+const log = debug('cli')
+log.error = debug('cli:error')
 
 module.exports = Command.extend({
   desc: 'Shows IPFS Node ID info',
@@ -14,11 +15,23 @@ module.exports = Command.extend({
     }
   },
 
-  run: name => {
-    const node = new IPFS()
-    node.id((err, id) => {
-      if (err) { return log.error(err) }
-      console.log(id)
-    })
+  run: (name) => {
+    if (utils.isDaemonOn()) {
+      const ctl = utils.getAPICtl()
+      ctl.id((err, result) => {
+        if (err) {
+          return log.error(err)
+        }
+        console.log(result)
+      })
+    } else {
+      const node = new IPFS()
+      node.id((err, id) => {
+        if (err) {
+          return log.error(err)
+        }
+        console.log(id)
+      })
+    }
   }
 })

src/cli/utils.js

@@ -0,0 +1,32 @@
+const fs = require('fs')
+const os = require('os')
+const APIctl = require('ipfs-api')
+const multiaddr = require('multiaddr')
+const debug = require('debug')
+const log = debug('cli')
+log.error = debug('cli:error')
+
+exports = module.exports
+
+const repoPath = process.env.IPFS_PATH || os.homedir() + '/.ipfs'
+
+exports.isDaemonOn = isDaemonOn
+function isDaemonOn () {
+  try {
+    fs.readFileSync(repoPath + '/api')
+    log('daemon is on')
+    return true
+  } catch (err) {
+    log('daemon is off')
+    return false
+  }
+}
+
+exports.getAPICtl = () => {
+  if (!isDaemonOn) {
+    throw new Error('daemon is not on')
+  }
+
+  const apiAddr = multiaddr(fs.readFileSync(repoPath + '/api').toString())
+  return APIctl(apiAddr.toString())
+}

tests/test-cli/index.js

@@ -26,7 +26,7 @@ describe('cli', () => {
 
   const tests = fs.readdirSync(__dirname)
   tests.filter(file => {
-    if (file === 'index.js') {
+    if (file === 'index.js' || file === 'api.js') {
       return false
     } else {
       return true

tests/test-cli/test-bootstrap.js

@@ -2,7 +2,6 @@
 
 const expect = require('chai').expect
 const nexpect = require('nexpect')
-const httpAPI = require('../../src/http-api')
 
 describe('id', () => {
   describe('api offline', () => {
@@ -75,18 +74,6 @@ describe('id', () => {
   })
 
   describe('api running', () => {
-    before((done) => {
-      httpAPI.start(err => {
-        expect(err).to.not.exist
-        done()
-      })
-    })
-
-    after((done) => {
-      httpAPI.stop((err) => {
-        expect(err).to.not.exist
-        done()
-      })
-    })
+    // TODO
   })
 })

tests/test-cli/test-config.js

@@ -2,29 +2,10 @@
 
 const expect = require('chai').expect
 const nexpect = require('nexpect')
-const httpAPI = require('../../src/http-api')
 const fs = require('fs')
 
 describe('config', () => {
   describe('api offline', () => {
-    // TODO
-  })
-
-  describe('api running', () => {
-    before((done) => {
-      httpAPI.start(err => {
-        expect(err).to.not.exist
-        done()
-      })
-    })
-
-    after((done) => {
-      httpAPI.stop((err) => {
-        expect(err).to.not.exist
-        done()
-      })
-    })
-
     const repoTests = require('./index').repoTests
     const configPath = repoTests + '/config'
 
@@ -103,4 +84,8 @@ describe('config', () => {
         })
     })
   })
+
+  describe('api running', () => {
+    // TODO
+  })
 })

tests/test-cli/test-id.js

@@ -35,7 +35,7 @@ describe('id', () => {
 
   describe('api running', () => {
     before((done) => {
-      httpAPI.start(err => {
+      httpAPI.start((err) => {
         expect(err).to.not.exist
         done()
       })

tests/test-cli/test-version.js

@@ -2,7 +2,6 @@
 
 const expect = require('chai').expect
 const nexpect = require('nexpect')
-const httpAPI = require('../../src/http-api')
 
 describe('version', () => {
   describe('api offline', () => {
@@ -18,28 +17,6 @@ describe('version', () => {
   })
 
   describe('api running', () => {
-    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'])
-        .expect('0.4.0-dev')
-        .run((err, stdout, exitcode) => {
-          expect(err).to.not.exist
-          expect(exitcode).to.equal(0)
-          done()
-        })
-    })
+    // TODO
   })
 })

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

@@ -33,7 +33,7 @@ describe('id', () => {
       done()
     })
 
-    it('get the version', (done) => {
+    it('get the id', (done) => {
       ctl.id((err, result) => {
         expect(err).to.not.exist
         expect(result).to.deep.equal(idResult)