/bitcoin

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[qa] Extend tests #7684

Merged
merged 4 commits into from Mar 16, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
Tests
Projects
None yet
Milestone
No milestone
2 participants
@MarcoFalke @laanwj
@MarcoFalke
Member

MarcoFalke commented Mar 14, 2016

Please review commit-by-commit:

  • Parsing negative amounts must fail
  • No need to set up a network in wallet.py
  • Reuse a function (move-only)
  • Use constants (refactor-only)

@laanwj laanwj added the Tests label Mar 15, 2016

@laanwj laanwj merged commit fad8cfb into bitcoin:master Mar 16, 2016
1 check passed

1 check passed

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

laanwj added a commit that referenced this pull request Mar 16, 2016

@laanwj
Merge #7684: [qa] Extend tests 
fad8cfb [qa] mininode: Add and use CONSTs (MarcoFalke)
fa8cd46 [qa] Move create_tx() to util.py (MarcoFalke)
fad7dc8 [qa] wallet: speed up tests (MarcoFalke)
fa3a81a [tests] Extend util_ParseMoney test case (MarcoFalke)
This commit was signed with a verified signature.
@laanwj laanwj Wladimir J. van der Laan
GPG key ID: 74810B012346C9A6 Learn about signing commits
3d0dfdb

@MarcoFalke MarcoFalke deleted the MarcoFalke:Mf1603-qaCleanup1 branch Mar 17, 2016

@sdaftuar sdaftuar referenced this pull request Mar 18, 2016

Merged

Tests: fix missing import in mempool_packages #7709

MarcoFalke added a commit to MarcoFalke/bitcoin that referenced this pull request Apr 5, 2016

@MarcoFalke
[qa] Extend tests 
Github-Pull: #7684
Rebased-From: fa3a81a fad7dc8 fad8cfb
ad8c743

zander added a commit to zander/bitcoinclassic that referenced this pull request Jun 16, 2016

@MarcoFalke @zander
[qa] Extend tests 
Github-Pull: #7684
Rebased-From: fa3a81a fad7dc8 fad8cfb

Conflicts:
	qa/rpc-tests/maxuploadtarget.py
0b3a67c

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 11, 2016

@MarcoFalke @nomnombtc
[qa] Extend tests 
nomnombtc: minus the changes to maxuploadtarget.py

Github-Pull: #7684
Rebased-From: fa3a81a fad7dc8 fad8cfb
3043049

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 12, 2016

@MarcoFalke @nomnombtc
[qa] Extend tests 
nomnombtc: minus the changes to maxuploadtarget.py

Github-Pull: #7684
Rebased-From: fa3a81a fad7dc8 fad8cfb
b61d3e8

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 12, 2016

@MarcoFalke @nomnombtc
[qa] Extend tests 
Github-Pull: #7684
Rebased-From: fa3a81a fad7dc8 fad8cfb
d50efdd

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 13, 2016

@MarcoFalke @nomnombtc
[qa] Extend tests 
nomnombtc: minus the changes to maxuploadtarget.py

Github-Pull: #7684
Rebased-From: fa3a81a fad7dc8 fad8cfb
676df45

sickpig added a commit to sickpig/BitcoinUnlimited that referenced this pull request Nov 14, 2016

@MarcoFalke @sickpig
[qa] Extend tests 
Github-Pull: #7684
Rebased-From: fa3a81a fad7dc8 fad8cfb
This commit was signed with a verified signature.
@sickpig sickpig Andrea Suisani
GPG key ID: A9CA1D0EE3789741 Learn about signing commits
38e1d5c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment