Remove count field, convert error to warning

This commit is contained in:
Phani Teja Marupaka
2020-06-30 17:00:53 -07:00
parent e5b0ceb4e3
commit 9567d7ef16
3 changed files with 41 additions and 9 deletions

View File

@@ -55,7 +55,6 @@ openAPI:
name: replicas
value: "3"
setBy: me
count: 1
`,
expectedResources: `
apiVersion: apps/v1
@@ -66,6 +65,44 @@ spec:
replicas: 3 # {"$openapi":"replicas"}
`,
},
{
name: "add replicas no match",
args: []string{"replicas", "3", "--description", "hello world", "--set-by", "me"},
input: `
apiVersion: apps/v1
kind: Deployment
metadata:
name: nginx-deployment
spec:
foo: 2
`,
inputOpenAPI: `
apiVersion: v1alpha1
kind: Example
`,
expectedOpenAPI: `
apiVersion: v1alpha1
kind: Example
openAPI:
definitions:
io.k8s.cli.setters.replicas:
description: hello world
x-k8s-cli:
setter:
name: replicas
value: "3"
setBy: me
`,
expectedResources: `
apiVersion: apps/v1
kind: Deployment
metadata:
name: nginx-deployment
spec:
foo: 2
`,
},
{
name: "error if substitution with same name exists",
args: []string{"my-image", "3", "--description", "hello world", "--set-by", "me"},
@@ -118,7 +155,6 @@ openAPI:
name: replicas
value: "3"
setBy: me
count: 1
`,
expectedResources: `
apiVersion: apps/v1
@@ -190,7 +226,6 @@ openAPI:
- b
- c
setBy: me
count: 2
`,
expectedResources: `
apiVersion: example.com/v1beta1
@@ -290,7 +325,6 @@ openAPI:
- b
- c
setBy: me
count: 1
`,
expectedResources: `
apiVersion: example.com/v1beta1
@@ -330,7 +364,6 @@ openAPI:
name: replicas
value: "3"
setBy: me
count: 1
`,
expectedResources: `
apiVersion: apps/v1