nanocurrency / nano-node

Nano is digital currency. Its ticker is: XNO and its currency symbol is: Ӿ
https://nano.org
BSD 3-Clause "New" or "Revised" License
3.49k stars 787 forks source link

qt_test is failing on develop #3421

Closed dsiganos closed 3 years ago

dsiganos commented 3 years ago

Summary

qt_test fails on latest develop (commit: a5b9c3f51e3da14cc6275400472f2573c9c78fde)

ds@server:/fast/branches/develop2/build$ ./qt_test 
[==========] Running 26 tests from 3 test suites.
[----------] Global test environment set-up.
[----------] 23 tests from wallet
[ RUN      ] wallet.construction
Assertion (selected_ratio_button != nullptr) failed
nano_qt::advanced_actions::advanced_actions(nano_qt::wallet&)
/fast/branches/develop2/nano-node/nano/qt/qt.cpp:1841

 0# 0x000055C561EA69C4 in ./qt_test
 1# 0x000055C561EA6F78 in ./qt_test
 2# 0x000055C5618679D5 in ./qt_test
 3# 0x000055C56185D196 in ./qt_test
 4# 0x000055C5612C47A8 in ./qt_test
 5# 0x000055C5612C2E51 in ./qt_test
 6# 0x000055C5612C1161 in ./qt_test
 7# 0x000055C5612BE917 in ./qt_test
 8# 0x000055C5612BCDB2 in ./qt_test
 9# 0x000055C5612B9C03 in ./qt_test
10# 0x000055C5612B6902 in ./qt_test
11# 0x000055C5612B3FFE in ./qt_test
12# 0x000055C56128D766 in ./qt_test
13# 0x000055C5618400E4 in ./qt_test
14# 0x000055C561838C1F in ./qt_test
15# 0x000055C56180DE10 in ./qt_test
16# 0x000055C56180E852 in ./qt_test
17# 0x000055C56180F16C in ./qt_test
18# 0x000055C56181EC24 in ./qt_test
19# 0x000055C5618410A5 in ./qt_test
20# 0x000055C561839CB7 in ./qt_test
21# 0x000055C56181D33B in ./qt_test
22# 0x000055C56128704C in ./qt_test
23# 0x000055C561284432 in ./qt_test
24# __libc_start_main in /lib/x86_64-linux-gnu/libc.so.6
25# 0x000055C56128421E in ./qt_test

Aborted (core dumped)

Node version

a5b9c3f51e3da14cc6275400472f2573c9c78fde

Build details

manual build

OS and version

ubuntu 20.04

Steps to reproduce the behavior

run qt_test

Expected behavior

pass

Actual behavior

fails

Possible solution

No response

Supporting files

No response

zhyatt commented 3 years ago

@clemahieu Is this now resolved?

dsiganos commented 3 years ago

qt_test is passing now.

dsiganos commented 3 years ago

It was fixed by this commit: https://github.com/nanocurrency/nano-node/commit/db6c1624c9550b6ac887d8102bfe517b81a7b5c9