Merge pull request kubernetes#86144 from tahsinrahman/dont-serialize-internal

Do not serialize internal types in ComponentConfig tests

Author: k8s-ci-robot

pkg/proxy/apis/config/scheme/testdata/KubeProxyConfiguration/after/__internal.yaml

@@ -0,0 +1,2 @@
+apiVersion: kubeproxy.config.k8s.io/v1alpha1
+kind: KubeProxyConfiguration

pkg/proxy/apis/config/scheme/testdata/KubeProxyConfiguration/after/v1alpha1.yaml

@@ -13,11 +13,11 @@ go_library(
     visibility = ["//visibility:public"],
     deps = [
         "//staging/src/k8s.io/apimachinery/pkg/api/apitesting/naming:go_default_library",
+        "//staging/src/k8s.io/apimachinery/pkg/api/equality:go_default_library",
         "//staging/src/k8s.io/apimachinery/pkg/runtime:go_default_library",
         "//staging/src/k8s.io/apimachinery/pkg/runtime/schema:go_default_library",
         "//staging/src/k8s.io/apimachinery/pkg/runtime/serializer:go_default_library",
         "//staging/src/k8s.io/apimachinery/pkg/util/errors:go_default_library",
-        "//staging/src/k8s.io/apimachinery/pkg/util/runtime:go_default_library",
         "//staging/src/k8s.io/apimachinery/pkg/util/sets:go_default_library",
         "//vendor/github.com/google/go-cmp/cmp:go_default_library",
     ],

pkg/proxy/apis/config/scheme/testdata/KubeProxyConfiguration/before/__internal.yaml

@@ -27,24 +27,31 @@ import (
 
 // DefaultingTest run defaulting tests for given scheme
 func DefaultingTest(t *testing.T, scheme *runtime.Scheme, codecs serializer.CodecFactory) {
-	tc := GetDefaultingTestCases(scheme)
-	RunTestsOnYAMLData(t, scheme, tc, codecs)
+	cases := GetDefaultingTestCases(t, scheme, codecs)
+	RunTestsOnYAMLData(t, cases)
 }
 
 // GetDefaultingTestCases returns defaulting testcases for given scheme
-func GetDefaultingTestCases(scheme *runtime.Scheme) []TestCase {
+func GetDefaultingTestCases(t *testing.T, scheme *runtime.Scheme, codecs serializer.CodecFactory) []TestCase {
 	cases := []TestCase{}
 	for gvk := range scheme.AllKnownTypes() {
+		if gvk.Version == runtime.APIVersionInternal {
+			continue
+		}
 		beforeDir := fmt.Sprintf("testdata/%s/before", gvk.Kind)
 		afterDir := fmt.Sprintf("testdata/%s/after", gvk.Kind)
 		filename := fmt.Sprintf("%s.yaml", gvk.Version)
 
+		codec, err := getCodecForGV(codecs, gvk.GroupVersion())
+		if err != nil {
+			t.Fatal(err)
+		}
+
 		cases = append(cases, TestCase{
 			name:  fmt.Sprintf("default_%s", gvk.Version),
 			in:    filepath.Join(beforeDir, filename),
-			inGVK: gvk,
 			out:   filepath.Join(afterDir, filename),
-			outGV: gvk.GroupVersion(),
+			codec: codec,
 		})
 	}
 	return cases