search

tari-project / tari

The Tari protocol
https://tari.com
BSD 3-Clause "New" or "Revised" License
339 stars 210 forks source link

chore: fix integration tests #6378

Closed SWvheerden closed 1 month ago

SWvheerden commented 1 month ago

Description

fixes integration tests due to regression bug

github-actions[bot] commented 1 month ago

Test Results (CI)

    3 files    120 suites   43m 7s :stopwatch: 1 295 tests 1 295 :white_check_mark: 0 :zzz: 0 :x: 3 877 runs  3 877 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 2d4a9b24.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

Test Results (Integration tests)

 2 files  + 2  11 suites  +11   15m 45s :stopwatch: + 15m 45s 35 tests +35  34 :white_check_mark: +34  0 :zzz: ±0  1 :x: +1  36 runs  +36  35 :white_check_mark: +35  0 :zzz: ±0  1 :x: +1 

For more details on these failures, see this check.

Results for commit 2d4a9b24. ± Comparison against base commit 43b2317e.

:recycle: This comment has been updated with latest results.

hansieodendaal commented 1 month ago

This test still fails:

All 3 runs failed: Scenario: Verify gprc cna create block with more than 1 coinbase:
Step panicked. Captured output: called `Result::unwrap()` on an `Err` value: InvalidSize
  Scenario: Verify gprc cna create block with more than 1 coinbase
   ✔  Given I have a seed node SEED_A
   ✔  When I have 1 base nodes connected to all seed nodes
   ✘  Then generate a block with 2 coinbases from node SEED_A
SWvheerden commented 1 month ago

This test still fails:

All 3 runs failed: Scenario: Verify gprc cna create block with more than 1 coinbase:
Step panicked. Captured output: called `Result::unwrap()` on an `Err` value: InvalidSize
  Scenario: Verify gprc cna create block with more than 1 coinbase
   ✔  Given I have a seed node SEED_A
   ✔  When I have 1 base nodes connected to all seed nodes
   ✘  Then generate a block with 2 coinbases from node SEED_A

A bit jumping the gun here, but its fixed now😀