chore: unnecessary use of fmt.Sprintf

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)
 	}
 

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())
 	}

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))