Should not print, if printer is not ready (#3037)

Signed-off-by: Felipe Avelar <[email protected]>

Author: felipe-avelar

internal/console/kics.go

@@ -3,7 +3,6 @@ package console
 import (
 	"context"
 	"fmt"
-	"strings"
 	"time"
 
 	"github.com/Checkmarx/kics/internal/console/printer"
@@ -128,10 +127,7 @@ func Execute() error {
 
 	if err := rootCmd.ExecuteContext(ctx); err != nil {
 		sentry.CaptureException(err)
-		if !(strings.HasPrefix(err.Error(), "unknown shorthand flag") ||
-			strings.HasPrefix(err.Error(), "unknown flag") ||
-			strings.HasPrefix(err.Error(), "unknown command") ||
-			strings.HasPrefix(err.Error(), "initialization error -")) {
+		if printer.IsInitialized() {
 			log.Err(err).Msg("Failed to run application")
 		}
 		return err

internal/console/printer/printer.go

@@ -57,7 +57,8 @@ var (
 	outFileLogger    interface{}
 	outConsoleLogger interface{}
 
-	loggerFile interface{}
+	loggerFile  interface{}
+	initialized bool
 )
 
 // SetupPrinter - configures stdout and log options with given FlagSet
@@ -101,10 +102,15 @@ func SetupPrinter(flags *pflag.FlagSet) error {
 	if err != nil {
 		return err
 	}
-
+	initialized = true
 	return nil
 }
 
+// IsInitialized returns true if printer is ready, false otherwise
+func IsInitialized() bool {
+	return initialized
+}
+
 func getFlagValue(flagName string, flags *pflag.FlagSet) bool {
 	v, _ := strconv.ParseBool(flags.Lookup(flagName).Value.String())
 	return v

internal/console/printer/printer_test.go

@@ -8,6 +8,7 @@ import (
 	"testing"
 
 	"github.com/rs/zerolog"
+	"github.com/stretchr/testify/require"
 )
 
 type args struct {
@@ -306,3 +307,7 @@ func TestPrinter_NoColor(t *testing.T) {
 		})
 	}
 }
+
+func TestPrinter_IsInitialized(t *testing.T) {
+	require.False(t, IsInitialized())
+}