Remove fmt.sprint() usage (cosmos#5403)

* remove fmt.sprint() usuage * Update test code. --------- Co-authored-by: DimitrisJim <d.f.hilliard@gmail.com> Co-authored-by: Carlos Rodriguez <carlos@interchain.io>
notional-labs · Dec 20, 2023 · cea0a82 · cea0a82
1 parent 335e4ff
commit cea0a82
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/modules/apps/27-interchain-accounts/types/port.go b/modules/apps/27-interchain-accounts/types/port.go
@@ -1,7 +1,6 @@
 package types
 
 import (
-	"fmt"
 	"strings"
 
 	errorsmod "cosmossdk.io/errors"
@@ -13,5 +12,6 @@ func NewControllerPortID(owner string) (string, error) {
 		return "", errorsmod.Wrap(ErrInvalidAccountAddress, "owner address cannot be empty")
 	}
 
-	return fmt.Sprint(ControllerPortPrefix, owner), nil
+	ownerWithPrefix := ControllerPortPrefix + owner
+	return ownerWithPrefix, nil
 }
diff --git a/modules/apps/27-interchain-accounts/types/port_test.go b/modules/apps/27-interchain-accounts/types/port_test.go
@@ -1,8 +1,6 @@
 package types_test
 
 import (
-	"fmt"
-
 	"github.com/cosmos/ibc-go/v8/modules/apps/27-interchain-accounts/types"
 	ibctesting "github.com/cosmos/ibc-go/v8/testing"
 )
@@ -22,7 +20,7 @@ func (suite *TypesTestSuite) TestNewControllerPortID() {
 		{
 			"success",
 			func() {},
-			fmt.Sprint(types.ControllerPortPrefix, TestOwnerAddress),
+			types.ControllerPortPrefix + TestOwnerAddress,
 			true,
 		},
 		{