Address todos

cosmos · Feb 6, 2025 · 6374d66 · 6374d66
1 parent 2926b5f
commit 6374d66
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 6 deletions.
diff --git a/tests/integration/changeover.go b/tests/integration/changeover.go
@@ -27,7 +27,7 @@ func (suite *CCVTestSuite) TestRecycleTransferChannel() {
 	// Create transfer channel manually
 	distrTransferMsg := channeltypes.NewMsgChannelOpenInit(
 		transfertypes.PortID,
-		transfertypes.V1, // TODO(wllmshao): check if this is correct
+		transfertypes.V1,
 		channeltypes.UNORDERED,
 		[]string{suite.path.EndpointA.ConnectionID},
 		transfertypes.PortID,

diff --git a/tests/integration/distribution.go b/tests/integration/distribution.go
@@ -690,7 +690,6 @@ func (s *CCVTestSuite) TestIBCTransferMiddleware() {
 			s.Require().NoError(err)
 
 			// execute middleware OnRecvPacket logic
-			// TODO(wllmshao): check if this version string is ok
 			ack := cbs.OnRecvPacket(s.providerCtx(), transfertypes.V1, packet, sdk.AccAddress{})
 
 			// compute expected rewards with provider denom

diff --git a/x/ccv/consumer/ibc_module.go b/x/ccv/consumer/ibc_module.go
@@ -162,7 +162,7 @@ func (am AppModule) OnChanOpenAck(
 
 	distrTransferMsg := channeltypes.NewMsgChannelOpenInit(
 		transfertypes.PortID,
-		transfertypes.V1, // TODO(wllmshao): check if this is correct
+		transfertypes.V1,
 		channeltypes.UNORDERED,
 		connHops,
 		transfertypes.PortID,

diff --git a/x/ccv/consumer/ibc_module_test.go b/x/ccv/consumer/ibc_module_test.go
@@ -220,7 +220,7 @@ func TestOnChanOpenAck(t *testing.T) {
 				// Expected msg
 				distrTransferMsg := channeltypes.NewMsgChannelOpenInit(
 					transfertypes.PortID,
-					transfertypes.V1, // TODO(wllmshao): check if this is correct
+					transfertypes.V1,
 					channeltypes.UNORDERED,
 					[]string{"connectionID"},
 					transfertypes.PortID,

diff --git a/x/ccv/consumer/keeper/genesis.go b/x/ccv/consumer/keeper/genesis.go
@@ -71,7 +71,7 @@ func (k Keeper) InitGenesis(ctx sdk.Context, state *types.GenesisState) []abci.V
 				panic(err)
 			}
 
-			// TODO(wllmshao): this forces the client to be tendermint, unsure if this is ok
+			// this means the client must be tendermint
 			cid, err := k.clientKeeper.CreateClient(ctx, ibchost.Tendermint, clientStateBytes, consensusStateBytes)
 			if err != nil {
 				// If the client creation fails, the chain MUST NOT start

diff --git a/x/ccv/provider/keeper/consumer_lifecycle.go b/x/ccv/provider/keeper/consumer_lifecycle.go
@@ -343,7 +343,7 @@ func (k Keeper) CreateConsumerClient(
 		return err
 	}
 
-	// TODO(wllmshao): this forces the client to be tendermint, unsure if this is ok
+	// this means the client must be tendermint
 	clientID, err := k.clientKeeper.CreateClient(ctx, ibchost.Tendermint, clientStateBytes, consensusStateBytes)
 	if err != nil {
 		return err