Page MenuHomePhabricator

Merge #11838: qa: Add getrawtransaction in_active_chain=False test
ClosedPublic

Authored by jasonbcox on Apr 13 2019, 16:00.

Details

Summary

fa4c16d qa: Add getrawtransaction in_active_chain=False test (MarcoFalke)

Pull request description:

#10275 accidentally forgot to add a test for `in_active_chain==False`.

This adds a test and also removes the special casing of `blockhash.IsNull()`, which makes no sense imo.

Tree-SHA512: 6c51295820b3dcd53b0b48020ab2b8c8f5864cd5061ddab2b35d35d643eb3e60ef95ff20c06c985a2e47f7080e82f27f3e00ee61c85dce627776d5ea6febee8f

Backport of Core PR 11838
https://github.com/bitcoin/bitcoin/pull/11838/files
Completes T593

Test Plan

make check
test_runner.py rpc_rawtransaction

Diff Detail

Repository
rABC Bitcoin ABC
Branch
pr11838
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5447
Build 8956: Bitcoin ABC Buildbot (legacy)
Build 8955: arc lint + arc unit