Merge pull request #1102 from ethereum/fixTryTillResponse

Fix tryTillResponse test

Merge pull request #1102 from ethereum/fixTryTillResponse
f52a555c · yann300 · GitHub · 0fc53e2f · 075d03ef · f52a555c
Unverified Commit f52a555c authored Feb 26, 2018 by yann300 Committed by GitHub Feb 26, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

txRunner.js src/app/execution/txRunner.js +2 -2

No files found.
--- a/src/app/execution/txRunner.js
+++ b/src/app/execution/txRunner.js
@@ -204,8 +204,8 @@ TxRunner.prototype.runInNode = function (from, to, data, value, gasLimit, useCal

 function tryTillResponse (txhash, done) {
  executionContext.web3().eth.getTransactionReceipt(txhash, function (err, result) {
-    if (!err && !result) {
-      // Try again with a bit of delay
+    if (err || !result) {
+      // Try again with a bit of delay if error or if result still null
      setTimeout(function () { tryTillResponse(txhash, done) }, 500)
    } else {
      done(err, {