Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve error reporting from VerifyAmounts #1254

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Improve error reporting from VerifyAmounts

b14e739
Select commit
Loading
Failed to load commit list.
Open

Improve error reporting from VerifyAmounts #1254

Improve error reporting from VerifyAmounts
b14e739
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Cirrus CI / Win64 [unit tests, no gui tests, no boost::process, no functional tests] [focal] failed Aug 1, 2023 in 12m 9s

Task Summary

Instruction ci failed in 11:05

Details

✅ 00:28 clone
✅ 00:27 merge_base
✅ 00:03 ccache
✅ 00:01 depends_built
❌ 11:05 ci

test/blind_tests.cpp:366:21: note:   no known conversion for argument 1 from ‘const string’ {aka ‘const std::__cxx11::basic_string<char>’} to ‘bool’
In file included from ./primitives/confidential.h:6,
                 from ./primitives/transaction.h:14,
                 from ./blind.h:10,
                 from test/blind_tests.cpp:6:
./asset.h:91:6: note: candidate: ‘bool operator!(const CAmountMap&)’
   91 | bool operator!(const CAmountMap& a); // Check if all values are 0
      |      ^~~~~~~~
./asset.h:91:34: note:   no known conversion for argument 1 from ‘const string’ {aka ‘const std::__cxx11::basic_string<char>’} to ‘const CAmountMap&’ {aka ‘const std::map<CAsset, long long int>&’}
   91 | bool operator!(const CAmountMap& a); // Check if all values are 0
      |                ~~~~~~~~~~~~~~~~~~^
In file included from /tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/test_tools.hpp:45,
                 from /tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/unit_test.hpp:18,
                 from test/blind_tests.cpp:13:
test/blind_tests.cpp:368:21: error: no match for ‘operator!’ (operand type is ‘const string’ {aka ‘const std::__cxx11::basic_string<char>’})
  368 |         BOOST_CHECK(!VerifyAmounts(inputs, CTransaction(txtemp), nullptr, false));
      |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
test/blind_tests.cpp:368:21: note: candidate: ‘operator!(bool)’ <built-in>
test/blind_tests.cpp:368:21: note:   no known conversion for argument 1 from ‘const string’ {aka ‘const std::__cxx11::basic_string<char>’} to ‘bool’
In file included from ./primitives/confidential.h:6,
                 from ./primitives/transaction.h:14,
                 from ./blind.h:10,
                 from test/blind_tests.cpp:6:
./asset.h:91:6: note: candidate: ‘bool operator!(const CAmountMap&)’
   91 | bool operator!(const CAmountMap& a); // Check if all values are 0
      |      ^~~~~~~~
./asset.h:91:34: note:   no known conversion for argument 1 from ‘const string’ {aka ‘const std::__cxx11::basic_string<char>’} to ‘const CAmountMap&’ {aka ‘const std::map<CAsset, long long int>&’}
   91 | bool operator!(const CAmountMap& a); // Check if all values are 0
      |                ~~~~~~~~~~~~~~~~~~^
In file included from /tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/tools/old/impl.hpp:20,
                 from /tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/test_tools.hpp:46,
                 from /tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/unit_test.hpp:18,
                 from test/blind_tests.cpp:13:
/tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/tools/assertion_result.hpp: In instantiation of ‘boost::test_tools::assertion_result::assertion_result(const BoolConvertable&) [with BoolConvertable = std::__cxx11::basic_string<char>]’:
test/blind_tests.cpp:75:9:   required from here
/tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/tools/assertion_result.hpp:57:74: error: no match for ‘operator!’ (operand type is ‘const std::__cxx11::basic_string<char>’)
   57 |     assertion_result( BoolConvertable const& pv_ ) : p_predicate_value( !!pv_ ) {}
      |                                                                          ^~~~
/tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/tools/assertion_result.hpp:57:74: note: candidate: ‘operator!(bool)’ <built-in>
/tmp/cirrus-ci-build/depends/x86_64-w64-mingw32/include/boost/test/tools/assertion_result.hpp:57:74: note:   no known conversion for argument 1 from ‘const std::__cxx11::basic_string<char>’ to ‘bool’
In file included from ./primitives/confidential.h:6,
                 from ./primitives/transaction.h:14,
                 from ./blind.h:10,
                 from test/blind_tests.cpp:6:
./asset.h:91:6: note: candidate: ‘bool operator!(const CAmountMap&)’
   91 | bool operator!(const CAmountMap& a); // Check if all values are 0
      |      ^~~~~~~~
./asset.h:91:34: note:   no known conversion for argument 1 from ‘const std::__cxx11::basic_string<char>’ to ‘const CAmountMap&’ {aka ‘const std::map<CAsset, long long int>&’}
   91 | bool operator!(const CAmountMap& a); // Check if all values are 0
      |                ~~~~~~~~~~~~~~~~~~^
make[2]: *** [Makefile:18296: test/test_bitcoin-blind_tests.o] Error 1
make[2]: Leaving directory '/tmp/cirrus-ci-build/ci/scratch/build/elements-x86_64-w64-mingw32/src'
make[1]: *** [Makefile:18578: all-recursive] Error 1
make[1]: Leaving directory '/tmp/cirrus-ci-build/ci/scratch/build/elements-x86_64-w64-mingw32/src'
make: *** [Makefile:829: all-recursive] Error 1
������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������