Merge pull request #1342 from testwill/fmt

chore: unnecessary use of fmt.Sprintf
gruntwork-io · Sep 19, 2023 · f99fb38 · f99fb38
2 parents af9ab8c + 96109ad
commit f99fb38
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/modules/dns-helper/errors.go b/modules/dns-helper/errors.go
@@ -6,7 +6,7 @@ import "fmt"
 type NoResolversError struct{}
 
 func (err NoResolversError) Error() string {
-	return fmt.Sprintf("No resolvers set for DNSLookupE call")
+	return "No resolvers set for DNSLookupE call"
 }
 
 // QueryTypeError is an error that occurs if the DNS query type is not supported

diff --git a/modules/docker/inspect_test.go b/modules/docker/inspect_test.go
@@ -63,7 +63,7 @@ func TestInspectWithRandomExposedPort(t *testing.T) {
 	var unexpectedPort uint16 = 1234
 	options := &RunOptions{
 		Detach:       true,
-		OtherOptions: []string{fmt.Sprintf("-P")},
+		OtherOptions: []string{"-P"},
 	}
 
 	id := RunAndGetID(t, dockerInspectTestImage, options)

diff --git a/modules/shell/command_test.go b/modules/shell/command_test.go
@@ -121,13 +121,13 @@ func TestRunCommandWithHugeLineOutput(t *testing.T) {
 	t.Parallel()
 
 	// generate a ~100KB line
-	bashCode := fmt.Sprintf(`
+	bashCode := `
 for i in {0..35000}
 do
   echo -n foo
 done
 echo
-`)
+`
 
 	cmd := Command{
 		Command: "bash",