Add a test case under api/krusty for #5878 (#2)

Author: tsekine354

api/krusty/openapicustomschema_test.go

@@ -643,3 +643,58 @@ spec:
 		assert.Equal(t, kubernetesapi.DefaultOpenAPI, openapi.GetSchemaVersion())
 	})
 }
+
+// Tett for issue #5878
+func TestCustomOpenApiFieldWithoutMergePatchExtension(t *testing.T) {
+	runOpenApiTest(t, func(t *testing.T) {
+		t.Helper()
+		th := kusttest_test.MakeHarness(t)
+		writeTestSchema(th, "./")
+                th.WriteF("yetanothercrd_base.yaml", `
+apiVersion: example.com/v1alpha1
+kind: MyCRD
+metadata:
+  name: service
+spec:
+  custom:
+    objects:
+    - name: foo
+      value: foo
+    - name: bar
+      value: bar
+`)
+                th.WriteF("yetanothercrd_patch.yaml", `
+apiVersion: example.com/v1alpha1
+kind: MyCRD
+metadata:
+  name: service
+spec:
+  custom:
+    objects:
+    - name: bar
+      value: patched
+`)
+                th.WriteK(".", `
+resources:
+- yetanothercrd_base.yaml
+openapi:
+  path: mycrd_schema.json
+patches:
+- path: yetanothercrd_patch.yaml
+`)
+                m := th.Run(".", th.MakeDefaultOptions())
+		th.AssertActualEqualsExpected(m, `
+apiVersion: example.com/v1alpha1
+kind: MyCRD
+metadata:
+  name: service
+spec:
+  custom:
+    objects:
+    - name: foo
+      value: foo
+    - name: bar
+      value: patched
+`)
+        })
+}

api/krusty/testdata/customschema.json

@@ -15,12 +15,15 @@
           "properties": {
             "template": {
               "$ref": "#/definitions/io.k8s.api.core.v1.PodTemplateSpec"
+            },
+            "custom": {
+              "$ref": "#/definitions/com.example.v1alpha1.CustomSpec"
             }
           },
           "type": "object"
         },
         "status": {
-           "properties": {
+          "properties": {
             "success": {
               "type": "boolean"
             }
@@ -89,7 +92,7 @@
           "type": "string"
         },
         "ports": {
-         "items": {
+          "items": {
             "$ref": "#/definitions/io.k8s.api.core.v1.ContainerPort"
           },
           "type": "array",
@@ -105,7 +108,7 @@
       "type": "object"
     },
     "io.k8s.api.core.v1.ContainerPort": {
-     "properties": {
+      "properties": {
         "containerPort": {
           "format": "int32",
           "type": "integer"
@@ -118,6 +121,27 @@
         }
       },
       "type": "object"
+    },
+    "com.example.v1alpha1.CustomSpec": {
+      "properties": {
+        "objects": {
+          "items": {
+            "properties": {
+              "name": {
+                "type": "string"
+              },
+              "value": {
+                "type": "string"
+              }
+            }
+          },
+          "type": "array",
+          "x-kubernetes-list-map-keys": [
+            "name"
+          ],
+          "x-kubernetes-list-type": "map"
+        }
+      }
     }
   }
 }

api/krusty/testdata/customschema.yaml

@@ -11,6 +11,8 @@ definitions:
         properties:
           template:
             "$ref": "#/definitions/io.k8s.api.core.v1.PodTemplateSpec"
+          custom:
+            "$ref": "#/definitions/com.example.v1alpha1.CustomSpec"
         type: object
       status:
         properties:
@@ -70,9 +72,23 @@ definitions:
   io.k8s.api.core.v1.ContainerPort:
     properties:
       containerPort:
+        format: int32
         type: integer
       name:
         type: string
       protocol:
         type: string
     type: object
+  com.example.v1alpha1.CustomSpec:
+    properties:
+      objects:
+        items:
+          properties:
+            name:
+              type: string
+            value:
+              type: string
+        type: array
+        x-kubernetes-list-map-keys:
+          - name
+        x-kubernetes-list-type: map