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.

Sign up for GitHub

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

Jump to bottom

[E2E] identify more e2e tests specific for Osmosis and write up issues to add them #1202

Closed
2 tasks
Tracked by #1416
p0mvn opened this issue Apr 5, 2022 · 5 comments
Closed
2 tasks
Tracked by #1416

[E2E] identify more e2e tests specific for Osmosis and write up issues to add them #1202

p0mvn opened this issue Apr 5, 2022 · 5 comments
Labels
C:docs Improvements or additions to documentation T:task ⚙️ A task belongs to a story

Comments

@p0mvn
Copy link
Member

p0mvn commented Apr 5, 2022 •
edited
Loading

Background

We should start adding new tests once #1201 is complete with existing tests ported from other repositories. However, we can begin identifying new test cases that we would like to add in the future.

Acceptance Criteria

  • e2e test plan is created
  • new issues are written up for the next sprint.
@p0mvn p0mvn mentioned this issue Apr 5, 2022
Closed
3 tasks
@p0mvn p0mvn added T:task ⚙️ A task belongs to a story C:docs Improvements or additions to documentation labels Apr 5, 2022
@p0mvn p0mvn moved this to 🔍 Needs Review in Osmosis Chain Development Apr 5, 2022
@p0mvn
Copy link
Member Author

p0mvn commented Apr 5, 2022

I'm not sure how this could look like yet but it would be great to performance test and send thousands of queries to an e2e relayer to try and catch app hash in these tests

@alexanderbez
Copy link
Contributor

As long as we have beefy machines, we can definitely do this!

@p0mvn
Copy link
Member Author

p0mvn commented May 3, 2022 •
edited
Loading

Based on the problems experienced in: #1356

We should have tests for nodes with:

  • grpc enabled
    • query old height
    • query latest height
    • do not provide a height
  • grpc disabled
    • query old height
    • query latest height
    • do not provide a height

@p0mvn p0mvn changed the title identify more e2e tests specific for Osmosis and write up issues to add them [E2E] identify more e2e tests specific for Osmosis and write up issues to add them May 4, 2022
@p0mvn p0mvn mentioned this issue May 4, 2022
Closed
4 tasks
@mattverse
Copy link
Member

Hey @p0mvn do you think this issue can be closed since we have #1235 or would you define it a separate issue?

@czarcas7ic
Copy link
Member

I would say this issue can be closed, because at the very least we have #1605

@p0mvn p0mvn closed this as completed May 31, 2022
Repository owner moved this from Needs Review 🔍 to Done ✅ in Osmosis Chain Development May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C:docs Improvements or additions to documentation T:task ⚙️ A task belongs to a story
Projects
Osmosis Chain Development
Archived in project
Milestone
No milestone
Development

No branches or pull requests
4 participants
@alexanderbez @p0mvn @czarcas7ic @mattverse