Merge pull request #1182 from fibonacci1729/master

fix(tiller): re-enable skipped test, golint, & gofmt all the things
pull/1176/head
Brian 8 years ago committed by GitHub
commit b982afd805

@ -225,10 +225,10 @@ func (s *releaseServer) GetReleaseContent(c ctx.Context, req *services.GetReleas
if req.Version <= 0 { if req.Version <= 0 {
rel, err := s.env.Releases.Deployed(req.Name) rel, err := s.env.Releases.Deployed(req.Name)
return &services.GetReleaseContentResponse{Release: rel}, err return &services.GetReleaseContentResponse{Release: rel}, err
} else { }
rel, err := s.env.Releases.Get(req.Name, req.Version) rel, err := s.env.Releases.Get(req.Name, req.Version)
return &services.GetReleaseContentResponse{Release: rel}, err return &services.GetReleaseContentResponse{Release: rel}, err
}
} }
func (s *releaseServer) UpdateRelease(c ctx.Context, req *services.UpdateReleaseRequest) (*services.UpdateReleaseResponse, error) { func (s *releaseServer) UpdateRelease(c ctx.Context, req *services.UpdateReleaseRequest) (*services.UpdateReleaseResponse, error) {

@ -646,8 +646,6 @@ func TestUninstallPurgeRelease(t *testing.T) {
} }
func TestUninstallPurgeDeleteRelease(t *testing.T) { func TestUninstallPurgeDeleteRelease(t *testing.T) {
t.Skip("TestUninstallPurgeDeleteRelease")
c := context.Background() c := context.Background()
rs := rsFixture() rs := rsFixture()
rs.env.Releases.Create(releaseStub()) rs.env.Releases.Create(releaseStub())
@ -667,7 +665,7 @@ func TestUninstallPurgeDeleteRelease(t *testing.T) {
} }
_, err2 := rs.UninstallRelease(c, req2) _, err2 := rs.UninstallRelease(c, req2)
if err2 != nil { if err2 != nil && err2.Error() != "'angry-panda' has no deployed releases" {
t.Errorf("Failed uninstall: %s", err2) t.Errorf("Failed uninstall: %s", err2)
} }
} }

Loading…
Cancel
Save