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

proposal: introduce an e2e test suite #1124

Closed
alexanderbez opened this issue Mar 22, 2022 · 5 comments · Fixed by #1193
Closed

proposal: introduce an e2e test suite #1124

alexanderbez opened this issue Mar 22, 2022 · 5 comments · Fixed by #1193
Assignees
Labels

Comments

@alexanderbez
Copy link
Contributor

alexanderbez commented Mar 22, 2022

I propose we introduce a programatic e2e integration test suite that allows us to test functionality from the most basic operations like IBC transfers and native sends, to AMM operations.

Examples of this can be found here:

If there is consensus on this, I'll be happy to tackle it or guide others on the approach used :)

@ValarDragon
Copy link
Member

This sounds awesome! Its something @nikever @czarcas7ic and @daniel-farina have been looking into

I think we should be really happy to use something that already exists! Is there some docs for how these end to end tests work? (Like how end-to-end are they? Is it starting a single node testnet, and then posting txs & running queries to it?)

@alexanderbez
Copy link
Contributor Author

alexanderbez commented Mar 22, 2022

Yeah, so in each implementation I've done, there is a doc.go that outlines what's going on, but I never spent that much time on that file/doc, but we totally could make it very elaborate.

The basic gist of it is:

  • Setup data and files (in tmp dirs, e.g. genesis)
  • Build and/or pull docker images (via dockertest)
  • Run containers (via dockertest)
  • Write your tests in Go! You have access to all gRPC, API and RPC endpoints.

@ValarDragon
Copy link
Member

Ah sweet, just checked the file! This sounds awesome, it even does IBC tests. I think we should totally use this

@daniel-farina
Copy link
Contributor

@czarcas7ic to investigate further with @alexanderbez.

@p0mvn
Copy link
Member

p0mvn commented Apr 4, 2022

Just FYI, as per the DevOps call, @czarcas7ic and I are looking into this and will seek help from the rest of the team if we get blocked

Planning to spike the barebones implementation here: https://github.com/p0mvn/osmosis so that we don't pollute the main repository pipeline with test runs

To start, the plan is to be able to run one chain with 2 validators (similar to gaia). The second chain and ibc can be done in a separate PR to limit the scope

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants