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

Make test test_local_deployment use chain fork instead of the real chain #353

Open
gabrielfior opened this issue Aug 20, 2024 · 0 comments
Labels
bug Something isn't working high priority

Comments

@gabrielfior
Copy link
Contributor

gabrielfior commented Aug 20, 2024

The test test_local_deployment uses Gnosis Mainnet in its attempt to run the agent - basically it triggers agent.run (without placing bets), but it does try to redeem positions.
This is interfering with the operation of the deployed agent - see errors e.g. here
https://github.com/gnosis/prediction-market-agent-tooling/actions/runs/10479070131/job/29023879784#step:7:262
and here
https://github.com/gnosis/prediction-market-agent-tooling/actions/runs/10478961453/job/29023515375#step:7:223

Note that the position_id as well as the index on the logs increases, thus if you re-try long enough the tests will eventually pass.

The test would be more robust if executed on a local fork (using the existing pytest fixture local_web3), since all positions exist there as well.

@gabrielfior gabrielfior added the bug Something isn't working label Aug 20, 2024
@gabrielfior gabrielfior changed the title Make test test_local_deployment use chain fork instead of the real chain Make test test_local_deployment use chain fork instead of the real chain + change retry methods Aug 21, 2024
@gabrielfior gabrielfior changed the title Make test test_local_deployment use chain fork instead of the real chain + change retry methods Make test test_local_deployment use chain fork instead of the real chain + change retry conditions when calling web3py Aug 21, 2024
@gabrielfior gabrielfior changed the title Make test test_local_deployment use chain fork instead of the real chain + change retry conditions when calling web3py Make test test_local_deployment use chain fork instead of the real chain Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high priority
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

1 participant