Add unit testing for the CompressScript functions
ClosedPublic
Actions

Authored by PiRK on Oct 30 2020, 13:26.

Details

Reviewers

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rABCc6b104465b0b: Add unit testing for the CompressScript functions

Summary

This is a backport of Core PR17220

Test Plan

ninja && ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Oct 30 2020, 13:26

Owners added a reviewer: Restricted Owners Package.Oct 30 2020, 13:26

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 30 2020, 13:26

PiRK requested review of this revision.Oct 30 2020, 13:26

teamcity edited the summary of this revision. (Show Details)Oct 30 2020, 13:26

Harbormaster completed remote builds in B13765: Diff 25269.Oct 30 2020, 13:48

Build Bitcoin ABC Diffs / Diff Testing (build-diff) failed.

Tail of the build log:

[0m[0;32mwallet_create_tx.py                              | ✓ Passed  | 10 s
[0m[0;32mwallet_createwallet.py                           | ✓ Passed  | 3 s
[0m[0;32mwallet_createwallet.py --usecli                  | ✓ Passed  | 2 s
[0m[0;32mwallet_disable.py                                | ✓ Passed  | 1 s
[0m[0;32mwallet_dump.py                                   | ✓ Passed  | 4 s
[0m[0;32mwallet_encryption.py                             | ✓ Passed  | 6 s
[0m[0;32mwallet_hd.py                                     | ✓ Passed  | 6 s
[0m[0;32mwallet_import_rescan.py                          | ✓ Passed  | 9 s
[0m[0;32mwallet_import_with_label.py                      | ✓ Passed  | 1 s
[0m[0;32mwallet_importmulti.py                            | ✓ Passed  | 4 s
[0m[0;32mwallet_importprunedfunds.py                      | ✓ Passed  | 2 s
[0m[0;32mwallet_keypool.py                                | ✓ Passed  | 3 s
[0m[0;32mwallet_keypool_topup.py                          | ✓ Passed  | 4 s
[0m[0;32mwallet_labels.py                                 | ✓ Passed  | 3 s
[0m[0;32mwallet_listreceivedby.py                         | ✓ Passed  | 20 s
[0m[0;32mwallet_listsinceblock.py                         | ✓ Passed  | 3 s
[0m[0;32mwallet_listtransactions.py                       | ✓ Passed  | 19 s
[0m[0;32mwallet_multiwallet.py                            | ✓ Passed  | 48 s
[0m[0;32mwallet_multiwallet.py --usecli                   | ✓ Passed  | 18 s
[0m[0;32mwallet_reorgsrestore.py                          | ✓ Passed  | 6 s
[0m[0;32mwallet_resendwallettransactions.py               | ✓ Passed  | 26 s
[0m[0;32mwallet_txn_clone.py                              | ✓ Passed  | 3 s
[0m[0;32mwallet_txn_clone.py --mineblock                  | ✓ Passed  | 5 s
[0m[0;32mwallet_txn_doublespend.py                        | ✓ Passed  | 2 s
[0m[0;32mwallet_txn_doublespend.py --mineblock            | ✓ Passed  | 5 s
[0m[0;32mwallet_watchonly.py                              | ✓ Passed  | 1 s
[0m[0;32mwallet_watchonly.py --usecli                     | ✓ Passed  | 1 s
[0m[0;32mwallet_zapwallettxes.py                          | ✓ Passed  | 2 s
[0m[1m
ALL                                              | ✓ Passed  | 859 s (accumulated) 
[0mRuntime: 172 s

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

OK
[258/737] Running seeder test suite
PASSED: seeder test suite
[274/737] Running pow test suite
PASSED: pow test suite
[279/737] Building CXX object src/test/CMakeFiles/test_bitcoin.dir/script_tests.cpp.o
In file included from /usr/include/boost/test/unit_test.hpp:19,
                 from ../../src/test/script_tests.cpp:30:
../../src/test/script_tests.cpp: In member function ‘void script_tests::script_build::test_method()’:
../../src/test/script_tests.cpp:541:22: note: variable tracking size limit exceeded with -fvar-tracking-assignments, retrying without
 BOOST_AUTO_TEST_CASE(script_build) {
                      ^~~~~~~~~~~~
[708/737] bitcoin: testing coinselector_tests
FAILED: src/test/CMakeFiles/check-bitcoin-coinselector_tests 
cd /work/abc-ci-builds/build-diff/src/test && /usr/bin/cmake -E make_directory /work/abc-ci-builds/build-diff/test/junit && /usr/bin/cmake -E make_directory /work/abc-ci-builds/build-diff/test/log && /usr/bin/cmake -E env /work/cmake/utils/log-and-print-on-failure.sh /work/abc-ci-builds/build-diff/test/log/bitcoin-coinselector_tests.log /work/abc-ci-builds/build-diff/src/test/test_bitcoin --run_test=coinselector_tests --logger=HRF,test_suite:JUNIT,message,bitcoin-coinselector_tests.xml --catch_system_errors=no
Running 4 test cases...
Entering test module "Bitcoin ABC unit tests"
../../src/wallet/test/coinselector_tests.cpp(22): Entering test suite "coinselector_tests"
../../src/wallet/test/coinselector_tests.cpp(149): Entering test case "bnb_search_test"
Segmentation fault (core dumped)
[720/737] bitcoin-upgrade-activated: testing coins_tests
ninja: build stopped: subcommand failed.
Build build-diff failed with exit code 1

@bot build-diff

teamcity awarded a token.Oct 30 2020, 16:29

Fabien accepted this revision.Oct 30 2020, 16:32

This revision is now accepted and ready to land.Oct 30 2020, 16:32

rebase

Harbormaster completed remote builds in B13776: Diff 25281.Oct 30 2020, 18:03

Closed by commit rABCc6b104465b0b: Add unit testing for the CompressScript functions (authored by marcaiaf <mmachicao@m19r.de>, committed by PiRK). · Explain WhyOct 30 2020, 19:31

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCc6b104465b0b: Add unit testing for the CompressScript functions.