github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-27 05:27:21 +00:00
Code Issues Projects Releases Wiki Activity

add test case for issue 106277 (PR 106268) (#106532)

Browse Source 
* add test case for issue 106277 - fix edit of non-registered custom API types

* remove unneeded struct, as suggested
This commit is contained in:
Philip Hamer 2022-01-11 18:53:23 -05:00 committed by GitHub
parent 1f382b171e
commit ea2dff12e6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 195 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

195
staging/src/k8s.io/kubectl/pkg/cmd/util/editor/editoptions_test.go
View File

@ -18,6 +18,7 @@ package editor
import ( import (
"reflect" "reflect"
"strings"
"testing" "testing"
corev1 "k8s.io/api/core/v1" corev1 "k8s.io/api/core/v1"
@ -25,6 +26,7 @@ import (
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured" "k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
"k8s.io/apimachinery/pkg/runtime" "k8s.io/apimachinery/pkg/runtime"
"k8s.io/apimachinery/pkg/runtime/schema"
"k8s.io/apimachinery/pkg/types" "k8s.io/apimachinery/pkg/types"
"k8s.io/cli-runtime/pkg/genericclioptions" "k8s.io/cli-runtime/pkg/genericclioptions"
"k8s.io/cli-runtime/pkg/resource" "k8s.io/cli-runtime/pkg/resource"
@ -277,3 +279,196 @@ func TestManagedFieldsExtractAndRestore(t *testing.T) {
}) })
} }
} }
type testVisitor struct {
updatedInfos []*resource.Info
}
func (tv *testVisitor) Visit(f resource.VisitorFunc) error {
var err error
for _, ui := range tv.updatedInfos {
err = f(ui, err)
}
return err
}
var unregMapping = &meta.RESTMapping{
Resource: schema.GroupVersionResource{
Group: "a",
Version: "b",
Resource: "c",
},
GroupVersionKind: schema.GroupVersionKind{
Group: "a",
Version: "b",
Kind: "d",
},
}
func TestEditOptions_visitToPatch(t *testing.T) {
expectedErr := func(err error) bool {
return err != nil && strings.Contains(err.Error(), "At least one of apiVersion, kind and name was changed")
}
type args struct {
originalInfos []*resource.Info
patchVisitor resource.Visitor
results *editResults
}
tests := []struct {
name string
args args
checkErr func(err error) bool
}{
{
name: "name-diff",
args: args{
originalInfos: []*resource.Info{
{
Namespace: "ns",
Name: "before",
Object: &unstructured.Unstructured{
Object: map[string]interface{}{
"apiVersion": "v1",
"kind": "Thingy",
"metadata": map[string]interface{}{
"uid": "12345",
"namespace": "ns",
"name": "before",
},
"spec": map[string]interface{}{},
},
},
Mapping: unregMapping,
},
},
patchVisitor: &testVisitor{
updatedInfos: []*resource.Info{
{
Namespace: "ns",
Name: "after",
Object: &unstructured.Unstructured{
Object: map[string]interface{}{
"apiVersion": "v1",
"kind": "Thingy",
"metadata": map[string]interface{}{
"uid": "12345",
"namespace": "ns",
"name": "after",
},
"spec": map[string]interface{}{},
},
},
Mapping: unregMapping,
},
},
},
results: &editResults{},
},
checkErr: expectedErr,
},
{
name: "kind-diff",
args: args{
originalInfos: []*resource.Info{
{
Namespace: "ns",
Name: "myname",
Object: &unstructured.Unstructured{
Object: map[string]interface{}{
"apiVersion": "v1",
"kind": "Thingy",
"metadata": map[string]interface{}{
"uid": "12345",
"namespace": "ns",
"name": "myname",
},
"spec": map[string]interface{}{},
},
},
Mapping: unregMapping,
},
},
patchVisitor: &testVisitor{
updatedInfos: []*resource.Info{
{
Namespace: "ns",
Name: "myname",
Object: &unstructured.Unstructured{
Object: map[string]interface{}{
"apiVersion": "v1",
"kind": "OtherThingy",
"metadata": map[string]interface{}{
"uid": "12345",
"namespace": "ns",
"name": "myname",
},
"spec": map[string]interface{}{},
},
},
Mapping: unregMapping,
},
},
},
results: &editResults{},
},
checkErr: expectedErr,
},
{
name: "apiver-diff",
args: args{
originalInfos: []*resource.Info{
{
Namespace: "ns",
Name: "myname",
Object: &unstructured.Unstructured{
Object: map[string]interface{}{
"apiVersion": "v1",
"kind": "Thingy",
"metadata": map[string]interface{}{
"uid": "12345",
"namespace": "ns",
"name": "myname",
},
"spec": map[string]interface{}{},
},
},
Mapping: unregMapping,
},
},
patchVisitor: &testVisitor{
updatedInfos: []*resource.Info{
{
Namespace: "ns",
Name: "myname",
Object: &unstructured.Unstructured{
Object: map[string]interface{}{
"apiVersion": "v1alpha1",
"kind": "Thingy",
"metadata": map[string]interface{}{
"uid": "12345",
"namespace": "ns",
"name": "myname",
},
"spec": map[string]interface{}{},
},
},
Mapping: unregMapping,
},
},
},
results: &editResults{},
},
checkErr: expectedErr,
},
}
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
o := &EditOptions{}
if err := o.visitToPatch(tt.args.originalInfos, tt.args.patchVisitor, tt.args.results); !tt.checkErr(err) {
t.Errorf("EditOptions.visitToPatch() %s error = %v", tt.name, err)
}
})
}
}