ICA Code Coverage

[seantking]

There code coverage for tests is low & needs to improve.

Unit tests should be written for :

• https://github.com/cosmos/interchain-accounts/tree/master/x/ibc-account/types
• https://github.com/cosmos/interchain-accounts/tree/master/x/ibc-account/keeper
• https://github.com/cosmos/interchain-accounts/tree/master/x/ibc-account/client/cli

Integration tests should be written for:

• https://github.com/cosmos/interchain-accounts/tree/master/x/ibc-account/keeper

[colin-axner]

Within types files which need tests:

• account.go
• genesis.go (needs basic validation logic as well)
• hook.go
• packet.go

Within keeper files which need tests:

• account.go
• grpc_query.go
• handshake.go
• keeper.go
• relay.go

Top level files which need tests:

• module.go (ibc callbacks)
• genesis.go

CLI tests are difficult to do for IBC due to the structure of in-process app testing. CLI tests are very low priority