unit test rpc.account_representative_set_epoch_2 failed in CI

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.48k stars 787 forks source link

unit test rpc.account_representative_set_epoch_2 failed in CI #3770

Closed dsiganos closed 2 years ago

dsiganos commented 2 years ago

 [ RUN      ] rpc.account_representative_set_epoch_2
/workspace/nano/rpc_test/rpc.cpp:2739: Failure
Expected equality of these values:
  1
  response.count ("error")
    Which is: 0
[  FAILED  ] rpc.account_representative_set_epoch_2 (596 ms)

https://github.com/nanocurrency/nano-node/runs/5735186737?check_suite_focus=true#step:6:2030

dsiganos commented 2 years ago

This has at least 2 problems:

one problem is that the unit test does not wait until the epoch blocks are cemented
but even after fixing that problem, the test still fails on rare occasions