mirror of
https://github.com/kubernetes-sigs/kustomize.git
synced 2026-06-11 00:52:55 +00:00
691b7d1df3
When a patch appends a new node, it should honor the key style from the patch. Prior to this commit, no style was applied, leading to problems when the key value could be interpreted as a different type based on its content. For example, "9110" needs quoting to ensure it is seen as a string in yaml.
367 lines
5.9 KiB
Go
367 lines
5.9 KiB
Go
// Copyright 2019 The Kubernetes Authors.
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
package merge3_test
|
|
|
|
var mapTestCases = []testCase{
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Add the annotations map field`,
|
|
origin: `
|
|
kind: Deployment`,
|
|
update: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
d: e # add these annotations
|
|
`,
|
|
local: `
|
|
kind: Deployment`,
|
|
expected: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
d: e # add these annotations`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Add an annotation to the field`,
|
|
origin: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b`,
|
|
update: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b
|
|
d: e # add these annotations`,
|
|
local: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
g: h # keep these annotations`,
|
|
expected: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
g: h # keep these annotations
|
|
d: e # add these annotations`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Add an annotation to the field, field missing from dest`,
|
|
origin: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b # ignored because unchanged`,
|
|
update: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b # ignored because unchanged
|
|
d: e`,
|
|
local: `
|
|
kind: Deployment`,
|
|
expected: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
d: e`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Update an annotation on the field, field messing rom the dest`,
|
|
origin: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b
|
|
d: c`,
|
|
update: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b
|
|
d: e # set these annotations`,
|
|
local: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
g: h # keep these annotations`,
|
|
expected: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
g: h # keep these annotations
|
|
d: e # set these annotations`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Add an annotation to the field, field missing from dest`,
|
|
origin: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b # ignored because unchanged`,
|
|
update: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b # ignored because unchanged
|
|
d: e`,
|
|
local: `
|
|
kind: Deployment`,
|
|
expected: `
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
d: e`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Remove an annotation`,
|
|
origin: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b`,
|
|
update: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations: {}`,
|
|
local: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
c: d
|
|
a: b`,
|
|
expected: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
c: d`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
// TODO(#36) support ~annotations~: {} deletion
|
|
{
|
|
description: `Specify a field as empty that isn't present in the source`,
|
|
origin: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo`,
|
|
update: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
annotations: null`,
|
|
local: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
annotations:
|
|
a: b`,
|
|
expected: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Remove an annotation`,
|
|
origin: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b`,
|
|
update: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment`,
|
|
local: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
c: d
|
|
a: b`,
|
|
expected: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
c: d`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Remove annotations field`,
|
|
origin: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b`,
|
|
update: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment`,
|
|
local: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo`,
|
|
expected: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Remove annotations field, but keep in dest`,
|
|
origin: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
annotations:
|
|
a: b`,
|
|
update: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment`,
|
|
local: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
annotations:
|
|
foo: bar # keep this annotation even though the parent field was removed`,
|
|
expected: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
annotations:
|
|
foo: bar # keep this annotation even though the parent field was removed`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Remove annotations, but they are already empty`,
|
|
origin: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
annotations:
|
|
a: b
|
|
`,
|
|
update: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
`,
|
|
local: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
annotations: {}
|
|
`,
|
|
expected: `
|
|
apiVersion: apps/v1
|
|
kind: Deployment
|
|
metadata:
|
|
name: foo
|
|
annotations: {}
|
|
`},
|
|
|
|
//
|
|
// Test Case
|
|
//
|
|
{
|
|
description: `Verify key style behavior`,
|
|
origin: `
|
|
apiVersion: v1
|
|
kind: ConfigMap
|
|
metadata:
|
|
name: foo
|
|
data:
|
|
unchanged: origin
|
|
unchanged-varying-key-style: origin
|
|
deleted-in-update: origin
|
|
deleted-in-local: origin
|
|
`,
|
|
update: `
|
|
apiVersion: v1
|
|
kind: ConfigMap
|
|
metadata:
|
|
name: foo
|
|
data:
|
|
unchanged: origin
|
|
'unchanged-varying-key-style': origin
|
|
deleted-in-local: origin
|
|
'added-in-update': 'update'
|
|
'added-in-update-and-local-same-value': 'update-and-local'
|
|
'added-in-update-and-local-diff-value': 'update'
|
|
`,
|
|
local: `
|
|
apiVersion: v1
|
|
kind: ConfigMap
|
|
metadata:
|
|
name: foo
|
|
data:
|
|
unchanged: origin
|
|
"unchanged-varying-key-style": origin
|
|
deleted-in-update: origin
|
|
"added-in-local": "local"
|
|
"added-in-update-and-local-same-value": "update-and-local"
|
|
"added-in-update-and-local-diff-value": "local"
|
|
`,
|
|
expected: `
|
|
apiVersion: v1
|
|
kind: ConfigMap
|
|
metadata:
|
|
name: foo
|
|
data:
|
|
unchanged: origin
|
|
"unchanged-varying-key-style": origin
|
|
"added-in-local": "local"
|
|
"added-in-update-and-local-same-value": "update-and-local"
|
|
"added-in-update-and-local-diff-value": "update"
|
|
'added-in-update': 'update'
|
|
`,
|
|
},
|
|
}
|