search

nervosnetwork / ckb-cli

CKB command line interface
MIT License
56 stars 35 forks source link

Integration test is not stable #553

Closed doitian closed 9 months ago

doitian commented 1 year ago

I saw a lot of integtation test failures like this:

[2023-05-10T03:48:48Z INFO  ckb_cli_test::miner] mine until tx: balance capacity error: `transaction dependency provider error: `invalid cell status: "unknown"``

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error("invalid format, expected a 0x-prefixed hex string with 64 digits", line: 1, column: 88)', src/miner.rs:124:79
stack backtrace:
liuck8080 commented 1 year ago

That's because cell collector in SDK will get consumed cell, which status is not live, cause last transfer command will return an error message invalid cell status: "unknown" instead of a valid tx_hash, (https://github.com/nervosnetwork/ckb-cli/blob/develop/test/src/spec/udt/mod.rs#L165 ) then try to confirm the tx_hash will be panic, since it's not a correct tx_hash.

liuck8080 commented 1 year ago

This PR also fix integration test: https://github.com/nervosnetwork/ckb-cli/pull/554