Skip to content

Commit

Permalink
Merge branch 'main' into jim/4967-test-get-timestamp-at-height
Browse files Browse the repository at this point in the history
  • Loading branch information
DimitrisJim authored Oct 30, 2023
2 parents b685667 + 7cf30ef commit 09e2a58
Show file tree
Hide file tree
Showing 9 changed files with 21 additions and 39 deletions.
4 changes: 2 additions & 2 deletions docs/tutorials/01-fee/01-intro.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@ This is a tutorial for wiring up the ICS-29 Fee Middleware to a Cosmos SDK block

<HighlightBox type="prerequisite" title="Prerequisites">

- Basic Knowledge of [Go](https://golang.org/doc/tutorial/getting-started)
- Basic Knowledge of [JavaScript](https://developer.mozilla.org/en-US/docs/Web/JavaScript)
- Basic Knowledge of [TypeScript](https://www.typescriptlang.org/)
- Basic Knowledge of Cosmos SDK
- If you are new to Cosmos SDK, we recommend you to go through the first two categories of the [Developer Portal](https://tutorials.cosmos.network/academy/1-what-is-cosmos/)
- Basic Knowledge of [the Fee Middleware module](https://ibc.cosmos.network/main/middleware/ics29-fee/overview)

</HighlightBox>

Expand Down
8 changes: 7 additions & 1 deletion docs/tutorials/01-fee/02-setup-env.md
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,13 @@ go version

## Hermes

Install Hermes relayer version `v1.6.0` via cargo following the instructions on the [Hermes website](https://hermes.informal.systems/quick-start/installation.html#install-via-cargo). Test if Hermes is installed by running.
Install Hermes relayer version `v1.6.0` via cargo following the instructions on the [Hermes website](https://hermes.informal.systems/quick-start/installation.html#install-via-cargo) or by using the command below.

```bash
cargo install ibc-relayer-cli --version 1.6.0 --bin hermes --locked
```

Test if Hermes is installed by running the following command:

```bash
hermes version
Expand Down
8 changes: 0 additions & 8 deletions docs/tutorials/01-fee/04-wire-feeibc-mod.md
Original file line number Diff line number Diff line change
Expand Up @@ -164,14 +164,6 @@ Next, we need to add the fee middleware to the `SetOrderBeginBlockers`, `SetOrde
consensusparamtypes.ModuleName,
// this line is used by starport scaffolding # stargate/app/endBlockers
)
app.mm.SetOrderEndBlockers(
// ... other modules
vestingtypes.ModuleName,
consensusparamtypes.ModuleName,
// plus-diff-line
+ ibcfeetypes.ModuleName,
// this line is used by starport scaffolding # stargate/app/endBlockers
)

// NOTE: The genutils module must occur after staking so that pools are
// properly initialized with tokens from genesis accounts.
Expand Down
16 changes: 5 additions & 11 deletions docs/tutorials/01-fee/07-test-app.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@ You can find the React app we created in the previous section [here](https://git
Ignite supports running multiple chains locally with different configs. The source chain will be called earth and the destination chain will be called mars.
Add the following config files to the root of the project:

```yaml reference title="earth.yaml"
```yaml reference title="earth.yml"
https://github.com/srdtrk/cosmoverse2023-ibc-fee-demo/blob/96cb63bf2e60b4613a89841416066551dd666c0d/earth.yml
```

```yaml reference title="mars.yaml"
```yaml reference title="mars.yml"
https://github.com/srdtrk/cosmoverse2023-ibc-fee-demo/blob/96cb63bf2e60b4613a89841416066551dd666c0d/mars.yml
```

Expand All @@ -50,7 +50,7 @@ ignite chain serve -c mars.yml --reset-once
We first need to create a relayer configuration file. Add the following file to the root of the project:

```toml reference title="hermes/config.toml"
https://github.com/srdtrk/cosmoverse2023-ibc-fee-demo/blob/960d8b7e148cbe2207c3a743bac7c0985a5b653a/hermes/config.toml
https://github.com/srdtrk/cosmoverse2023-ibc-fee-demo/blob/0186b9ee979c288efbe3fe5fd071169d9dbcf91e/hermes/config.toml
```

We can move this file to the `~/.hermes` directory to avoid having to specify the path to the config file every time we run the relayer:
Expand Down Expand Up @@ -105,15 +105,9 @@ hermes create channel --channel-version '{"fee_version":"ics29-1","app_version":
```

This will create an incentivized IBC transfer channel between the two chains with the channel id `channel-0`, and channel version `{"fee_version":"ics29-1","app_version":"ics20-1"}`.
Next recall that the Fee Middleware only pays fees on the source chain. That's why we should register `damian` and `charlie` as each other's counterparty on both chains.

```bash title="Terminal 4"
hermes fee register-counterparty-payee --chain mars --channel channel-0 --port transfer --counterparty-payee cosmos1vapwvcsr0m32ptal6z6g9hjctywrw4yzyf6y6v
```

```bash title="Terminal 4"
hermes fee register-counterparty-payee --chain earth --channel channel-0 --port transfer --counterparty-payee cosmos1uu38gkyed0dte5f9xk20p8wcppulsjt90s7f8h
```
Next recall that the Fee Middleware only pays fees on the source chain. That's why we should register `damian` and `charlie` as each other's counterparty on both chains.
Luckily, the relayer does this for us under the hood because we've enabled the `auto_register_counterparty_payee` option in the config file.

Now we can run the relayer with the following command:

Expand Down
3 changes: 3 additions & 0 deletions modules/apps/27-interchain-accounts/host/types/codec.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package types
import (
codectypes "github.com/cosmos/cosmos-sdk/codec/types"
sdk "github.com/cosmos/cosmos-sdk/types"
"github.com/cosmos/cosmos-sdk/types/msgservice"
)

// RegisterInterfaces registers the interchain accounts host message types using the provided InterfaceRegistry
Expand All @@ -11,4 +12,6 @@ func RegisterInterfaces(registry codectypes.InterfaceRegistry) {
(*sdk.Msg)(nil),
&MsgUpdateParams{},
)

msgservice.RegisterMsgServiceDesc(registry, &_Msg_serviceDesc)
}
2 changes: 1 addition & 1 deletion modules/capability/capability_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ type CapabilityTestSuite struct {
}

func (suite *CapabilityTestSuite) SetupTest() {
encodingCfg := moduletestutil.MakeTestEncodingConfig(capability.AppModuleBasic{})
encodingCfg := moduletestutil.MakeTestEncodingConfig(capability.AppModule{})
suite.cdc = encodingCfg.Codec

suite.storeKey = storetypes.NewKVStoreKey(types.StoreKey)
Expand Down
2 changes: 1 addition & 1 deletion modules/capability/keeper/keeper_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ func (suite *KeeperTestSuite) SetupTest() {
key := storetypes.NewKVStoreKey(types.StoreKey)
testCtx := testutil.DefaultContextWithDB(suite.T(), key, storetypes.NewTransientStoreKey("transient_test"))
suite.ctx = testCtx.Ctx
encCfg := moduletestutil.MakeTestEncodingConfig(capability.AppModuleBasic{})
encCfg := moduletestutil.MakeTestEncodingConfig(capability.AppModule{})
suite.keeper = keeper.NewKeeper(encCfg.Codec, key, key)
}

Expand Down
15 changes: 1 addition & 14 deletions modules/capability/module.go
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import (
"time"

gwruntime "github.com/grpc-ecosystem/grpc-gateway/runtime"
"github.com/spf13/cobra"

"cosmossdk.io/core/appmodule"

Expand All @@ -26,7 +25,6 @@ import (

var (
_ module.AppModule = (*AppModule)(nil)
_ module.AppModuleBasic = (*AppModuleBasic)(nil)
_ module.AppModuleSimulation = (*AppModule)(nil)
_ module.HasName = (*AppModule)(nil)
_ module.HasConsensusVersion = (*AppModule)(nil)
Expand All @@ -48,11 +46,6 @@ func NewAppModuleBasic(cdc codec.Codec) AppModuleBasic {
return AppModuleBasic{cdc: cdc}
}

// Name returns the capability module's name.
func (AppModuleBasic) Name() string {
return types.ModuleName
}

// RegisterLegacyAminoCodec does nothing. Capability does not support amino.
func (AppModuleBasic) RegisterLegacyAminoCodec(cdc *codec.LegacyAmino) {}

Expand All @@ -77,12 +70,6 @@ func (AppModuleBasic) ValidateGenesis(cdc codec.JSONCodec, config client.TxEncod
func (AppModuleBasic) RegisterGRPCGatewayRoutes(_ client.Context, _ *gwruntime.ServeMux) {
}

// GetTxCmd returns the capability module's root tx command.
func (AppModuleBasic) GetTxCmd() *cobra.Command { return nil }

// GetQueryCmd returns the capability module's root query command.
func (AppModuleBasic) GetQueryCmd() *cobra.Command { return nil }

// ----------------------------------------------------------------------------
// AppModule
// ----------------------------------------------------------------------------
Expand Down Expand Up @@ -112,7 +99,7 @@ func (AppModule) IsAppModule() {}

// Name returns the capability module's name.
func (am AppModule) Name() string {
return am.AppModuleBasic.Name()
return types.ModuleName
}

// InitGenesis performs the capability module's genesis initialization It returns
Expand Down
2 changes: 1 addition & 1 deletion modules/capability/simulation/decoder_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import (
)

func TestDecodeStore(t *testing.T) {
encodingCfg := moduletestutil.MakeTestEncodingConfig(capability.AppModuleBasic{})
encodingCfg := moduletestutil.MakeTestEncodingConfig(capability.AppModule{})
dec := simulation.NewDecodeStore(encodingCfg.Codec)

capOwners := types.CapabilityOwners{
Expand Down

0 comments on commit 09e2a58

Please sign in to comment.