From 245c01bc0fed585c4ac8ed05edf7ebe1a65de80b Mon Sep 17 00:00:00 2001
From: frankiebee <frankie.diamond@gmail.com>
Date: Wed, 4 Apr 2018 14:56:30 -0700
Subject: [PATCH] transactions - make #_validateTxParams not async and
 "linting" wink wink nudge nudge

---
 app/scripts/controllers/transactions.js | 19 ++++++++++++-------
 test/unit/tx-controller-test.js         | 19 ++++++++-----------
 2 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/app/scripts/controllers/transactions.js b/app/scripts/controllers/transactions.js
index 9568fcbb9..a73a8b36d 100644
--- a/app/scripts/controllers/transactions.js
+++ b/app/scripts/controllers/transactions.js
@@ -185,7 +185,7 @@ module.exports = class TransactionController extends EventEmitter {
 
   async addUnapprovedTransaction (txParams) {
     // validate
-    await this._validateTxParams(txParams)
+    this._validateTxParams(txParams)
     this._normalizeTxParams(txParams)
     // construct txMeta
     let txMeta = this.txStateManager.generateTxMeta({txParams})
@@ -317,13 +317,18 @@ module.exports = class TransactionController extends EventEmitter {
   _normalizeTxParams (txParams) {
     delete txParams.chainId
 
-    if ( !txParams.to ) delete txParams.to
-    else txParams.to = ethUtil.addHexPrefix(txParams.to)
-
+    if ( !txParams.to ) {
+      delete txParams.to
+    } else {
+      txParams.to = ethUtil.addHexPrefix(txParams.to)
+    }
     txParams.from = ethUtil.addHexPrefix(txParams.from).toLowerCase()
 
-    if (!txParams.data) delete txParams.data
-    else txParams.data = ethUtil.addHexPrefix(txParams.data)
+    if (!txParams.data) {
+      delete txParams.data
+    } else {
+      txParams.data = ethUtil.addHexPrefix(txParams.data)
+    }
 
     if (txParams.value) txParams.value = ethUtil.addHexPrefix(txParams.value)
 
@@ -331,7 +336,7 @@ module.exports = class TransactionController extends EventEmitter {
     if (txParams.gasPrice) txParams.gas = ethUtil.addHexPrefix(txParams.gas)
   }
 
-  async _validateTxParams (txParams) {
+  _validateTxParams (txParams) {
     this._validateFrom(txParams)
     this._validateRecipient(txParams)
     if ('value' in txParams) {
diff --git a/test/unit/tx-controller-test.js b/test/unit/tx-controller-test.js
index 81d32ae29..3fec9758f 100644
--- a/test/unit/tx-controller-test.js
+++ b/test/unit/tx-controller-test.js
@@ -210,28 +210,25 @@ describe('Transaction Controller', function () {
     })
   })
 
-  describe('#validateTxParams', function () {
-    it('does not throw for positive values', function (done) {
+  describe('#_validateTxParams', function () {
+    it('does not throw for positive values', function () {
       var sample = {
         from: '0x1678a085c290ebd122dc42cba69373b5953b831d',
         value: '0x01',
       }
-      txController._validateTxParams(sample).then(() => {
-        done()
-      }).catch(done)
+      txController._validateTxParams(sample)
     })
 
-    it('returns error for negative values', function (done) {
+    it('returns error for negative values', function () {
       var sample = {
         from: '0x1678a085c290ebd122dc42cba69373b5953b831d',
         value: '-0x01',
       }
-      txController._validateTxParams(sample)
-      .then(() => done('expected to thrown on negativity values but didn\'t'))
-      .catch((err) => {
+      try {
+        txController._validateTxParams(sample)
+      } catch (err) {
         assert.ok(err, 'error')
-        done()
-      })
+      }
     })
   })