Tests: Explicitly set chain limits in replace-by-fee test #7137

Merged
merged 1 commit into from Dec 1, 2015

Conversation

Projects
None yet
5 participants
@sdaftuar
Member

sdaftuar commented Nov 30, 2015

The chains built in this test will fail with the lower chain limits introduced by #6771. I just changed them to be higher to avoid rewriting these tests, which aren't meant to test those chain limits anyway.

@petertodd Is it time to remove the qa/replace-by-fee directory? I can do that cleanup in this pull if you like.

Fixes #7120

@MarcoFalke

This comment has been minimized.

Show comment
Hide comment
@MarcoFalke

MarcoFalke Nov 30, 2015

Member

Nice, utACK cfdc662

remove the qa/replace-by-fee directory

+1

Member

MarcoFalke commented Nov 30, 2015

Nice, utACK cfdc662

remove the qa/replace-by-fee directory

+1

@paveljanik

This comment has been minimized.

Show comment
Hide comment
@paveljanik

paveljanik Dec 1, 2015

Contributor

ACK

Contributor

paveljanik commented Dec 1, 2015

ACK

@laanwj laanwj merged commit cfdc662 into bitcoin:master Dec 1, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

laanwj added a commit that referenced this pull request Dec 1, 2015

Merge pull request #7137
cfdc662 Explicitly set chain limits in replace-by-fee test (Suhas Daftuar)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment