chore: unnecessary use of fmt.Sprintf

Signed-off-by: guoguangwu <[email protected]>
hyperledger · Jul 25, 2023 · f628d3c · f628d3c
1 parent 7af45ce
commit f628d3c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 8 deletions.
diff --git a/pkg/client/common/selection/dynamicselection/pgresolver/pgresolverimpl.go b/pkg/client/common/selection/dynamicselection/pgresolver/pgresolverimpl.go
@@ -81,10 +81,10 @@ func (c *peerGroupResolver) Resolve(peers []fab.Peer) (PeerGroup, error) {
 			for i, grp := range peerGroups {
 				s += fmt.Sprintf("%d - %+v", i, grp)
 				if i+1 < len(peerGroups) {
-					s += fmt.Sprintf(" OR ")
+					s += " OR "
 				}
 			}
-			s += fmt.Sprintf(" ")
+			s += " "
 		}
 		logger.Debugf(s)
 	}
@@ -107,10 +107,10 @@ func (c *peerGroupResolver) getPeerGroups(peerRetriever MSPPeerRetriever) ([]Pee
 		for i, g := range mspGroups {
 			s += fmt.Sprintf("%+v", g)
 			if i+1 < len(mspGroups) {
-				s += fmt.Sprintf("  OR ")
+				s += "  OR "
 			}
 		}
-		s += fmt.Sprintf(" ")
+		s += " "
 		logger.Debugf(s)
 	}
 

diff --git a/pkg/common/errors/multi/multi.go b/pkg/common/errors/multi/multi.go
@@ -11,7 +11,6 @@ SPDX-License-Identifier: Apache-2.0
 package multi
 
 import (
-	"fmt"
 	"strings"
 )
 
@@ -71,7 +70,7 @@ func (errs Errors) Error() string {
 		return errs[0].Error()
 	}
 
-	errors := []string{fmt.Sprint("Multiple errors occurred:")}
+	errors := []string{"Multiple errors occurred:"}
 	for _, err := range errs {
 		errors = append(errors, err.Error())
 	}

diff --git a/pkg/fab/resource/resource_test.go b/pkg/fab/resource/resource_test.go
@@ -8,7 +8,6 @@ package resource
 
 import (
 	"bytes"
-	"fmt"
 	"io/ioutil"
 	"path/filepath"
 	"testing"
@@ -59,7 +58,7 @@ func TestCreateChannel(t *testing.T) {
 
 	// Setup mock orderer
 	verifyBroadcast := make(chan *fab.SignedEnvelope)
-	orderer := mocks.NewMockOrderer(fmt.Sprintf("0.0.0.0:1234"), verifyBroadcast)
+	orderer := mocks.NewMockOrderer("0.0.0.0:1234", verifyBroadcast)
 
 	// Create channel without envelope
 	reqCtx, cancel := contextImpl.NewRequest(ctx, contextImpl.WithTimeout(10*time.Second))