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

Interchain Tests #163

Merged
merged 27 commits into from
Jun 25, 2023
Merged

Interchain Tests #163

merged 27 commits into from
Jun 25, 2023

Conversation

anhductn2001
Copy link
Contributor

@anhductn2001 anhductn2001 commented May 20, 2023

Interchaintest orchestrates Go tests that utilize Docker containers for multiple IBC-compatible blockchains.
It allows users to quickly spin up custom testnets and dev environments to test IBC, chain infrastructures, smart contracts, etc.

  1. We will build a Migaloo image and use it for testing.
  2. Setup interchain (ChainConfig, OverrideConfig, numberOfVal, Name of Chain, ...)
  3. Write Testcase (StartChain, PushWasmCode, IBC Transfers, ...)

@@ -0,0 +1,32 @@
package interchaintest

Check warning

Code scanning / Revive (reported by Codacy)

should have a package comment

should have a package comment
)

var (
MigalooMainRepo = "ghcr.io/white-whale-defi-platform/migaloo-chain"

Check warning

Code scanning / Revive (reported by Codacy)

exported var MigalooMainRepo should have comment or be unexported

exported var MigalooMainRepo should have comment or be unexported
@faddat
Copy link
Collaborator

faddat commented May 20, 2023

Thanks for taking this issue! Please let me know if you need any help

@faddat
Copy link
Collaborator

faddat commented May 22, 2023

@anhductn2001 can you please put a more detailed description on this pr? Thanks!

@anhductn2001
Copy link
Contributor Author

Yes, I will. Sure!! And I also add document for this when it finished.

@anhductn2001
Copy link
Contributor Author

@faddat
Copy link
Collaborator

faddat commented May 27, 2023

yes I can, thanks for asking @anhductn2001

@faddat
Copy link
Collaborator

faddat commented May 27, 2023

@anhductn2001 sir here's your image:

https://github.com/White-Whale-Defi-Platform/migaloo-chain/pkgs/container/migaloo-chain-ictest

use the ghcr path you can find there :)

@anhductn2001
Copy link
Contributor Author

Please public it @faddat. The status is private now. TY!

@faddat
Copy link
Collaborator

faddat commented May 27, 2023

ahhh gotcha. 1 sec

@faddat
Copy link
Collaborator

faddat commented May 27, 2023

I think you need to ask @Sen-Com in slack sir

@anhductn2001
Copy link
Contributor Author

Yea, I will

@faddat
Copy link
Collaborator

faddat commented Jun 9, 2023

hey I think I am going to pull the docker image out of here, then merge it to main...

.gitignore Outdated
@@ -3,4 +3,5 @@ bin
.ash_history



go.work
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we will commit go.work but not go.sum

go.mod Outdated Show resolved Hide resolved
@faddat faddat mentioned this pull request Jun 9, 2023
21 tasks
@anhductn2001
Copy link
Contributor Author

hey I think I am going to pull the docker image out of here, then merge it to main...

Hmm, I am thinking about that. There seems to be no response from someone who has the permission to manage ghcr package yet

@faddat
Copy link
Collaborator

faddat commented Jun 15, 2023

lets do the go workspace. I like them

@anhductn2001
Copy link
Contributor Author

Workspace should be add manual in local env, sir. We dont need add go.work on repo, it should a go.work.example

@faddat
Copy link
Collaborator

faddat commented Jun 15, 2023

do like this:

https://github.com/osmosis-labs/osmosis/blob/main/go.work

commit go.work but don't commit go.work.sum

@anhductn2001
Copy link
Contributor Author

Finally, I have done with #179 @faddat

@anhductn2001
Copy link
Contributor Author

anhductn2001 commented Jun 18, 2023

do like this:

https://github.com/osmosis-labs/osmosis/blob/main/go.work

commit go.work but don't commit go.work.sum

We should temporarily remove it @faddat , because it causes errors when building binary. This is related to the dep version in interchaintest
I will fix it and add again soon

@anhductn2001 anhductn2001 marked this pull request as ready for review June 19, 2023 07:15
@anhductn2001
Copy link
Contributor Author

Can you review this @faddat?

@faddat
Copy link
Collaborator

faddat commented Jun 25, 2023

Yep, please call me next time you need a review if it doesn't happen promptly :)

@faddat faddat enabled auto-merge June 25, 2023 05:21
Copy link
Collaborator

@faddat faddat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like a basic implementation we will want to add upgrading testing, but -- great start!

@faddat faddat requested review from ThanhNhann and GNaD13 June 25, 2023 05:22
@faddat faddat disabled auto-merge June 25, 2023 05:39
@faddat faddat merged commit 4a1cbbf into release/v3.0.x Jun 25, 2023
@Sen-Com Sen-Com deleted the ducnt/ictest branch June 3, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants