cmd/ie/commands/test: Propagate error instead of exiting

Signed-off-by: Jose Blanquicet <josebl@microsoft.com>

Author: blanquicet

cmd/ie/commands/test.go

@@ -1,11 +1,10 @@
 package commands
 
 import (
+	"errors"
 	"fmt"
-	"os"
 
 	"github.com/Azure/InnovationEngine/internal/engine"
-	"github.com/Azure/InnovationEngine/internal/logging"
 	"github.com/spf13/cobra"
 )
 
@@ -22,11 +21,10 @@ var testCommand = &cobra.Command{
 	Use:   "test [markdown file]",
 	Args:  cobra.MinimumNArgs(1),
 	Short: "Test document commands against it's expected outputs.",
-	Run: func(cmd *cobra.Command, args []string) {
+	RunE: func(cmd *cobra.Command, args []string) error {
 		markdownFile := args[0]
 		if markdownFile == "" {
-			cmd.Help()
-			return
+			return errors.New("no markdown file specified")
 		}
 
 		verbose, _ := cmd.Flags().GetBool("verbose")
@@ -39,9 +37,7 @@ var testCommand = &cobra.Command{
 			CorrelationId: "",
 		})
 		if err != nil {
-			logging.GlobalLogger.Errorf("Error creating engine %s", err)
-			fmt.Printf("Error creating engine %s", err)
-			os.Exit(1)
+			return fmt.Errorf("creating engine: %w", err)
 		}
 
 		scenario, err := engine.CreateScenarioFromMarkdown(
@@ -50,11 +46,13 @@ var testCommand = &cobra.Command{
 			nil,
 		)
 		if err != nil {
-			logging.GlobalLogger.Errorf("Error creating scenario %s", err)
-			fmt.Printf("Error creating engine %s", err)
-			os.Exit(1)
+			return fmt.Errorf("creating scenario: %w", err)
 		}
 
-		innovationEngine.TestScenario(scenario)
+		if err := innovationEngine.TestScenario(scenario); err != nil {
+			return fmt.Errorf("testing scenario: %w", err)
+		}
+
+		return nil
 	},
 }