foundry icon indicating copy to clipboard operation
foundry copied to clipboard

Anvil API test - can_remove_pool_transactions fails on master

Open mw2000 opened this issue 1 year ago • 3 comments

Component

Anvil

Have you ensured that all of these are up to date?

  • [X] Foundry
  • [X] Foundryup

What version of Foundry are you on?

No response

What command(s) is the bug in?

cargo test anvil

Operating System

macOS (Apple Silicon)

Describe the bug

The anvil API test can_remove_pool_transactions is failing on the master branch. This is testing whether there is an initial pending transaction present.

mw2000 avatar Apr 12 '24 06:04 mw2000

cc @yash-atreya

mattsse avatar Apr 12 '24 10:04 mattsse

@mw2000 I'm unable to reproduce this, it's working for me on the latest commit bdc04c278f8. Anyways, we're migrating this code to Alloy currently.

yash-atreya avatar Apr 12 '24 15:04 yash-atreya

@yash-atreya just double checked commits and it seems to still be failing on the bdc04c2 commit for me. No worries if not able to reproduce and have future plans to migrate to alloy.

For more context, it fails at:

https://github.com/foundry-rs/foundry/blob/master/crates/anvil/tests/it/anvil_api.rs#L652

The error I get on my end:

---- anvil_api::can_remove_pool_transactions stdout ----
thread 'anvil_api::can_remove_pool_transactions' panicked at crates/anvil/tests/it/anvil_api.rs:652:5:
assertion `left == right` failed
  left: 0
 right: 1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace


failures:
    anvil_api::can_remove_pool_transactions

test result: FAILED. 27 passed; 1 failed; 0 ignored; 0 measured; 146 filtered out; finished in 4.52s

mw2000 avatar Apr 12 '24 16:04 mw2000

Marking as resolved as these tests have been passing since the migration to Alloy

Feel free to re-open if you are still experiencing issues with this @mw2000

zerosnacks avatar Jul 15 '24 11:07 zerosnacks