TEST/MINOR: fix some unit tests and e2e tests random failures

Author: hdurand0710

deploy/tests/e2e/crd-tcp/suite_test.go

@@ -21,6 +21,9 @@ import (
 	"io"
 	"strings"
 
+	"github.com/google/go-cmp/cmp"
+	"github.com/google/go-cmp/cmp/cmpopts"
+
 	"github.com/stretchr/testify/suite"
 
 	parser "github.com/haproxytech/config-parser/v5"
@@ -81,7 +84,19 @@ func (suite *CRDTCPSuite) BeforeTest(suiteName, testName string) {
 func (suite *CRDTCPSuite) checkFrontend(p parser.Parser, frontendName, param string, value common.ParserData) {
 	v, err := p.Get(parser.Frontends, frontendName, param)
 	suite.Require().NoError(err, "Could not get Haproxy config parser Frontend %s param %s", frontendName, param)
-	suite.Equal(value, v, fmt.Sprintf("Frontend param %s should be equal to %v but is %v", param, value, v))
+
+	// Sort functions for map[string]XXX
+	sortBinds := cmpopts.SortSlices(func(a, b types.Bind) bool {
+		return a.Path < b.Path
+	})
+	areEqual := cmp.Equal(v, value, sortBinds)
+
+	if !areEqual {
+		diff := cmp.Diff(v, value, sortBinds)
+		suite.T().Log(diff)
+	}
+
+	suite.Equal(areEqual, true, fmt.Sprintf("Frontend param %s should be equal to %v but is %v", param, value, v))
 }
 
 func (suite *CRDTCPSuite) checkBackend(p parser.Parser, backendName, param string, value common.ParserData) {

deploy/tests/integration/config-snippet/suite_test.go

@@ -15,6 +15,7 @@
 package configsnippet_test
 
 import (
+	"os"
 	"testing"
 
 	"github.com/haproxytech/kubernetes-ingress/deploy/tests/integration"
@@ -44,6 +45,8 @@ func (suite *DisableConfigSnippetSuite) BeforeTest(suiteName, testName string) {
 	suite.BaseSuite.BeforeTest(suiteName, testName)
 	// Add any needed update to the controller setting
 	// by updating suite.TestControllers[suite.T().Name()].XXXXX
+	os.Unsetenv("POD_NAME")
+	os.Unsetenv("POD_NAMESPACE")
 	testController := suite.TestControllers[suite.T().Name()]
 	testController.OSArgs.ConfigMap.Name = configMapName
 	testController.OSArgs.ConfigMap.Namespace = configMapNamespace

deploy/tests/integration/pod-maxconn/suite_test.go

@@ -15,6 +15,7 @@
 package podmaxconn
 
 import (
+	"os"
 	"testing"
 
 	"github.com/haproxytech/kubernetes-ingress/deploy/tests/integration"
@@ -44,6 +45,8 @@ func (suite *PodMaxConnSuite) BeforeTest(suiteName, testName string) {
 	suite.BaseSuite.BeforeTest(suiteName, testName)
 	// Add any needed update to the controller setting
 	// by updating suite.TestControllers[suite.T().Name()].XXXXX
+	os.Unsetenv("POD_NAME")
+	os.Unsetenv("POD_NAMESPACE")
 	testController := suite.TestControllers[suite.T().Name()]
 	testController.OSArgs.ConfigMap.Name = configMapName
 	testController.OSArgs.ConfigMap.Namespace = configMapNamespace

deploy/tests/integration/timeout-server/suite_test.go

@@ -15,6 +15,7 @@
 package timeoutserver
 
 import (
+	"os"
 	"testing"
 
 	"github.com/haproxytech/kubernetes-ingress/deploy/tests/integration"
@@ -44,6 +45,8 @@ func (suite *TimeoutServerSuite) BeforeTest(suiteName, testName string) {
 	suite.BaseSuite.BeforeTest(suiteName, testName)
 	// Add any needed update to the controller setting
 	// by updating suite.TestControllers[suite.T().Name()].XXXXX
+	os.Unsetenv("POD_NAME")
+	os.Unsetenv("POD_NAMESPACE")
 	testController := suite.TestControllers[suite.T().Name()]
 	testController.OSArgs.ConfigMap.Name = configMapName
 	testController.OSArgs.ConfigMap.Namespace = configMapNamespace

deploy/tests/ut/acls/suite_test.go

@@ -67,6 +67,10 @@ func (suite *ACLSuite) BeforeTest(suiteName, testName string) {
 	suite.T().Logf("temporary configuration dir %s", suite.test.TempDir)
 }
 
+func (suite *ACLSuite) TearDownSuite() {
+	os.Unsetenv("POD_NAME")
+}
+
 func (suite *ACLSuite) UseACLFixture() (eventChan chan k8ssync.SyncDataEvent) {
 	var osArgs utils.OSArgs
 	os.Args = []string{os.Args[0], "-e", "-t", "--config-dir=" + suite.test.TempDir}

deploy/tests/ut/httprequests/suite_test.go

@@ -67,6 +67,10 @@ func (suite *HTTPRequestsSuite) BeforeTest(suiteName, testName string) {
 	suite.T().Logf("temporary configuration dir %s", suite.test.TempDir)
 }
 
+func (suite *HTTPRequestsSuite) TearDownSuite() {
+	os.Unsetenv("POD_NAME")
+}
+
 func (suite *HTTPRequestsSuite) UseHTTPRequestsFixture() (eventChan chan k8ssync.SyncDataEvent) {
 	var osArgs utils.OSArgs
 	os.Args = []string{os.Args[0], "-e", "-t", "--config-dir=" + suite.test.TempDir}

go.mod

@@ -7,6 +7,7 @@ require (
 	github.com/fasthttp/router v1.4.20
 	github.com/go-openapi/swag v0.23.0
 	github.com/go-test/deep v1.1.0
+	github.com/google/go-cmp v0.6.0
 	github.com/google/renameio v1.0.1
 	github.com/haproxytech/client-native/v3 v3.1.2-0.20230607075433-231591da68ed
 	github.com/haproxytech/client-native/v5 v5.1.4
@@ -51,7 +52,6 @@ require (
 	github.com/gogo/protobuf v1.3.2 // indirect
 	github.com/golang/protobuf v1.5.4 // indirect
 	github.com/google/gnostic-models v0.6.8 // indirect
-	github.com/google/go-cmp v0.6.0 // indirect
 	github.com/google/gofuzz v1.2.0 // indirect
 	github.com/google/uuid v1.6.0 // indirect
 	github.com/haproxytech/go-logger v1.1.0 // indirect