test: update golden files and expectations for new YAML output order

After switching Resource.AsYAML() to use RNode.MustString(), YAML output
now preserves the original field order from source YAML instead of
alphabetical ordering. Update all golden files and test expectations to
match the new output order.

This is a follow-up to the MustString() migration in PR kubernetes-sigs#6046.

Author: Rindrics

api/krusty/fnplugin_test.go

@@ -113,20 +113,20 @@ metadata:
   labels:
     airshipit.org/ephemeral-user-data: "true"
   name: node1-bmc-secret
+type: Opaque
 stringData:
   userData: |
     bootcmd:
     - mkdir /mnt/vda
-type: Opaque
 ---
 apiVersion: apps/v1
 kind: Deployment
 metadata:
-  annotations:
-    tshirt-size: small
+  name: nginx
   labels:
     app: nginx
-  name: nginx
+  annotations:
+    tshirt-size: small
 spec:
   selector:
     matchLabels:
@@ -137,8 +137,8 @@ spec:
         app: nginx
     spec:
       containers:
-      - image: nginx
-        name: nginx
+      - name: nginx
+        image: nginx
 `, string(yml))
 	require.NoError(t, fSys.RemoveAll(tmpDir.String()))
 }
@@ -204,20 +204,20 @@ metadata:
   labels:
     airshipit.org/ephemeral-user-data: "true"
   name: node1-bmc-secret
+type: Opaque
 stringData:
   userData: |
     bootcmd:
     - mkdir /mnt/vda
-type: Opaque
 ---
 apiVersion: apps/v1
 kind: Deployment
 metadata:
-  annotations:
-    tshirt-size: small
+  name: nginx
   labels:
     app: nginx
-  name: nginx
+  annotations:
+    tshirt-size: small
 spec:
   selector:
     matchLabels:
@@ -228,8 +228,8 @@ spec:
         app: nginx
     spec:
       containers:
-      - image: nginx
-        name: nginx
+      - name: nginx
+        image: nginx
 `, string(yml))
 	require.NoError(t, fSys.RemoveAll(tmpDir.String()))
 }
@@ -295,20 +295,20 @@ metadata:
   labels:
     airshipit.org/ephemeral-user-data: "true"
   name: node1-bmc-secret
+type: Opaque
 stringData:
   userData: |
     bootcmd:
     - mkdir /mnt/vda
-type: Opaque
 ---
 apiVersion: apps/v1
 kind: Deployment
 metadata:
-  annotations:
-    tshirt-size: small
+  name: nginx
   labels:
     app: nginx
-  name: nginx
+  annotations:
+    tshirt-size: small
 spec:
   selector:
     matchLabels:
@@ -319,8 +319,8 @@ spec:
         app: nginx
     spec:
       containers:
-      - image: nginx
-        name: nginx
+      - name: nginx
+        image: nginx
 `, string(yml))
 	require.NoError(t, fSys.RemoveAll(tmpDir.String()))
 }
@@ -564,13 +564,13 @@ metadata:
     app: simplegenerator
   name: simplegenerator-svc
 spec:
+  selector:
+    app: simplegenerator
   ports:
   - name: http
     port: 8081
     protocol: TCP
     targetPort: 8081
-  selector:
-    app: simplegenerator
 `, string(actual))
 }
 
@@ -614,11 +614,11 @@ metadata:
 	assert.Equal(t, `apiVersion: apps/v1
 kind: Deployment
 metadata:
+  name: foo
   annotations:
     a-bool-value: "false"
     a-int-value: "0"
     a-string-value: ""
-  name: foo
 `, string(actual))
 }
 
@@ -677,16 +677,16 @@ data:
 	assert.Equal(t, `apiVersion: v1
 kind: Namespace
 metadata:
+  name: my-namespace
   labels:
     my-ns-name: function-test
-  name: my-namespace
 ---
 apiVersion: v1
 kind: Namespace
 metadata:
+  name: another-namespace
   labels:
     my-ns-name: function-test
-  name: another-namespace
 `, string(actual))
 }
 

api/krusty/originannotation_test.go

@@ -654,15 +654,15 @@ buildMetadata: [originAnnotations]
 	require.NoError(t, err)
 	assert.Contains(t, string(yml), `kind: ConfigMap
 metadata:
+  name: ldap-configmap-4d7m6k5b42
   annotations:
     config.kubernetes.io/origin: |
       repo: https://github.com/kubernetes-sigs/kustomize
       ref: v1.0.6
       configuredIn: examples/ldap/base/kustomization.yaml
       configuredBy:
         apiVersion: builtin
-        kind: ConfigMapGenerator
-  name: ldap-configmap-4d7m6k5b42`)
+        kind: ConfigMapGenerator`)
 	require.NoError(t, fSys.RemoveAll(tmpDir.String()))
 }
 

api/krusty/pluginenv_test.go

@@ -102,14 +102,14 @@ metadata:
 		th.GetT().Fatalf("unexpected error: %v", err)
 	}
 
-	expected := `apiVersion: v1
-env:
-  kustomize_plugin_config_root: ` + dir + `
-  kustomize_plugin_home: ` + pHome + `
-  pwd: ` + dir + `
-kind: GeneratedEnv
+	expected := `kind: GeneratedEnv
+apiVersion: v1
 metadata:
   name: hello
+env:
+  pwd: ` + dir + `
+  kustomize_plugin_home: ` + pHome + `
+  kustomize_plugin_config_root: ` + dir + `
 `
 
 	if string(actual) != expected {
@@ -144,14 +144,14 @@ metadata:
 		th.GetT().Fatalf("unexpected error: %v", err)
 	}
 
-	expected := `apiVersion: v1
-env:
-  kustomize_plugin_config_root: ` + dir + `
-  kustomize_plugin_home: ` + pHome + `
-  pwd: ` + dir + `
-kind: GeneratedEnv
+	expected := `kind: GeneratedEnv
+apiVersion: v1
 metadata:
   name: hello
+env:
+  pwd: ` + dir + `
+  kustomize_plugin_home: ` + pHome + `
+  kustomize_plugin_config_root: ` + dir + `
 `
 
 	if string(actual) != expected {

api/krusty/testdata/golden/TestAddManagedbyLabel.golden

@@ -1,9 +1,9 @@
 apiVersion: v1
 kind: Service
 metadata:
+  name: myService
   labels:
     app.kubernetes.io/managed-by: kustomize-(test)
-  name: myService
 spec:
   ports:
   - port: 7002

api/krusty/testdata/golden/TestAnnoOriginAndTransformerBuiltinLocal.golden

@@ -1,6 +1,7 @@
 apiVersion: v1
 kind: Service
 metadata:
+  name: foo-myService
   annotations:
     alpha.config.kubernetes.io/transformations: |
       - configuredIn: kustomization.yaml
@@ -9,7 +10,6 @@ metadata:
           kind: PrefixTransformer
     config.kubernetes.io/origin: |
       path: service.yaml
-  name: foo-myService
 spec:
   ports:
   - port: 7002

api/krusty/testdata/golden/TestAnnoOriginBuiltinGeneratorFromFileWithOverlay.golden

@@ -1,30 +1,30 @@
 apiVersion: v1
 kind: Secret
 metadata:
-  annotations:
-    config.kubernetes.io/origin: |
-      path: ../base/short_secret.yaml
   labels:
     airshipit.org/ephemeral-user-data: "true"
   name: node1-bmc-secret
+  annotations:
+    config.kubernetes.io/origin: |
+      path: ../base/short_secret.yaml
+type: Opaque
 stringData:
   userData: |
     bootcmd:
     - mkdir /mnt/vda
-type: Opaque
 ---
 apiVersion: v1
-data:
-  crisis: "true"
-  fruit: Indian Gooseberry
-  year: "2020"
 kind: ConfigMap
 metadata:
+  name: bob-79t79mt227
   annotations:
     config.kubernetes.io/origin: |
       configuredIn: ../base/configmap.yaml
       configuredBy:
         apiVersion: builtin
         kind: ConfigMapGenerator
         name: notImportantHere
-  name: bob-79t79mt227
+data:
+  crisis: "true"
+  fruit: Indian Gooseberry
+  year: "2020"

api/krusty/testdata/golden/TestAnnoOriginConfigMapGeneratorMerge.golden

@@ -1,15 +1,15 @@
 apiVersion: v1
-data:
-  crisis: "true"
-  fruit: Indian Gooseberry
-  month: "12"
-  year: "2020"
 kind: ConfigMap
 metadata:
+  name: bob-bk46gm59c6
   annotations:
     config.kubernetes.io/origin: |
       configuredIn: ../base/kustomization.yaml
       configuredBy:
         apiVersion: builtin
         kind: ConfigMapGenerator
-  name: bob-bk46gm59c6
+data:
+  crisis: "true"
+  fruit: Indian Gooseberry
+  month: "12"
+  year: "2020"

api/krusty/testdata/golden/TestAnnoOriginConfigMapGeneratorReplace.golden

@@ -1,12 +1,12 @@
 apiVersion: v1
-data:
-  month: "12"
 kind: ConfigMap
 metadata:
+  name: bob-f8t5fhtbhc
   annotations:
     config.kubernetes.io/origin: |
       configuredIn: kustomization.yaml
       configuredBy:
         apiVersion: builtin
         kind: ConfigMapGenerator
-  name: bob-f8t5fhtbhc
+data:
+  month: "12"

api/krusty/testdata/golden/TestAnnoOriginCustomExecGenerator.golden

@@ -1,31 +1,31 @@
 apiVersion: v1
 kind: Secret
 metadata:
-  annotations:
-    config.kubernetes.io/origin: |
-      path: short_secret.yaml
   labels:
     airshipit.org/ephemeral-user-data: "true"
   name: node1-bmc-secret
+  annotations:
+    config.kubernetes.io/origin: |
+      path: short_secret.yaml
+type: Opaque
 stringData:
   userData: |
     bootcmd:
     - mkdir /mnt/vda
-type: Opaque
 ---
 apiVersion: apps/v1
 kind: Deployment
 metadata:
+  name: nginx
+  labels:
+    app: nginx
   annotations:
     config.kubernetes.io/origin: |
       configuredIn: gener.yaml
       configuredBy:
         kind: executable
         name: demo
     tshirt-size: small
-  labels:
-    app: nginx
-  name: nginx
 spec:
   selector:
     matchLabels:
@@ -36,5 +36,5 @@ spec:
         app: nginx
     spec:
       containers:
-      - image: nginx
-        name: nginx
+      - name: nginx
+        image: nginx