Page MenuHomePhabricator

[avalanche] Extract a proof verification facility from the RPC
ClosedPublic

Authored by Fabien on Jun 8 2021, 12:39.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Maniphest Tasks
Restricted Maniphest Task
Commits
rABC10b624ffb4f5: [avalanche] Extract a proof verification facility from the RPC
Summary

This makes it possible to reuse the verification from other RPCs and
return consistent messages. There is no change in behavior.

Depends on D9649.

Ref T1611.

Test Plan
./test/functional/test_runner.py abc_rpc_avalancheproof

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_extract_verifyproof_rpc
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15941
Build 31782: Build Difflint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 31781: arc lint + arc unit

Event Timeline

Fabien requested review of this revision.Jun 8 2021, 12:40

Tail of the build log:

wallet_importdescriptors.py             | ✓ Passed  | 7 s
wallet_importmulti.py                   | ✓ Passed  | 6 s
wallet_importprunedfunds.py             | ✓ Passed  | 2 s
wallet_keypool.py                       | ✓ Passed  | 3 s
wallet_keypool_topup.py                 | ✓ Passed  | 4 s
wallet_labels.py                        | ✓ Passed  | 3 s
wallet_listreceivedby.py                | ✓ Passed  | 19 s
wallet_listsinceblock.py                | ✓ Passed  | 6 s
wallet_listtransactions.py              | ✓ Passed  | 9 s
wallet_multiwallet.py                   | ✓ Passed  | 19 s
wallet_multiwallet.py --usecli          | ✓ Passed  | 18 s
wallet_reorgsrestore.py                 | ✓ Passed  | 5 s
wallet_resendwallettransactions.py      | ✓ Passed  | 7 s
wallet_txn_clone.py                     | ✓ Passed  | 3 s
wallet_txn_clone.py --mineblock         | ✓ Passed  | 4 s
wallet_txn_doublespend.py               | ✓ Passed  | 3 s
wallet_txn_doublespend.py --mineblock   | ✓ Passed  | 4 s
wallet_watchonly.py                     | ✓ Passed  | 1 s
wallet_watchonly.py --usecli            | ✓ Passed  | 1 s
wallet_zapwallettxes.py                 | ✓ Passed  | 6 s

ALL                                     | ✓ Passed  | 1381 s (accumulated) 
Runtime: 278 s

----------------------------------------------------------------------
Ran 5 tests in 0.002s

OK

[28/453] Test Bitcoin RPC authentication...
...
----------------------------------------------------------------------
Ran 3 tests in 0.004s

OK
[30/453] cd /work/contrib/devtools/chainparams && /usr/bin/python3.7 ./test_make_chainparams.py
.....
----------------------------------------------------------------------
Ran 5 tests in 0.001s

OK
[200/453] Running seeder test suite
PASSED: seeder test suite
[204/453] Running avalanche test suite
PASSED: avalanche test suite
[208/453] Running pow test suite
PASSED: pow test suite
[210/453] Running bitcoin-qt test suite
PASSED: bitcoin-qt test suite
[352/453] bitcoin: testing wallet_tests
FAILED: src/test/CMakeFiles/check-bitcoin-wallet_tests 
cd /work/abc-ci-builds/build-debug/src/test && /usr/bin/cmake -E make_directory /work/abc-ci-builds/build-debug/test/junit && /usr/bin/cmake -E make_directory /work/abc-ci-builds/build-debug/test/log && /usr/bin/cmake -E env /work/cmake/utils/log-and-print-on-failure.sh /work/abc-ci-builds/build-debug/test/log/bitcoin-wallet_tests.log /work/abc-ci-builds/build-debug/src/test/test_bitcoin --run_test=wallet_tests --logger=HRF,message:JUNIT,message,bitcoin-wallet_tests.xml --catch_system_errors=no
pure virtual method called
terminate called without an active exception
Aborted (core dumped)
[448/453] Running secp256k1 test suite
PASSED: secp256k1 test suite
[450/453] Running utility command for check-bitcoin-coins_tests
ninja: build stopped: cannot make progress due to previous errors.
Build build-debug failed with exit code 1

Rebase on D9649 to get more factorized code

This revision is now accepted and ready to land.Jun 8 2021, 15:19