diff options
author | Suhas Daftuar <sdaftuar@chaincode.com> | 2017-10-20 16:10:08 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2017-10-23 17:23:07 +0200 |
commit | fdad04e6170cfab5028b64a02a091c717795c65e (patch) | |
tree | 9a82f3914dd44c02ff28061bc79288997c122ae3 | |
parent | 019c492490077a7581ab526b4208439200cbff24 (diff) | |
download | bitcoin-fdad04e6170cfab5028b64a02a091c717795c65e.tar.xz |
qa: Fix replace-by-fee race condition failures
Github-Pull: #11538
Rebased-From: c96b2e4f096780c75e3fa8acba496d71322583a1
-rwxr-xr-x | test/functional/replace-by-fee.py | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/test/functional/replace-by-fee.py b/test/functional/replace-by-fee.py index 5e1f33991b..43005e98b9 100755 --- a/test/functional/replace-by-fee.py +++ b/test/functional/replace-by-fee.py @@ -72,6 +72,10 @@ class ReplaceByFeeTest(BitcoinTestFramework): ["-mempoolreplacement=0"]] def run_test(self): + # Leave IBD and ensure nodes are synced + self.nodes[0].generate(1) + self.sync_all() + make_utxo(self.nodes[0], 1*COIN) self.log.info("Running test simple doublespend...") @@ -110,13 +114,18 @@ class ReplaceByFeeTest(BitcoinTestFramework): """Simple doublespend""" tx0_outpoint = make_utxo(self.nodes[0], int(1.1*COIN)) + # make_utxo may have generated a bunch of blocks, so we need to sync + # before we can spend the coins generated, or else the resulting + # transactions might not be accepted by our peers. + self.sync_all() + tx1a = CTransaction() tx1a.vin = [CTxIn(tx0_outpoint, nSequence=0)] tx1a.vout = [CTxOut(1*COIN, CScript([b'a']))] tx1a_hex = txToHex(tx1a) tx1a_txid = self.nodes[0].sendrawtransaction(tx1a_hex, True) - self.sync_all([self.nodes]) + self.sync_all() # Should fail because we haven't changed the fee tx1b = CTransaction() |