From 34b356ee5f5a0ab17607ff1f84cf3a3ad17b60d4 Mon Sep 17 00:00:00 2001 From: Bruno Miguel Custodio Date: Wed, 12 Dec 2018 12:07:45 +0000 Subject: [PATCH] mock: do not panic on unknown pods Signed-off-by: Bruno Miguel Custodio --- providers/mock/mock.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/providers/mock/mock.go b/providers/mock/mock.go index 55713bb04..b48208c01 100644 --- a/providers/mock/mock.go +++ b/providers/mock/mock.go @@ -190,8 +190,7 @@ func (p *MockProvider) GetPod(ctx context.Context, namespace, name string) (pod if pod, ok := p.pods[key]; ok { return pod, nil } - - return nil, nil + return nil, strongerrors.NotFound(fmt.Errorf("pod \"%s/%s\" is not known to the provider", namespace, name)) } // GetContainerLogs retrieves the logs of a container by name from the provider. @@ -255,7 +254,7 @@ func (p *MockProvider) GetPodStatus(ctx context.Context, namespace, name string) pod, err := p.GetPod(ctx, namespace, name) if err != nil { - return status, err + return nil, err } for _, container := range pod.Spec.Containers {