Fix test assertion

kubernetes · Nov 13, 2024 · 8eac53c · 8eac53c
1 parent f3e7f3a
commit 8eac53c
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 7 deletions.
diff --git a/test/e2e/lua/dynamic_configuration.go b/test/e2e/lua/dynamic_configuration.go
@@ -212,7 +212,8 @@ func createIngress(f *framework.Framework, host, deploymentName string) {
 
 	f.WaitForNginxServer(host,
 		func(server string) bool {
-			return strings.Contains(server, fmt.Sprintf("server_name %s;", host)) &&
+			return (strings.Contains(server, fmt.Sprintf("server_name %s;", host)) ||
+				strings.Contains(server, fmt.Sprintf("server_name %s ;", host))) &&
 				strings.Contains(server, "proxy_pass http://upstream_balancer;")
 		})
 }

diff --git a/test/e2e/metrics/metrics.go b/test/e2e/metrics/metrics.go
@@ -43,7 +43,8 @@ var _ = framework.IngressNginxDescribe("[metrics] exported prometheus metrics",
 		f.EnsureIngress(framework.NewSingleIngress(host, "/", host, f.Namespace, framework.EchoService, 80, nil))
 		f.WaitForNginxServer(host,
 			func(server string) bool {
-				return strings.Contains(server, fmt.Sprintf("server_name %s;", host)) &&
+				return (strings.Contains(server, fmt.Sprintf("server_name %s;", host)) ||
+					strings.Contains(server, fmt.Sprintf("server_name %s ;", host))) &&
 					strings.Contains(server, "proxy_pass http://upstream_balancer;")
 			})
 	})

diff --git a/test/e2e/settings/badannotationvalues.go b/test/e2e/settings/badannotationvalues.go
@@ -50,7 +50,8 @@ var _ = framework.DescribeAnnotation("Bad annotation values", func() {
 
 		f.WaitForNginxServer(host,
 			func(server string) bool {
-				return !strings.Contains(server, fmt.Sprintf("server_name %s;", host))
+				return !strings.Contains(server, fmt.Sprintf("server_name %s;", host)) &&
+					!strings.Contains(server, fmt.Sprintf("server_name %s ;", host))
 			})
 
 		f.WaitForNginxServer(host,
@@ -87,7 +88,8 @@ var _ = framework.DescribeAnnotation("Bad annotation values", func() {
 
 		f.WaitForNginxServer(host,
 			func(server string) bool {
-				return !strings.Contains(server, fmt.Sprintf("server_name %s;", host))
+				return !strings.Contains(server, fmt.Sprintf("server_name %s;", host)) &&
+					!strings.Contains(server, fmt.Sprintf("server_name %s ;", host))
 			})
 
 		f.WaitForNginxServer(host,
@@ -123,7 +125,8 @@ var _ = framework.DescribeAnnotation("Bad annotation values", func() {
 
 		f.WaitForNginxServer(hostValid,
 			func(server string) bool {
-				return strings.Contains(server, fmt.Sprintf("server_name %s;", hostValid))
+				return strings.Contains(server, fmt.Sprintf("server_name %s;", hostValid)) ||
+					strings.Contains(server, fmt.Sprintf("server_name %s ;", hostValid))
 			})
 
 		f.WaitForNginxServer(hostValid,
@@ -156,7 +159,8 @@ var _ = framework.DescribeAnnotation("Bad annotation values", func() {
 
 		f.WaitForNginxServer(host,
 			func(server string) bool {
-				return !strings.Contains(server, fmt.Sprintf("server_name %s;", host))
+				return !strings.Contains(server, fmt.Sprintf("server_name %s;", host)) &&
+					!strings.Contains(server, fmt.Sprintf("server_name %s ;", host))
 			})
 
 		f.WaitForNginxServer(host,

diff --git a/test/e2e/settings/limit_rate.go b/test/e2e/settings/limit_rate.go
@@ -41,7 +41,8 @@ var _ = framework.DescribeSetting("Configmap - limit-rate", func() {
 		f.EnsureIngress(ing)
 
 		f.WaitForNginxServer(host, func(server string) bool {
-			return strings.Contains(server, fmt.Sprintf("server_name %s;", host))
+			return strings.Contains(server, fmt.Sprintf("server_name %s;", host)) ||
+				strings.Contains(server, fmt.Sprintf("server_name %s ;", host))
 		})
 
 		wlKey := "limit-rate"