Tests should pass on `Wormchain-on-CosmosSDK-v47`

jonathanpberger commented 3 months ago

### Tasks
- [x] Pass Current Unit Tests
- [x] ICT: Upgrade to `v0.47`
- [x] import steven's ICT into `v47`
- [x] ICT: Tx, Cosm Wasm, ICT, Sub Messages
- [x] Unit Test: `Wasmd` Ante Handler
- [x] ICT: `Wasmd` Ante Handler
- [x] Unit Test: Wormhole Module Proposals
- [x] ICT: Wormhole Module Proposals
- [x] staking module should pass
- [x] All CI Pipelines should pass

joelsmith-2019 commented 2 months ago

All tests passing! Before closing this issue, I am doing a small investigation on how updating the guardian set affects actively running validators. I may spin up a test to validate this as well.

joelsmith-2019 commented 1 month ago

Currently trying to get the Tilt tests to pass. Made tons of progress the past couple weeks by updating the TypeScript SDK, the contracts tools TS project, the guardian node, outdated Cosmos message logic, etc.

Working through some out of gas fees with the ibc-relayer. Playing with different gas adjustment levels to see if that gets the job done.

joelsmith-2019 commented 3 weeks ago

Figured out why we keep getting batch transactions errors. In the Cosmos SDK, they removed a broadcast mode, block, where you could submit a transaction and you won't get a response until the transaction is committed in a block. This mode was deprecated in v0.46, so we now have to switch to a new mode called sync. I was under the impression that they were synonymous and the cosmos team was simply trying to switch to a different naming convention. I was unfortunately wrong, the new sync mode simply waits for a check tx execution response only. So it doesn't actually wait for it to be committed in a block. Hence, we keep running into batch tx not found, because the tx literally does not exist on chain yet.

I am working on implementing a blocking call to check for a transaction that has been included in a block.

strangelove-ventures / wormhole

Tests should pass on `Wormchain-on-CosmosSDK-v47` #64