Merge pull request #31354 from TerryHowe/fix-flake-upgrade-test

fix: flake upgrade test
3 months ago · acb1b2c651
parent eec14e6869 a24318c7f1
commit acb1b2c651
1 changed files with 1 additions and 1 deletions
--- a/pkg/action/upgrade_test.go
+++ b/pkg/action/upgrade_test.go
@ -60,9 +60,9 @@ func TestUpgradeRelease_Success(t *testing.T) {

 	ctx, done := context.WithCancel(t.Context())
 	res, err := upAction.RunWithContext(ctx, rel.Name, buildChart(), vals)
-	done()
 	req.NoError(err)
 	is.Equal(res.Info.Status, release.StatusDeployed)
+	done()

 	// Detecting previous bug where context termination after successful release
 	// caused release to fail.