Fix printing test results on error (#218)

* Fix a regression that would cause test results no to be printed on error
* No longer print usage output on error

Signed-off-by: Reinhard Naegele <unguiculus@gmail.com>

ct/cmd/install.go

@@ -91,11 +91,12 @@ func install(cmd *cobra.Command, args []string) error {
 		fmt.Println(err)
 	}
 	results, err := testing.InstallCharts()
+	testing.PrintResults(results)
+
 	if err != nil {
 		return fmt.Errorf("Error installing charts: %s", err)
 	}
 
 	fmt.Println("All charts installed successfully")
-	testing.PrintResults(results)
 	return nil
 }

ct/cmd/lint.go

@@ -83,11 +83,12 @@ func lint(cmd *cobra.Command, args []string) error {
 		return err
 	}
 	results, err := testing.LintCharts()
+	testing.PrintResults(results)
+
 	if err != nil {
 		return fmt.Errorf("Error linting charts: %s", err)
 	}
 
 	fmt.Println("All charts linted successfully")
-	testing.PrintResults(results)
 	return nil
 }

ct/cmd/lintAndInstall.go

@@ -51,11 +51,12 @@ func lintAndInstall(cmd *cobra.Command, args []string) error {
 		return err
 	}
 	results, err := testing.LintAndInstallCharts()
+	testing.PrintResults(results)
+
 	if err != nil {
 		return fmt.Errorf("Error linting and installing charts: %s", err)
 	}
 
 	fmt.Println("All charts linted and installed successfully")
-	testing.PrintResults(results)
 	return nil
 }

ct/cmd/root.go

@@ -39,6 +39,7 @@ func NewRootCmd() *cobra.Command {
 			* all charts
 
 			in given chart directories.`),
+		SilenceUsage: true,
 	}
 
 	cmd.AddCommand(newLintCmd())