From 78f2794d39ccfff79a6562d6fdc628f375291598 Mon Sep 17 00:00:00 2001 From: Dan Finlay Date: Thu, 30 Jun 2016 11:34:34 -0700 Subject: [PATCH] Remove useless and buggy clearWallet function --- app/scripts/lib/config-manager.js | 2 -- app/scripts/lib/idStore.js | 3 --- test/unit/config-manager-test.js | 11 ----------- 3 files changed, 16 deletions(-) diff --git a/app/scripts/lib/config-manager.js b/app/scripts/lib/config-manager.js index ca115d49e..caaae8a75 100644 --- a/app/scripts/lib/config-manager.js +++ b/app/scripts/lib/config-manager.js @@ -149,8 +149,6 @@ ConfigManager.prototype.getCurrentRpcAddress = function () { } } -ConfigManager.prototype.clearWallet = function () {} - ConfigManager.prototype.setData = function (data) { this.migrator.saveData(data) } diff --git a/app/scripts/lib/idStore.js b/app/scripts/lib/idStore.js index f705c07a7..cbc8c1e48 100644 --- a/app/scripts/lib/idStore.js +++ b/app/scripts/lib/idStore.js @@ -43,9 +43,6 @@ function IdentityStore (opts = {}) { IdentityStore.prototype.createNewVault = function (password, entropy, cb) { delete this._keyStore - if (this.configManager) { - this.configManager.clearWallet() - } this._createIdmgmt(password, null, entropy, (err) => { if (err) return cb(err) diff --git a/test/unit/config-manager-test.js b/test/unit/config-manager-test.js index 69c462286..8974a6bc5 100644 --- a/test/unit/config-manager-test.js +++ b/test/unit/config-manager-test.js @@ -42,17 +42,6 @@ describe('config-manager', function() { }) }) - describe('#clearWallet', function() { - it('should not erase confirmation', function() { - configManager.setConfirmed(true) - assert.equal(configManager.getConfirmed(), true) - - configManager.clearWallet() - - assert.equal(configManager.getConfirmed(), true) - }) - }) - describe('#setConfig', function() { window.localStorage = {} // Hacking localStorage support into JSDom