fix: return info on removed key #10

Author: richardschneider

src/keychain.js

@@ -186,11 +186,12 @@ class Keychain {
       return _error(callback, `Invalid key name '${name}'`)
     }
     const dsname = DsName(name)
-    self.store.has(dsname, (err, exists) => {
+    self._getKeyInfo(name, (err, keyinfo) => {
       if (err) return _error(callback, err)
-      if (!exists) return _error(callback, `Key '${name}' does not exist'`)
-
-      self.store.delete(dsname, callback)
+      self.store.delete(dsname, (err) => {
+        if (err) return _error(callback, err)
+        callback(null, keyinfo)
+      })
     })
   }
 

test/keychain.spec.js

@@ -345,8 +345,11 @@ module.exports = (datastore1, datastore2) => {
       })
 
       it('can remove a known key', (done) => {
-        ks.removeKey(renamedRsaKeyName, (err) => {
+        ks.removeKey(renamedRsaKeyName, (err, key) => {
           expect(err).to.not.exist()
+          expect(key).to.exist()
+          expect(key).to.have.property('name', renamedRsaKeyName)
+          expect(key).to.have.property('id', rsaKeyInfo.id)
           done()
         })
       })