refactor: kube utils (#1689)

Signed-off-by: Charles-Edouard Brétéché <[email protected]>

Author: eddycharly

pkg/client/pet.go

@@ -24,6 +24,7 @@ import (
 	"github.com/kyverno/chainsaw/pkg/runner/summary"
 	"github.com/kyverno/chainsaw/pkg/runner/timeout"
 	"github.com/kyverno/chainsaw/pkg/testing"
+	"github.com/kyverno/chainsaw/pkg/utils/kube"
 	"github.com/kyverno/pkg/ext/output/color"
 	corev1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/api/errors"
@@ -133,14 +134,14 @@ func (p *testProcessor) Run(ctx context.Context, bindings binding.Bindings, nspa
 		if nspacer == nil || p.test.Test.Spec.Namespace != "" {
 			var ns corev1.Namespace
 			if p.test.Test.Spec.Namespace != "" {
-				ns = client.Namespace(p.test.Test.Spec.Namespace)
+				ns = kube.Namespace(p.test.Test.Spec.Namespace)
 			} else {
-				ns = client.PetNamespace()
+				ns = kube.PetNamespace()
 			}
 			namespace = &ns
 		}
 		if namespace != nil {
-			object := client.ToUnstructured(namespace)
+			object := kube.ToUnstructured(namespace)
 			bindings = apibindings.RegisterNamedBinding(ctx, bindings, "namespace", object.GetName())
 			if p.test.Test.Spec.NamespaceTemplate != nil && p.test.Test.Spec.NamespaceTemplate.Value != nil {
 				template := v1alpha1.Any{

pkg/runner/processors/test.go

@@ -24,6 +24,7 @@ import (
 	"github.com/kyverno/chainsaw/pkg/runner/summary"
 	"github.com/kyverno/chainsaw/pkg/runner/timeout"
 	"github.com/kyverno/chainsaw/pkg/testing"
+	"github.com/kyverno/chainsaw/pkg/utils/kube"
 	"github.com/kyverno/pkg/ext/output/color"
 	"k8s.io/apimachinery/pkg/api/errors"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -84,8 +85,8 @@ func (p *testsProcessor) Run(ctx context.Context, bindings binding.Bindings) {
 	bindings = apibindings.RegisterClusterBindings(ctx, bindings, clusterConfig, clusterClient)
 	if clusterClient != nil {
 		if p.config.Namespace.Name != "" {
-			namespace := client.Namespace(p.config.Namespace.Name)
-			object := client.ToUnstructured(&namespace)
+			namespace := kube.Namespace(p.config.Namespace.Name)
+			object := kube.ToUnstructured(&namespace)
 			bindings = apibindings.RegisterNamedBinding(ctx, bindings, "namespace", object.GetName())
 			if p.config.Namespace.Template != nil && p.config.Namespace.Template.Value != nil {
 				template := v1alpha1.Any{

pkg/runner/processors/tests.go

@@ -1,6 +1,9 @@
-package client
+package kube
 
 import (
+	"fmt"
+
+	petname "github.com/dustinkirkland/golang-petname"
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
@@ -18,5 +21,5 @@ func Namespace(name string) corev1.Namespace {
 }
 
 func PetNamespace() corev1.Namespace {
-	return Namespace(Pet())
+	return Namespace(fmt.Sprintf("chainsaw-%s", petname.Generate(2, "-")))
 }

pkg/client/namespace.go renamed to pkg/utils/kube/namespace.go

@@ -1,4 +1,4 @@
-package client
+package kube
 
 import (
 	"testing"

pkg/client/namespace_test.go renamed to pkg/utils/kube/namespace_test.go

@@ -1,4 +1,4 @@
-package client
+package kube
 
 import (
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"

pkg/client/unstructured.go renamed to pkg/utils/kube/unstructured.go

@@ -1,4 +1,4 @@
-package client
+package kube
 
 import (
 	"testing"