From f5de16c91174fbbf208e5aef8f542d3bbbb3cb93 Mon Sep 17 00:00:00 2001
From: kumavis <kumavis@users.noreply.github.com>
Date: Fri, 7 Jul 2017 14:32:03 -0700
Subject: [PATCH] test - tx controller - fix promise handling
---
test/unit/tx-controller-test.js | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/test/unit/tx-controller-test.js b/test/unit/tx-controller-test.js
index 9dfe0b982..a5af13915 100644
--- a/test/unit/tx-controller-test.js
+++ b/test/unit/tx-controller-test.js
@@ -270,7 +270,7 @@ describe('Transaction Controller', function () {
})
- it('does not overwrite set values', function (done) {
+ it('does not overwrite set values', function () {
this.timeout(15000)
const wrongValue = '0x05'
@@ -289,9 +289,7 @@ describe('Transaction Controller', function () {
const pubStub = sinon.stub(txController.txProviderUtils, 'publishTransaction')
.callsArgWithAsync(1, null, originalValue)
- txController.approveTransaction(txMeta.id).then((err) => {
- assert.ifError(err, 'should not error')
-
+ return txController.approveTransaction(txMeta.id).then(() => {
const result = txController.getTx(txMeta.id)
const params = result.txParams
@@ -303,7 +301,6 @@ describe('Transaction Controller', function () {
priceStub.restore()
signStub.restore()
pubStub.restore()
- done()
})
})
})