[fake] return the correct error message for missing releases

pull/4346/head
Mike Lundy 6 years ago
parent 327080161d
commit a6ab04be4d

@ -18,7 +18,6 @@ package helm // import "k8s.io/helm/pkg/helm"
import ( import (
"errors" "errors"
"fmt"
"math/rand" "math/rand"
"sync" "sync"
@ -27,6 +26,7 @@ import (
"k8s.io/helm/pkg/proto/hapi/release" "k8s.io/helm/pkg/proto/hapi/release"
rls "k8s.io/helm/pkg/proto/hapi/services" rls "k8s.io/helm/pkg/proto/hapi/services"
"k8s.io/helm/pkg/proto/hapi/version" "k8s.io/helm/pkg/proto/hapi/version"
storage "k8s.io/helm/pkg/storage/driver"
) )
// FakeClient implements Interface // FakeClient implements Interface
@ -117,7 +117,7 @@ func (c *FakeClient) DeleteRelease(rlsName string, opts ...DeleteOption) (*rls.U
} }
} }
return nil, fmt.Errorf("No such release: %s", rlsName) return nil, storage.ErrReleaseNotFound(rlsName)
} }
// GetVersion returns a fake version // GetVersion returns a fake version
@ -161,7 +161,7 @@ func (c *FakeClient) ReleaseStatus(rlsName string, opts ...StatusOption) (*rls.G
}, nil }, nil
} }
} }
return nil, fmt.Errorf("No such release: %s", rlsName) return nil, storage.ErrReleaseNotFound(rlsName)
} }
// ReleaseContent returns the configuration for the matching release name in the fake release client. // ReleaseContent returns the configuration for the matching release name in the fake release client.
@ -173,7 +173,7 @@ func (c *FakeClient) ReleaseContent(rlsName string, opts ...ContentOption) (resp
}, nil }, nil
} }
} }
return resp, fmt.Errorf("No such release: %s", rlsName) return resp, storage.ErrReleaseNotFound(rlsName)
} }
// ReleaseHistory returns a release's revision history. // ReleaseHistory returns a release's revision history.

Loading…
Cancel
Save