implements delete wallet, in backuptab

js/controllers/backup.js

@@ -41,7 +41,7 @@ angular.module('copayApp.controllers').controller('BackupController',
     $scope.deleteWallet = function() {
       var w = $rootScope.wallet;
       w.disconnect();
-      walletFactory.remove(w.id, function() {
+      walletFactory.delete(w.id, function() {
         controllerUtils.logout();
       });
     };

js/models/core/WalletFactory.js

@@ -177,10 +177,10 @@ WalletFactory.prototype.getWallets = function() {
   return ret;
 };
 
-WalletFactory.prototype.remove = function(walletId, cb) {
+WalletFactory.prototype.delete = function(walletId, cb) {
   var s = this.storage;
   this.log('## DELETING WALLET ID:' + walletId); //TODO
-  s.get(walletId, 'opts');
+  s.deleteWallet(walletId);
   return cb();
 };
 

js/models/storage/LocalEncrypted.js

@@ -135,7 +135,6 @@ Storage.prototype.getWalletIds = function() {
 
 Storage.prototype.getWallets = function() {
   var wallets = [];
-  var uniq = {};
   var ids = this.getWalletIds();
 
   for (var i in ids) {
@@ -148,23 +147,19 @@ Storage.prototype.getWallets = function() {
 };
 
 Storage.prototype.deleteWallet = function(walletId) {
-  var walletIds = [];
+  var toDelete = {};
-  var uniq = {};
+  toDelete['nameFor::' + walletId] = 1;
+
   for (var i = 0; i < localStorage.length; i++) {
     var key = localStorage.key(i);
     var split = key.split('::');
-    if (split.length == 2) {
+    if (split.length == 2 && split[0] === walletId) {
-      var walletId = split[0];
+      toDelete[key] = 1;
-
-      if (walletId === 'nameFor') continue;
-
-      if (typeof uniq[walletId] === 'undefined') {
-        walletIds.push(walletId);
-        uniq[walletId] = 1;
     }
   }
+  for (var i in toDelete) {
+    this.removeGlobal(i);
   }
- 
 };
 
 

test/mocks/FakeStorage.js

@@ -1,4 +1,3 @@
-
 var FakeStorage = function() {
   this.storage = {};
 };
@@ -13,28 +12,87 @@ FakeStorage.prototype.setGlobal = function (id, payload) {
 
 FakeStorage.prototype.getGlobal = function(id) {
   return this.storage[id];
-}
+};
+
+
+FakeStorage.prototype.removeGlobal = function(id) {
+  delete this.storage[id];
+};
+
 
 FakeStorage.prototype.set = function(wid, id, payload) {
-  this.storage[wid + '-' + id] = payload;
+  this.storage[wid + '::' + id] = payload;
 };
 
 FakeStorage.prototype.get = function(wid, id) {
-  return this.storage[wid + '-' +id];
+  return this.storage[wid + '::' + id];
-}
+};
 
 FakeStorage.prototype.clear = function() {
   delete this['storage'];
+};
+
+FakeStorage.prototype.getWalletIds = function() {
+  var walletIds = [];
+  var uniq = {};
+
+  for (var ii in this.storage) {
+    var split = ii.split('::');
+    if (split.length == 2) {
+      var walletId = split[0];
+      if (walletId !== 'nameFor' && typeof uniq[walletId] === 'undefined') {
+        walletIds.push(walletId);
+        uniq[walletId] = 1;
       }
+    }
+  }
+  return walletIds;
+};
+
+FakeStorage.prototype.deleteWallet = function(walletId) {
+  var toDelete = {};
+  toDelete['nameFor::' + walletId] = 1;
+
+  for (var key in this.storage) {
+    var split = key.split('::');
+    if (split.length == 2 && split[0] === walletId) {
+      toDelete[key] = 1;
+    }
+  }
+  for (var i in toDelete) {
+    this.removeGlobal(i);
+  }
+};
+
+
+FakeStorage.prototype.getName = function(walletId) {
+  return this.getGlobal('nameFor::' + walletId);
+};
+
+
+FakeStorage.prototype.setName = function(walletId, name) {
+  this.setGlobal('nameFor::' + walletId, name);
+};
+
 
 FakeStorage.prototype.getWallets = function() {
-  return [];
+  var wallets = [];
+  var ids = this.getWalletIds();
+
+  for (var i in ids) {
+    wallets.push({
+      id: ids[i],
+      name: this.getName(ids[i]),
+    });
+  }
+  return wallets;
 };
 
 FakeStorage.prototype.setFromObj = function(walletId, obj) {
-  for (var i in obj) {
+  for (var k in obj) {
-    this.storage[walletId + '-' + i] = obj[i];
+    this.set(walletId, k, obj[k]);
-  };
+  }
+  this.setName(walletId, obj.opts.name);
 };
 
 module.exports = require('soop')(FakeStorage);

test/mocks/FakeWallet.js

@@ -1,8 +1,10 @@
-
 var FakeWallet = function() {
+  this.id = 'testID';
   this.balance = 10000;
   this.safeBalance = 1000;
-  this.balanceByAddr={'1CjPR7Z5ZSyWk6WtXvSFgkptmpoi4UM9BC': 1000};
+  this.balanceByAddr = {
+    '1CjPR7Z5ZSyWk6WtXvSFgkptmpoi4UM9BC': 1000
+  };
 };
 
 FakeWallet.prototype.set = function(balance, safeBalance, balanceByAddr) {
@@ -28,6 +30,18 @@ FakeWallet.prototype.getBalance=function(cb){
   return cb(null, this.balance, this.balanceByAddr, this.safeBalance);
 };
 
+FakeWallet.prototype.setEnc = function(enc) {
+  this.enc = enc;
+};
+
+FakeWallet.prototype.toEncryptedObj = function() {
+  return this.enc;
+};
+
+FakeWallet.prototype.disconnect = function() {
+  this.disconnectCalled = 1;
+};
+
 // This mock is meant for karma, module.exports is not necesary.
 try {
   module.exports = require('soop')(FakeWallet);

test/test.WalletFactory.js

@@ -122,4 +122,28 @@ describe('WalletFactory model', function() {
 
   });
 
+  it('should be able to get current wallets', function() {
+    var wf = new WalletFactory(config, '0.0.1');
+    var w = wf.create({
+      name: 'test wallet'
+    });
+    var ws = wf.getWallets();
+    ws.length.should.equal(1);
+    ws[0].name.should.equal('test wallet');
+  });
+
+  it('should be able to delete wallet', function(done) {
+    var wf = new WalletFactory(config, '0.0.1');
+    var w = wf.create({
+      name: 'test wallet'
+    });
+    var ws = wf.getWallets();
+    ws.length.should.equal(1);
+    wf.delete(ws[0].id, function() {
+      ws = wf.getWallets();
+      ws.length.should.equal(0);
+      done();
+    });
+  });
+
 });

test/unit/controllers/controllersSpec.js

@@ -1,6 +1,14 @@
 //
 // test/unit/controllers/controllersSpec.js
 //
+
+// Replace saveAs plugin
+saveAsLastCall = null;
+saveAs = function(o) {
+  saveAsLastCall = o;
+};
+
+
 describe("Unit: Controllers", function() {
 
   var scope;
@@ -9,6 +17,48 @@ describe("Unit: Controllers", function() {
   beforeEach(module('copayApp.services'));
   beforeEach(module('copayApp.controllers'));
 
+  var config = {
+    requiredCopayers: 3,
+    totalCopayers: 5,
+    spendUnconfirmed: 1,
+    reconnectDelay: 100,
+    networkName: 'testnet',
+  };
+
+
+
+  describe.only('Backup Controller', function() {
+    var ctrl;
+    beforeEach(inject(function($controller, $rootScope) {
+      scope = $rootScope.$new();
+
+      $rootScope.wallet = new FakeWallet(config);
+      ctrl = $controller('BackupController', {
+        $scope: scope,
+        $modal: {},
+      });
+    }));
+
+    it('should have a Backup controller', function() {
+      expect(scope.title).equal('Backup');
+    });
+
+    it(' Backup controller #download', function() {
+      scope.wallet.setEnc('1234567');
+      expect(saveAsLastCall).equal(null);
+      scope.download();
+      expect(saveAsLastCall.size).equal(7);
+      expect(saveAsLastCall.type).equal('text/plain;charset=utf-8');
+    });
+
+
+    it(' Backup controller #delete', function() {
+      expect(scope.wallet).not.equal(undefined);
+      scope.deleteWallet();
+      expect(scope.wallet).equal(undefined);
+    });
+  });
+
   describe('Address Controller', function() {
     var addressCtrl;
     beforeEach(inject(function($controller, $rootScope) {
@@ -114,4 +164,3 @@ describe("Unit: Controllers", function() {
   });
 
 });
-