From e568e2a23a4fe95092554aa6b3b31bd1ba5a7a7e Mon Sep 17 00:00:00 2001
From: Rob Scott <robertjscott@google.com>
Date: Mon, 15 May 2023 17:07:34 +0000
Subject: [PATCH] Replacing fmt.Sprintln with plain strings

---
 apis/v1beta1/validation/gateway.go      |  8 ++++----
 apis/v1beta1/validation/gateway_test.go | 12 ++++++------
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/apis/v1beta1/validation/gateway.go b/apis/v1beta1/validation/gateway.go
index d78972808c..3119efb2c0 100644
--- a/apis/v1beta1/validation/gateway.go
+++ b/apis/v1beta1/validation/gateway.go
@@ -121,7 +121,7 @@ func ValidateTLSCertificateRefs(listeners []gatewayv1b1.Listener, path *field.Pa
 	for i, c := range listeners {
 		if isProtocolInSubset(c.Protocol, protocolsTLSRequired) && c.TLS != nil {
 			if *c.TLS.Mode == gatewayv1b1.TLSModeTerminate && len(c.TLS.CertificateRefs) == 0 {
-				errs = append(errs, field.Forbidden(path.Index(i).Child("tls").Child("certificateRefs"), fmt.Sprintln("should be set and not empty when TLSModeType is Terminate")))
+				errs = append(errs, field.Forbidden(path.Index(i).Child("tls").Child("certificateRefs"), "should be set and not empty when TLSModeType is Terminate"))
 			}
 		}
 	}
@@ -135,7 +135,7 @@ func ValidateListenerNames(listeners []gatewayv1b1.Listener, path *field.Path) f
 	nameMap := make(map[gatewayv1b1.SectionName]struct{}, len(listeners))
 	for i, c := range listeners {
 		if _, found := nameMap[c.Name]; found {
-			errs = append(errs, field.Duplicate(path.Index(i).Child("name"), fmt.Sprintln("must be unique within the Gateway")))
+			errs = append(errs, field.Duplicate(path.Index(i).Child("name"), "must be unique within the Gateway"))
 		}
 		nameMap[c.Name] = struct{}{}
 	}
@@ -156,7 +156,7 @@ func validateHostnameProtocolPort(listeners []gatewayv1b1.Listener, path *field.
 		port := listener.Port
 		hostnameProtocolPort := fmt.Sprintf("%s:%s:%d", *hostname, protocol, port)
 		if hostnameProtocolPortSets.Has(hostnameProtocolPort) {
-			errs = append(errs, field.Duplicate(path.Index(i), fmt.Sprintln("combination of port, protocol, and hostname must be unique for each listener")))
+			errs = append(errs, field.Duplicate(path.Index(i), "combination of port, protocol, and hostname must be unique for each listener"))
 		} else {
 			hostnameProtocolPortSets.Insert(hostnameProtocolPort)
 		}
@@ -173,7 +173,7 @@ func validateGatewayAddresses(addresses []gatewayv1b1.GatewayAddress, path *fiel
 		if address.Type != nil {
 			if *address.Type == gatewayv1b1.IPAddressType {
 				if _, err := netip.ParseAddr(address.Value); err != nil {
-					errs = append(errs, field.Invalid(path.Index(i), address.Value, fmt.Sprintln("invalid ip address")))
+					errs = append(errs, field.Invalid(path.Index(i), address.Value, "invalid ip address"))
 				}
 				if ipAddrSet.Has(address.Value) {
 					errs = append(errs, field.Duplicate(path.Index(i), address.Value))
diff --git a/apis/v1beta1/validation/gateway_test.go b/apis/v1beta1/validation/gateway_test.go
index ab4ad963bc..0feaec2627 100644
--- a/apis/v1beta1/validation/gateway_test.go
+++ b/apis/v1beta1/validation/gateway_test.go
@@ -169,7 +169,7 @@ func TestValidateGateway(t *testing.T) {
 				{
 					Type:     field.ErrorTypeForbidden,
 					Field:    "spec.listeners[0].tls.certificateRefs",
-					Detail:   "should be set and not empty when TLSModeType is Terminate\n",
+					Detail:   "should be set and not empty when TLSModeType is Terminate",
 					BadValue: "",
 				},
 			},
@@ -187,7 +187,7 @@ func TestValidateGateway(t *testing.T) {
 				{
 					Type:     field.ErrorTypeForbidden,
 					Field:    "spec.listeners[0].tls.certificateRefs",
-					Detail:   "should be set and not empty when TLSModeType is Terminate\n",
+					Detail:   "should be set and not empty when TLSModeType is Terminate",
 					BadValue: "",
 				},
 			},
@@ -209,7 +209,7 @@ func TestValidateGateway(t *testing.T) {
 				{
 					Type:     field.ErrorTypeDuplicate,
 					Field:    "spec.listeners[1].name",
-					BadValue: "must be unique within the Gateway\n",
+					BadValue: "must be unique within the Gateway",
 				},
 			},
 		},
@@ -233,7 +233,7 @@ func TestValidateGateway(t *testing.T) {
 				{
 					Type:     field.ErrorTypeDuplicate,
 					Field:    "spec.listeners[1]",
-					BadValue: "combination of port, protocol, and hostname must be unique for each listener\n",
+					BadValue: "combination of port, protocol, and hostname must be unique for each listener",
 				},
 			},
 		},
@@ -254,7 +254,7 @@ func TestValidateGateway(t *testing.T) {
 				{
 					Type:     field.ErrorTypeDuplicate,
 					Field:    "spec.listeners[1]",
-					BadValue: "combination of port, protocol, and hostname must be unique for each listener\n",
+					BadValue: "combination of port, protocol, and hostname must be unique for each listener",
 				},
 			},
 		},
@@ -363,7 +363,7 @@ func TestValidateGateway(t *testing.T) {
 				{
 					Type:     field.ErrorTypeInvalid,
 					Field:    "spec.addresses[0]",
-					Detail:   "invalid ip address\n",
+					Detail:   "invalid ip address",
 					BadValue: "1.2.3.4:8080",
 				},
 				{