test(e2e): modify controlplaneinfo data

Signed-off-by: Alessio Greggi <ale_grey_91@hotmail.it>
kubescape · Jul 5, 2023 · f92a442 · f92a442
1 parent 722d455
commit f92a442
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 24 deletions.
diff --git a/e2e/controlplaneinfo_kind_test.go b/e2e/controlplaneinfo_kind_test.go
@@ -11,58 +11,61 @@ var pkiFiles = &sensor.ControlPlaneInfo{
 	PKIDIr: &ds.FileInfo{Path: "/etc/kubernetes/pki"},
 	PKIFiles: []*ds.FileInfo{
 		{
-			Path: "/etc/kubernetes/pki/etcd/peer.crt",
+			Path: "/etc/kubernetes/pki/apiserver-etcd-client.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/etcd/healthcheck-client.crt",
+			Path: "/etc/kubernetes/pki/apiserver-etcd-client.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/etcd/ca.crt",
+			Path: "/etc/kubernetes/pki/apiserver-kubelet-client.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/etcd/ca.key",
+			Path: "/etc/kubernetes/pki/apiserver-kubelet-client.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/etcd/server.crt",
+			Path: "/etc/kubernetes/pki/apiserver.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/etcd/server.key",
+			Path: "/etc/kubernetes/pki/apiserver.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/etcd/healthcheck-client.key",
+			Path: "/etc/kubernetes/pki/ca.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/etcd/peer.key",
+			Path: "/etc/kubernetes/pki/ca.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/ca.crt",
+			Path: "/etc/kubernetes/pki/etcd/ca.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/apiserver.crt",
+			Path: "/etc/kubernetes/pki/etcd/ca.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/ca.key",
+			Path: "/etc/kubernetes/pki/etcd/healthcheck-client.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/front-proxy-client.crt",
+			Path: "/etc/kubernetes/pki/etcd/healthcheck-client.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/front-proxy-ca.key",
+			Path: "/etc/kubernetes/pki/etcd/peer.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/apiserver-etcd-client.crt",
+			Path: "/etc/kubernetes/pki/etcd/peer.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/front-proxy-ca.crt",
+			Path: "/etc/kubernetes/pki/etcd/server.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/apiserver-kubelet-client.crt",
+			Path: "/etc/kubernetes/pki/etcd/server.key",
 		},
 		{
-			Path: "/etc/kubernetes/pki/apiserver-etcd-client.key",
+			Path: "/etc/kubernetes/pki/front-proxy-ca.crt",
 		},
 		{
-			Path: "/etc/kubernetes/pki/apiserver.key",
+			Path: "/etc/kubernetes/pki/front-proxy-ca.key",
+		},
+		{
+			Path: "/etc/kubernetes/pki/front-proxy-client.crt",
 		},
 		{
 			Path: "/etc/kubernetes/pki/front-proxy-client.key",
@@ -73,9 +76,6 @@ var pkiFiles = &sensor.ControlPlaneInfo{
 		{
 			Path: "/etc/kubernetes/pki/sa.pub",
 		},
-		{
-			Path: "/etc/kubernetes/pki/apiserver-kubelet-client.key",
-		},
 	},
 }
 

diff --git a/e2e/controlplaneinfo_test.go b/e2e/controlplaneinfo_test.go
@@ -4,6 +4,7 @@ package e2e_test
 
 import (
 	"encoding/json"
+	"fmt"
 	"io/ioutil"
 	"net/http"
 
@@ -41,9 +42,9 @@ var _ = Describe("ControlPlaneInfo", func() {
 			Expect(resultBody.PKIDIr.Path).To(Equal(pkiFiles.PKIDIr.Path))
 
 			// (leave it there for debugging)
-			//for i := range resultBody.PKIFiles {
-			//	fmt.Println(resultBody.PKIFiles[i].Path)
-			//}
+			for i := range resultBody.PKIFiles {
+				fmt.Println(resultBody.PKIFiles[i].Path)
+			}
 			for i := range resultBody.PKIFiles {
 				Expect(resultBody.PKIFiles[i].Path).To(Equal(pkiFiles.PKIFiles[i].Path))
 			}

diff --git a/sensor/verboseutils.go b/sensor/verboseutils.go
@@ -7,6 +7,7 @@ import (
 	"io"
 	"os"
 	"path"
+	"sort"
 
 	"github.com/kubescape/go-logger"
 	"github.com/kubescape/go-logger/helpers"
@@ -79,6 +80,8 @@ func makeHostDirFilesInfoVerbose(ctx context.Context, dir string, recursive bool
 
 	var fileNames []string
 	for fileNames, err = dirInfo.Readdirnames(100); err == nil; fileNames, err = dirInfo.Readdirnames(100) {
+		// add sorting to make tests deterministic
+		sort.Strings(fileNames)
 		for i := range fileNames {
 			filePath := path.Join(dir, fileNames[i])