From 9b27eb83fef553a1e82b56ae1630451283825404 Mon Sep 17 00:00:00 2001 From: Sargun Dhillon Date: Tue, 13 Aug 2019 10:30:02 -0700 Subject: [PATCH] Make mock_test follow the aformentioned documentation --- node/mock_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/node/mock_test.go b/node/mock_test.go index 76189167c..3d58a0caa 100644 --- a/node/mock_test.go +++ b/node/mock_test.go @@ -180,7 +180,7 @@ func (p *mockV0Provider) GetPod(ctx context.Context, namespace, name string) (po } if pod, ok := p.pods.Load(key); ok { - return pod.(*v1.Pod), nil + return pod.(*v1.Pod).DeepCopy(), nil } return nil, errdefs.NotFoundf("pod \"%s/%s\" is not known to the provider", namespace, name) } @@ -195,7 +195,7 @@ func (p *mockV0Provider) GetPodStatus(ctx context.Context, namespace, name strin return nil, err } - return &pod.Status, nil + return pod.Status.DeepCopy(), nil } // GetPods returns a list of all pods known to be "running". @@ -205,7 +205,7 @@ func (p *mockV0Provider) GetPods(ctx context.Context) ([]*v1.Pod, error) { var pods []*v1.Pod p.pods.Range(func(key, pod interface{}) bool { - pods = append(pods, pod.(*v1.Pod)) + pods = append(pods, pod.(*v1.Pod).DeepCopy()) return true })