mirror of
https://github.com/kubernetes/client-go.git
synced 2025-07-03 10:16:40 +00:00
Add test to detect unintentional changes in dynamic client requests.
Kubernetes-commit: a803c8034d60a81b0da71ea8631e27888a607476
This commit is contained in:
Ben Luddy
Kubernetes Publisher
parent
02e371e437
commit
b17c363481
232
dynamic/golden_test.go
Normal file
232
dynamic/golden_test.go
Normal file
@ -0,0 +1,232 @@
|
||||
package dynamic_test
|
||||
|
||||
import (
|
||||
"context"
|
||||
"fmt"
|
||||
"net"
|
||||
"net/http"
|
||||
"net/http/httptest"
|
||||
"net/http/httputil"
|
||||
"net/url"
|
||||
"os"
|
||||
"path/filepath"
|
||||
"testing"
|
||||
"time"
|
||||
|
||||
"github.com/google/go-cmp/cmp"
|
||||
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
||||
"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
|
||||
"k8s.io/apimachinery/pkg/runtime/schema"
|
||||
"k8s.io/apimachinery/pkg/types"
|
||||
"k8s.io/client-go/dynamic"
|
||||
"k8s.io/client-go/rest"
|
||||
)
|
||||
|
||||
func TestGoldenRequest(t *testing.T) {
|
||||
for _, tc := range []struct {
|
||||
name string
|
||||
do func(context.Context, dynamic.Interface) error
|
||||
}{
|
||||
{
|
||||
name: "create",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").Create(
|
||||
ctx,
|
||||
&unstructured.Unstructured{Object: map[string]interface{}{
|
||||
"metadata": map[string]interface{}{"name": "mips"},
|
||||
}},
|
||||
metav1.CreateOptions{FieldValidation: "warn"},
|
||||
"fin",
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "update",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").Update(
|
||||
ctx,
|
||||
&unstructured.Unstructured{Object: map[string]interface{}{
|
||||
"metadata": map[string]interface{}{"name": "mips"},
|
||||
}},
|
||||
metav1.UpdateOptions{FieldValidation: "warn"},
|
||||
"fin",
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "updatestatus",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").UpdateStatus(
|
||||
ctx,
|
||||
&unstructured.Unstructured{Object: map[string]interface{}{
|
||||
"metadata": map[string]interface{}{"name": "mips"},
|
||||
}},
|
||||
metav1.UpdateOptions{FieldValidation: "warn"},
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "delete",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
return client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").Delete(
|
||||
ctx,
|
||||
"mips",
|
||||
metav1.DeleteOptions{DryRun: []string{metav1.DryRunAll}},
|
||||
"fin",
|
||||
)
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "deletecollection",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
return client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").DeleteCollection(
|
||||
ctx,
|
||||
metav1.DeleteOptions{DryRun: []string{metav1.DryRunAll}},
|
||||
metav1.ListOptions{ResourceVersion: "42"},
|
||||
)
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "get",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").Get(
|
||||
ctx,
|
||||
"mips",
|
||||
metav1.GetOptions{ResourceVersion: "42"},
|
||||
"fin",
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "list",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").List(
|
||||
ctx,
|
||||
metav1.ListOptions{ResourceVersion: "42"},
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "watch",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").Watch(
|
||||
ctx,
|
||||
metav1.ListOptions{ResourceVersion: "42"},
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "patch",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").Patch(
|
||||
ctx,
|
||||
"mips",
|
||||
types.StrategicMergePatchType,
|
||||
[]byte("{\"foo\":\"bar\"}\n"),
|
||||
metav1.PatchOptions{FieldManager: "baz"},
|
||||
"fin",
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "apply",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").Apply(
|
||||
ctx,
|
||||
"mips",
|
||||
&unstructured.Unstructured{Object: map[string]interface{}{
|
||||
"metadata": map[string]interface{}{"name": "mips"},
|
||||
}},
|
||||
metav1.ApplyOptions{Force: true},
|
||||
"fin",
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "applystatus",
|
||||
do: func(ctx context.Context, client dynamic.Interface) error {
|
||||
_, err := client.Resource(schema.GroupVersionResource{Group: "flops", Version: "v1alpha1", Resource: "flips"}).Namespace("mops").ApplyStatus(
|
||||
ctx,
|
||||
"mips",
|
||||
&unstructured.Unstructured{Object: map[string]interface{}{
|
||||
"metadata": map[string]interface{}{"name": "mips"},
|
||||
}},
|
||||
metav1.ApplyOptions{Force: true},
|
||||
)
|
||||
return err
|
||||
},
|
||||
},
|
||||
} {
|
||||
t.Run(tc.name, func(t *testing.T) {
|
||||
handled := make(chan struct{})
|
||||
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
||||
defer close(handled)
|
||||
|
||||
got, err := httputil.DumpRequest(r, true)
|
||||
if err != nil {
|
||||
t.Fatal(err)
|
||||
}
|
||||
|
||||
path := filepath.Join("testdata", filepath.FromSlash(t.Name()))
|
||||
|
||||
if os.Getenv("UPDATE_DYNAMIC_CLIENT_FIXTURES") == "true" {
|
||||
err := os.WriteFile(path, got, os.FileMode(0755))
|
||||
if err != nil {
|
||||
t.Fatalf("failed to update fixture: %v", err)
|
||||
}
|
||||
}
|
||||
|
||||
want, err := os.ReadFile(path)
|
||||
if err != nil {
|
||||
t.Fatalf("failed to load fixture: %v", err)
|
||||
}
|
||||
if diff := cmp.Diff(got, want); diff != "" {
|
||||
t.Errorf("unexpected difference from expected bytes:\n%s", diff)
|
||||
}
|
||||
}))
|
||||
defer srv.Close()
|
||||
|
||||
client, err := dynamic.NewForConfig(&rest.Config{
|
||||
Host: "example.com",
|
||||
UserAgent: "TestGoldenRequest",
|
||||
Transport: &http.Transport{
|
||||
// The client will send a static Host header while always
|
||||
// connecting to the test server.
|
||||
DialContext: func(ctx context.Context, network string, addr string) (net.Conn, error) {
|
||||
u, err := url.Parse(srv.URL)
|
||||
if err != nil {
|
||||
return nil, fmt.Errorf("failed to parse test server url: %w", err)
|
||||
}
|
||||
return (&net.Dialer{}).DialContext(ctx, "tcp", u.Host)
|
||||
},
|
||||
},
|
||||
})
|
||||
if err != nil {
|
||||
t.Fatal(err)
|
||||
}
|
||||
|
||||
ctx, cancel := context.WithTimeout(context.Background(), time.Second)
|
||||
defer cancel()
|
||||
if err := tc.do(ctx, client); err != nil {
|
||||
// This test detects server-perceptible changes to the request. As
|
||||
// long as the server receives the expected request, a non-nil error
|
||||
// returned from a client method is not a failure.
|
||||
t.Logf("client returned non-nil error: %v", err)
|
||||
}
|
||||
|
||||
select {
|
||||
case <-handled:
|
||||
default:
|
||||
t.Fatal("no request received")
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
Loading…
Reference in New Issue
Block a user