Switching to --include-tests argument and defaulting to false

Author: regner

cmd/upgrade.go

@@ -85,7 +85,7 @@ func newChartCommand() *cobra.Command {
 	f.BoolVar(&diff.resetValues, "reset-values", false, "reset the values to the ones built into the chart and merge in any new values")
 	f.BoolVar(&diff.allowUnreleased, "allow-unreleased", false, "enables diffing of releases that are not yet deployed via Helm")
 	f.BoolVar(&diff.noHooks, "no-hooks", false, "disable diffing of hooks")
-	f.BoolVar(&diff.noTests, "no-tests", false, "disable diffing of the test hooks specifically")
+	f.BoolVar(&diff.includeTests, "include-tests", false, "enable the diffing of the helm test hooks")
 	f.BoolVar(&diff.devel, "devel", false, "use development versions, too. Equivalent to version '>0.0.0-0'. If --version is set, this is ignored.")
 	f.StringArrayVar(&diff.suppressedKinds, "suppress", []string{}, "allows suppression of the values listed in the diff output")
 	f.IntVarP(&diff.outputContext, "context", "C", -1, "output NUM lines of context around changes")
@@ -166,8 +166,8 @@ func (d *diffCmd) run() error {
 			currentSpecs = manifest.Parse(releaseResponse.Release.Manifest, releaseResponse.Release.Namespace)
 			newSpecs = manifest.Parse(upgradeResponse.Release.Manifest, upgradeResponse.Release.Namespace)
 		} else {
-			currentSpecs = manifest.ParseRelease(releaseResponse.Release, d.noTests)
-			newSpecs = manifest.ParseRelease(upgradeResponse.Release, d.noTests)
+			currentSpecs = manifest.ParseRelease(releaseResponse.Release, d.includeTests)
+			newSpecs = manifest.ParseRelease(upgradeResponse.Release, d.includeTests)
 		}
 	}
 

manifest/parse.go

@@ -61,10 +61,10 @@ func splitSpec(token string) (string, string) {
 	return "", ""
 }
 
-func ParseRelease(release *release.Release, noTests bool) map[string]*MappingResult {
+func ParseRelease(release *release.Release, includeTests bool) map[string]*MappingResult {
 	manifest := release.Manifest
 	for _, hook := range release.Hooks {
-		if noTests && isTestHook(hook.Events) {
+		if !includeTests && isTestHook(hook.Events) {
 			continue
 		}