Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

flaky/tests: full_client::frame::staking::tx_bond Transaction outdated #1240

Closed
lexnv opened this issue Nov 3, 2023 · 2 comments · Fixed by #1305
Closed

flaky/tests: full_client::frame::staking::tx_bond Transaction outdated #1240

lexnv opened this issue Nov 3, 2023 · 2 comments · Fixed by #1305
Assignees

Comments

@lexnv
Copy link
Collaborator

lexnv commented Nov 3, 2023

Detected by: https://github.com/paritytech/subxt/actions/runs/6745755385/job/18338194383

--- STDERR:              integration-tests full_client::frame::staking::tx_bond ---
thread 'full_client::frame::staking::tx_bond' panicked at testing/integration-tests/src/full_client/frame/staking.rs:216:5:
assertion failed: `Err(Transaction(Invalid("Invalid transaction: Transaction is outdated")))` does not match `Err(Error::Runtime(DispatchError::Module(err)))`
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
@niklasad1
Copy link
Member

niklasad1 commented Nov 3, 2023

Is it possible that same nonce is used twice for instance by another test?
We are still using a single substrate process right to run all integration test on right?

@jsdw
Copy link
Collaborator

jsdw commented Nov 3, 2023

We are still using a single substrate process right to run all integration test on right?

Every test should spawn a new substrate instance at the mo!

Looking at the test, I can't see anything that should be suspect here; two tx's are submitted but we wait_for_finalized_success on each of them. Maybe there's some race in the UnstableBackend where the finalized block that the second TX uses isn't as uptodate as it should be by the time we return from the first one? def worth looking into :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants