DRY things up a bit

trufflesuite · Sep 15, 2020 · c390f0d · c390f0d
1 parent 47058ca
commit c390f0d
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/lib/subproviders/geth_api_double.js b/lib/subproviders/geth_api_double.js
@@ -579,10 +579,11 @@ GethApiDouble.prototype.evm_mine.minLength = 1;
  */
 GethApiDouble.prototype.evm_unlockUnknownAccount = function(address, callback) {
   // check if given address is already unlocked
-  if (this.state.unlocked_accounts[address.toLowerCase()] !== true) {
-    if (this.state.personal_accounts[address.toLowerCase()] !== true) {
+  address = address.toLowerCase();
+  if (this.state.unlocked_accounts[address] !== true) {
+    if (this.state.personal_accounts[address] !== true) {
       // unlock the account
-      this.state.unlocked_accounts[address.toLowerCase()] = true;
+      this.state.unlocked_accounts[address] = true;
       return callback(null, true);
 
     } else {
@@ -609,13 +610,14 @@ GethApiDouble.prototype.evm_unlockUnknownAccount = function(address, callback) {
  */
 GethApiDouble.prototype.evm_lockUnknownAccount = function(address, callback) {
   // Checks if given address is already unlocked
-  if (this.state.unlocked_accounts[address.toLowerCase()]) {
-    if (this.state.personal_accounts[address.toLowerCase()]) {
+  address = address.toLowerCase();
+  if (this.state.unlocked_accounts[address]) {
+    if (this.state.personal_accounts[address]) {
       // if the account is known to the `personal` namespace we may not lock it
       return callback(new Error("cannot lock known/personal account"))
     } else {
       // unlock the account
-      delete this.state.unlocked_accounts[address.toLowerCase()];
+      delete this.state.unlocked_accounts[address];
       return callback(null, true);
     }
   }