Skip to content

test: Allow --usecli in more tests#35363

Open
maflcko wants to merge 4 commits into
bitcoin:masterfrom
maflcko:2605-test-use-cli-more
Open

test: Allow --usecli in more tests#35363
maflcko wants to merge 4 commits into
bitcoin:masterfrom
maflcko:2605-test-use-cli-more

Conversation

@maflcko
Copy link
Copy Markdown
Member

@maflcko maflcko commented May 22, 2026

Some tests disallow to be run under --usecli. This reduces the coverage and risks that bugs in the bitcoin-cli go unnoticed.

The commits should be self-explanatory and can be reviewed and tested one-by-one.

MarcoFalke added 4 commits May 22, 2026 17:48
Otherwise, bitcoin-cli will read the wrong config from the "future".
The echo and echojson RPCs are identical in the server. The only
difference is that echojson is in the client conversion table.
@DrahtBot DrahtBot added the Tests label May 22, 2026
@DrahtBot
Copy link
Copy Markdown
Contributor

DrahtBot commented May 22, 2026

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35363.

Reviews

See the guideline for information on the review process.
A summary of reviews will appear here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants