arkprotocol / templates

0 stars 0 forks source link

setup 2 chains for development #3

Closed taitruong closed 2 years ago

taitruong commented 2 years ago

objectives:

Todo:

taitruong commented 2 years ago

@tot-truong check #2 because in cw-ibc-demo repo it uses ts-relayer for integration tests. It also has scripts for starting and stopping scripts.

taitruong commented 2 years ago

Also check this repo out: https://github.com/confio/ibc-tests-ics20

Art3miX commented 2 years ago

This should be part of the #2 ibc template, so if we want to create a new ibc contract, we can use the template, and it will already have a test suite ready to write tests for that specific contract.

Ideally with a quick cargo commend (cargo integration-tests) that will run a script that handles ts-relayer and do the tests.

taitruong commented 2 years ago

I'd move at least start, stop scripts higher. Integration part makes sense for ibc template. By the way: this is answered via email reply ;).

⁣Get TypeApp for Android ​

On Sep 8, 2022, 00:00, at 00:00, Art3miX @.***> wrote:

This should be part of the #2 ibc template, so if we want to create a new ibc contract, we can use the template, and it will already have a test suite ready to write tests for that specific contract.

Ideally with a quick cargo commend (cargo integration-tests) that will run a script that handles ts-relayer and do the tests.

-- Reply to this email directly or view it on GitHub: https://github.com/arkprotocol/arkprotocol/issues/3#issuecomment-1239949355 You are receiving this because you authored the thread.

Message ID: @.***>

Art3miX commented 2 years ago

I'd move at least start, stop scripts higher. Integration part makes sense for ibc template. By the way: this is answered via email reply ;). ⁣Get TypeApp for Android ​ On Sep 8, 2022, 00:00, at 00:00, Art3miX @.> wrote: This should be part of the #2 ibc template, so if we want to create a new ibc contract, we can use the template, and it will already have a test suite ready to write tests for that specific contract. Ideally with a quick cargo commend (cargo integration-tests) that will run a script that handles ts-relayer and do the tests. -- Reply to this email directly or view it on GitHub: #3 (comment) You are receiving this because you authored the thread. Message ID: @.>

Well yea, those should be in the scripts folder at the root, because they are "global" to run a chain, nothing to do with specific contract.

Art3miX commented 2 years ago

check #10 PR, I added the 2 chains from the demo, i don't think we should waste time on setting our own chains atm. But we probably should add more test chains in the future.

I think we can close this issue once the PR merged.

Art3miX commented 2 years ago

We have a testing with 2 chains, should be closed?

tot-truong commented 2 years ago

We have a testing with 2 chains, should be closed?

Yes, I think with all the relayer and IBC stuff you've been doing this is outdated