Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[QA] fix possible reorg issue in (fund)rawtransaction(s).py RPC test #6417

merged 2 commits into from Jul 13, 2015


Copy link

@jonasschnelli jonasschnelli commented Jul 10, 2015

  • added missing mempool sync between blocks generating

Also removed checks of expected inputs because the coin selection is not exact foreseeable.

Fixes #6391.
Thanks @sdaftuar and @morcos for analyzing this issue!

….py RPC test

- added missing mempool sync between block generations
@jonasschnelli jonasschnelli force-pushed the 2015/07/qa_fix_missing_sync branch from 930a93a to 6ed38b0 Compare Jul 10, 2015
@jonasschnelli jonasschnelli changed the title [QA] fix possible reorg issue in RPC test [QA] fix possible reorg issue in (fund)rawtransaction(s).py RPC test Jul 10, 2015 is already in the standard test list
@jonasschnelli jonasschnelli force-pushed the 2015/07/qa_fix_missing_sync branch from f44ca0b to 9da8fc8 Compare Jul 10, 2015
Copy link

sdaftuar commented Jul 10, 2015

ACK, thanks for fixing!

@laanwj laanwj added the Tests label Jul 13, 2015
@laanwj laanwj merged commit 9da8fc8 into bitcoin:master Jul 13, 2015
1 check passed
laanwj added a commit that referenced this issue Jul 13, 2015
9da8fc8 [QA] remove from the extended test list (Jonas Schnelli)
6ed38b0 [QA] fix possible reorg issue in RPC test (Jonas Schnelli)
luke-jr pushed a commit to luke-jr/bitcoin that referenced this issue Jan 9, 2016
… RPC test

- added missing mempool sync between block generations

Github-Pull: bitcoin#6417
Partial-Rebased-From: 6ed38b0
furszy added a commit to PIVX-Project/PIVX that referenced this issue Jun 28, 2020
fc81158 [QA] Add test_change_position case to (random-zebra)
dd35760 [QA] Add test_option_feerate to rpc_fundrawtransaction functional test (random-zebra)
5bca4f4 Add more clear interface for CoinControl.h regarding individual feerate (random-zebra)
169bc3b [RPC] add feerate option to fundrawtransaction (random-zebra)
87dbdf8 [QA] Test new options in rpc_fundrawtransaction functional test (random-zebra)
bc9dc67 Add lockUnspents option to fundrawtransaction (random-zebra)
a3ac191 Add change options to fundrawtransaction (random-zebra)
0c1f7ba Add strict flag to RPCTypeCheckObj (random-zebra)
d655b42 Use CCoinControl selection in CWallet::FundTransaction (random-zebra)
76c8d54 [QA] Test watchonly addrs in fundrawtransaction tests (random-zebra)
134c5d2 Implement watchonly support in fundrawtransaction (random-zebra)
1b153e5 Update importaddress help to push its use to script-only (random-zebra)
7b4eb6d Add importpubkey method to import a watch-only pubkey (random-zebra)
816dabb Add p2sh option to importaddress to import redeemScripts (random-zebra)
60a20a4 Split up importaddress into helper functions (random-zebra)
cbffa80 Add logic to track pubkeys as watch-only, not just scripts (random-zebra)
12b38b0 Add have-pubkey distinction to ISMINE flags (random-zebra)
fab6556 Exempt unspendable transaction outputs from dust checks (random-zebra)
ab407ff [Tests] Fix and enable fundrawtransaction functional tests (random-zebra)
bc44ba0 [wallet] allow transaction without change if keypool is empty (random-zebra)
a2f8071 [wallet] CreateTransaction: simplify change address check (random-zebra)
761e60e Add fundrawtransaction RPC method (random-zebra)
ccb18dd Add FundTransaction method to wallet (random-zebra)
692b827 Add DummySignatureCreator which just creates zeroed sigs (random-zebra)

Pull request description:

  based on top of
  - [x] #1662

  This introduces a new wallet function, `CWallet::FundTransaction()` (and exposes it via RPC with `fundrawtransaction`), to fill a tx containing only vouts (or not enough vins to cover the vouts) with unspent coins from the wallet.

  `fundrawtransaction` will not modify existing inputs, and will add one change output (if needed) to the outputs. It will not sign the inputs (so can include also watch-only or multi-sig inputs, if enabled).

  backported from:
  - bitcoin#6088
  - bitcoin#17219 [`*`]
  - bitcoin#6417
  - bitcoin#6444
  - bitcoin#6415
  - bitcoin#6828
  - bitcoin#7296 (only bebe58b)
  - bitcoin#7506
  - bitcoin#7518
  - bitcoin#7967

  adapting the tests for the (more recent) framework.

  [`*`] Note: this has been included to be able to call `fundrawtransaction` without the need for an unencrypted wallet (for the change address key)

ACKs for top commit:
    re ACK fc81158 .
    ACK fc81158

Tree-SHA512: 10235ce6e672a1cfd4ae2cad9312864c82971f6a4aa1a8ed9489d85156f5c4126c293180a7f1b86b7c65d07caab484e9a6d7a87ebf032bee55adb98d3e08e7b9
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
None yet

Successfully merging this pull request may close these issues.

3 participants