From 16c6df0c4525d86115248693fc01ad13be5ec7c7 Mon Sep 17 00:00:00 2001 From: Austin Abro Date: Thu, 26 Dec 2024 17:26:03 +0000 Subject: [PATCH] make a working test Signed-off-by: Austin Abro --- pkg/kube/client.go | 2 +- pkg/kube/client_test.go | 2 + pkg/kube/kwait_test.go | 213 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 216 insertions(+), 1 deletion(-) create mode 100644 pkg/kube/kwait_test.go diff --git a/pkg/kube/client.go b/pkg/kube/client.go index 83fc81de9..cee39e585 100644 --- a/pkg/kube/client.go +++ b/pkg/kube/client.go @@ -171,7 +171,7 @@ func (c *Client) SetWaiter(ws WaitStrategy) error { func New(getter genericclioptions.RESTClientGetter, waiter Waiter) *Client { if getter == nil { getter = genericclioptions.NewConfigFlags(true) - } + } factory := cmdutil.NewFactory(getter) if waiter == nil { sw, err := getStatusWatcher(factory) diff --git a/pkg/kube/client_test.go b/pkg/kube/client_test.go index fbf8fce18..7822479b2 100644 --- a/pkg/kube/client_test.go +++ b/pkg/kube/client_test.go @@ -746,6 +746,8 @@ func TestWaitJob(t *testing.T) { if err != nil { t.Fatal(err) } + c.waiter = w + c.waiter = w resources, err := c.Build(objBody(job), false) if err != nil { t.Fatal(err) diff --git a/pkg/kube/kwait_test.go b/pkg/kube/kwait_test.go new file mode 100644 index 000000000..1d9a69959 --- /dev/null +++ b/pkg/kube/kwait_test.go @@ -0,0 +1,213 @@ +/* +Copyright The Helm Authors. + +Licensed under the Apache License, Version 2.0 (the "License"); +you may not use this file except in compliance with the License. +You may obtain a copy of the License at + + http://www.apache.org/licenses/LICENSE-2.0 + +Unless required by applicable law or agreed to in writing, software +distributed under the License is distributed on an "AS IS" BASIS, +WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +See the License for the specific language governing permissions and +limitations under the License. +*/ + +package kube // import "helm.sh/helm/v3/pkg/kube" + +import ( + "errors" + "fmt" + "testing" + "time" + + "github.com/stretchr/testify/require" + v1 "k8s.io/api/core/v1" + "k8s.io/apimachinery/pkg/apis/meta/v1/unstructured" + "k8s.io/apimachinery/pkg/runtime" + "k8s.io/apimachinery/pkg/runtime/schema" + "k8s.io/apimachinery/pkg/util/yaml" + dynamicfake "k8s.io/client-go/dynamic/fake" + "k8s.io/kubectl/pkg/scheme" + "sigs.k8s.io/cli-utils/pkg/kstatus/watcher" + "sigs.k8s.io/cli-utils/pkg/testutil" +) + +var podCurrentYaml = ` +apiVersion: v1 +kind: Pod +metadata: + name: good-pod + namespace: ns +status: + conditions: + - type: Ready + status: "True" + phase: Running +` + +var podYaml = ` +apiVersion: v1 +kind: Pod +metadata: + name: in-progress-pod + namespace: ns +` + +func TestRunHealthChecks(t *testing.T) { + t.Parallel() + tests := []struct { + name string + podYamls []string + expectErrs []error + }{ + { + name: "Pod is ready", + podYamls: []string{podCurrentYaml}, + expectErrs: nil, + }, + { + name: "one of the pods never becomes ready", + podYamls: []string{podYaml, podCurrentYaml}, + // TODO, make this better + expectErrs: []error{errors.New("not all resources ready: context deadline exceeded: in-progress-pod: Pod not ready, status: InProgress")}, + }, + } + + for _, tt := range tests { + t.Run(tt.name, func(t *testing.T) { + t.Parallel() + c := newTestClient(t) + fakeClient := dynamicfake.NewSimpleDynamicClient(scheme.Scheme) + fakeMapper := testutil.NewFakeRESTMapper( + v1.SchemeGroupVersion.WithKind("Pod"), + ) + // ctx, cancel := context.WithTimeout(context.Background(), 500*time.Millisecond) + // defer cancel() + pods := []runtime.Object{} + statusWatcher := watcher.NewDefaultStatusWatcher(fakeClient, fakeMapper) + for _, podYaml := range tt.podYamls { + m := make(map[string]interface{}) + err := yaml.Unmarshal([]byte(podYaml), &m) + require.NoError(t, err) + pod := &unstructured.Unstructured{Object: m} + pods = append(pods, pod) + fmt.Println(pod.GetName()) + podGVR := schema.GroupVersionResource{Group: "", Version: "v1", Resource: "pods"} + err = fakeClient.Tracker().Create(podGVR, pod, pod.GetNamespace()) + require.NoError(t, err) + } + c.waiter = &kstatusWaiter{ + sw: statusWatcher, + log: c.Log, + } + + resourceList := ResourceList{} + for _, pod := range pods { + list, err := c.Build(objBody(pod), false) + if err != nil { + t.Fatal(err) + } + resourceList = append(resourceList, list...) + } + + err := c.Wait(resourceList, time.Second*5) + if tt.expectErrs != nil { + require.EqualError(t, err, errors.Join(tt.expectErrs...).Error()) + return + } + require.NoError(t, err) + }) + } +} + +// func TestWait1(t *testing.T) { +// podList := newPodList("starfish", "otter", "squid") + +// var created *time.Time + +// c := newTestClient(t) +// c.Factory.(*cmdtesting.TestFactory).ClientConfigVal = cmdtesting.DefaultClientConfig() +// c.Factory.(*cmdtesting.TestFactory).Client = &fake.RESTClient{ +// NegotiatedSerializer: unstructuredSerializer, +// Client: fake.CreateHTTPClient(func(req *http.Request) (*http.Response, error) { +// p, m := req.URL.Path, req.Method +// t.Logf("got request %s %s", p, m) +// switch { +// case p == "/api/v1/namespaces/default/pods/starfish" && m == "GET": +// pod := &podList.Items[0] +// if created != nil && time.Since(*created) >= time.Second*5 { +// pod.Status.Conditions = []v1.PodCondition{ +// { +// Type: v1.PodReady, +// Status: v1.ConditionTrue, +// }, +// } +// } +// return newResponse(200, pod) +// case p == "/api/v1/namespaces/default/pods/otter" && m == "GET": +// pod := &podList.Items[1] +// if created != nil && time.Since(*created) >= time.Second*5 { +// pod.Status.Conditions = []v1.PodCondition{ +// { +// Type: v1.PodReady, +// Status: v1.ConditionTrue, +// }, +// } +// } +// return newResponse(200, pod) +// case p == "/api/v1/namespaces/default/pods/squid" && m == "GET": +// pod := &podList.Items[2] +// if created != nil && time.Since(*created) >= time.Second*5 { +// pod.Status.Conditions = []v1.PodCondition{ +// { +// Type: v1.PodReady, +// Status: v1.ConditionTrue, +// }, +// } +// } +// return newResponse(200, pod) +// case p == "/namespaces/default/pods" && m == "POST": +// resources, err := c.Build(req.Body, false) +// if err != nil { +// t.Fatal(err) +// } +// now := time.Now() +// created = &now +// return newResponse(200, resources[0].Object) +// default: +// t.Fatalf("unexpected request: %s %s", req.Method, req.URL.Path) +// return nil, nil +// } +// }), +// } +// cs, err := c.getKubeClient() +// require.NoError(t, err) +// checker := NewReadyChecker(cs, c.Log, PausedAsReady(true)) +// w := &waiter{ +// c: checker, +// log: c.Log, +// timeout: time.Second * 30, +// } +// c.waiter = w +// resources, err := c.Build(objBody(&podList), false) +// if err != nil { +// t.Fatal(err) +// } +// result, err := c.Create(resources) +// if err != nil { +// t.Fatal(err) +// } +// if len(result.Created) != 3 { +// t.Errorf("expected 3 resource created, got %d", len(result.Created)) +// } + +// if err := c.Wait(resources, time.Second*30); err != nil { +// t.Errorf("expected wait without error, got %s", err) +// } + +// if time.Since(*created) < time.Second*5 { +// t.Errorf("expected to wait at least 5 seconds before ready status was detected, but got %s", time.Since(*created)) +// } +// }