Page MenuHomePhabricator

Add bitcoin-cli -stdin and -stdinrpcpass + tests
ClosedPublic

Authored by matiu on Jan 30 2018, 14:18.

Details

Summary

Backport from Core's PR10997 and PR11125

PR10997:

79191f5 Add option -stdinrpcpass to allow RPC password to be read from standa…

and the tests for it:

PR11125:

29e1dfb [test] Add bitcoin-cli -stdin and -stdinrpcpass functional tests (João Barbosa)
ce379b4 [test] Replace check_output with low level version (João Barbosa)
232e3e8 [test] Add assert_raises_process_error to assert process errors (João Barbosa)
5c18a84 [test] Add support for custom arguments to TestNodeCLI (João Barbosa)
e127494 [test] Improve assert_raises_jsonrpc docstring (João Barbosa)
7696841 Fix style in -stdin and -stdinrpcpass handling (João Barbosa)
Test Plan

make check && ./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
t210
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1766
Build 1766: arc lint + arc unit

Event Timeline

matiu retitled this revision from Summary: Add bitcoin-cli -stdin and -stdinrpcpass functional tests Add support for custom arguments to TestNodeCLI Improve assert_raises_jsonrpc docstring Replace check_output with low level version Add assert_raises_process_error to assert... to Add bitcoin-cli -stdin and -stdinrpcpass functional tests.Jan 30 2018, 14:19
matiu edited the summary of this revision. (Show Details)
matiu retitled this revision from Add bitcoin-cli -stdin and -stdinrpcpass functional tests to [WIP] Add bitcoin-cli -stdin and -stdinrpcpass functional tests.Jan 30 2018, 14:21
matiu retitled this revision from [WIP] Add bitcoin-cli -stdin and -stdinrpcpass functional tests to Add bitcoin-cli -stdin and -stdinrpcpass functional tests.Jan 30 2018, 14:34
matiu edited the summary of this revision. (Show Details)
deadalnix requested changes to this revision.Jan 30 2018, 21:54
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/bitcoin-cli.cpp
345

This is not from PR11125

This revision now requires changes to proceed.Jan 30 2018, 21:54
matiu edited the summary of this revision. (Show Details)
  • rm code from bitcoin-cli.cpp from other PR.
matiu retitled this revision from Add bitcoin-cli -stdin and -stdinrpcpass functional tests to Add bitcoin-cli -stdin and -stdinrpcpass + tests.Jan 31 2018, 00:44
matiu edited the summary of this revision. (Show Details)
schancel edited the test plan for this revision. (Show Details)
schancel added a subscriber: schancel.
schancel added inline comments.
src/bitcoin-cli.cpp
345

I believe I just landed the patch for this. I had made the task for this a requirement for this PR. Rebasing should address it.

matiu edited the summary of this revision. (Show Details)

rebased

matiu planned changes to this revision.Feb 1 2018, 01:44
matiu marked 2 inline comments as done.
matiu added inline comments.
src/bitcoin-cli.cpp
345

PR11125 where the tests for PR10997
Sorry about it. I added PR10997 changed.

matiu requested review of this revision.Feb 1 2018, 01:44
This revision is now accepted and ready to land.Feb 2 2018, 14:28
This revision was automatically updated to reflect the committed changes.
matiu marked an inline comment as done.