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

chore: Migrate e2e tests to work with sync-to-blockchain autopilot mode #2945

Closed
MartinquaXD opened this issue Sep 5, 2024 · 2 comments
Closed
Labels
E:6.1 One auction per block See https://github.com/cowprotocol/pm/issues/35 for details

Comments

@MartinquaXD
Copy link
Contributor

Background

Before we make sync-to-blockchain the one and only run mode we need to make sure that e2e tests still pass.
Currently many test rely on the fact that new auctions get built periodically. When we switch to the synchronized mode this will only happen whenever we mint a new block.

Also the new run mode gives us a few more guarantees on the ordering on some events so we can revisit some tests to see if they can become stricter / more meaningful. One example was discussed here.

Acceptance criteria

e2e tests pass without flakiness when using sync-to-blockchain everywhere.

@MartinquaXD MartinquaXD added the E:6.1 One auction per block See https://github.com/cowprotocol/pm/issues/35 for details label Sep 5, 2024
Copy link

github-actions bot commented Nov 5, 2024

This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed.

@github-actions github-actions bot added the stale label Nov 5, 2024
@m-lord-renkse
Copy link
Contributor

#3040

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E:6.1 One auction per block See https://github.com/cowprotocol/pm/issues/35 for details
Projects
None yet
Development

No branches or pull requests

2 participants