feat: migrate CLI to use cobra

cmd/validator/commands/flags.go

@@ -0,0 +1,26 @@
+package cmd
+
+import (
+	validator "github.com/Boeing/config-file-validator/cmd/validator"
+	"github.com/spf13/cobra"
+)
+
+func CmdFlags(cmd *cobra.Command) {
+	cmd.PersistentFlags().
+		IntVar(&validator.Flags.Depth, "depth", 0, "Depth of recursion for the provided search paths. Set depth to 0 to disable recursive path traversal.")
+	cmd.PersistentFlags().
+		StringVar(&validator.Flags.ExcludeDirs, "exclude-dirs", "", "Subdirectories to exclude when searching for configuration files")
+	cmd.PersistentFlags().
+		StringVar(&validator.Flags.ExcludeFileTypes, "exclude-file-types", "", "A comma separated list of file types to ignore")
+	cmd.PersistentFlags().StringVar(&validator.Flags.Output, "output", "", "Destination to a file to output results")
+	cmd.PersistentFlags().
+		StringVar(&validator.Flags.ReportType, "reporter", "standard", "Format of the printed report. Options are standard and json")
+	cmd.PersistentFlags().
+		StringVar(&validator.Flags.GroupOutput, "groupby", "", "Group output by filetype, directory, pass-fail. Supported for Standard and JSON reports")
+	cmd.PersistentFlags().
+		StringVar(&validator.Flags.SearchPath, "search_path", ".", "search_path: The search path on the filesystem for configuration files. Defaults to the current working directory if no search_path provided.")
+}
+
+func init() {
+	CmdFlags(rootCmd)
+}

cmd/validator/commands/flags_test.go

@@ -0,0 +1,68 @@
+package cmd
+
+import (
+	"bytes"
+	"fmt"
+	"strings"
+	"testing"
+
+	cmd "github.com/Boeing/config-file-validator/cmd/validator"
+	"github.com/spf13/cobra"
+)
+
+func ExecuteTestHelper(t *testing.T, c *cobra.Command, args ...string) (string, error) {
+	t.Helper()
+
+	buf := new(bytes.Buffer)
+	c.SetOut(buf)
+	c.SetErr(buf)
+	c.SetArgs(args)
+
+	err := c.Execute()
+	return strings.TrimSpace(buf.String()), err
+}
+
+func Test_flags(t *testing.T) {
+	// We manipulate the Args to set them up for the testcases
+	cases := []struct {
+		Name         string
+		Args         []string
+		ExpectedExit int
+	}{
+		{"blank", []string{}, 0},
+		{"negative depth set", []string{"--depth", "-1", "--reporter", "standard"}, 1},
+		{"flags set, wrong reporter", []string{"--exclude-dirs", "subdir", "--reporter", "wrong"}, 1},
+		{"flags set, json reporter", []string{"--exclude-dirs", "subdir", "--reporter", "json"}, 0},
+		{"flags set, junit reported", []string{"--exclude-dirs", "subdir", "--reporter", "junit"}, 0},
+		{"bad path", []string{"--search_path", "/path/does/not/exit"}, 1},
+		//{"exclude file types set", []string{"--exclude-file-types", "json"}, 0},
+		{"multiple paths", []string{"../../../test/fixtures/subdir/good.json", "../../../test/fixtures/good.json"}, 0},
+		{"output set", []string{"--output", "../../../test/output", "--reporter", "json"}, 0},
+		{"empty string output set", []string{"--output", "", "--reporter", "json", "."}, 0},
+		{"wrong output set", []string{"--output", "/path/not/exist", "--reporter", "json", "."}, 1},
+		{"incorrect group", []string{"--groupby", "badgroup"}, 1},
+		{"correct group", []string{"--groupby", "directory"}, 0},
+	}
+
+	var exitStatus int
+
+	for _, tc := range cases {
+		fmt.Printf("Testing args: %v = %v\n", tc.Name, tc.Args)
+		root := &cobra.Command{
+			Use: "root",
+			Run: func(c *cobra.Command, args []string) {
+				exitStatus = cmd.ExecRoot(c)
+			},
+		}
+		CmdFlags(root)
+
+		_, err := ExecuteTestHelper(t, root, tc.Args...)
+		if err != nil {
+			t.Error(err)
+		}
+
+		if tc.ExpectedExit != exitStatus {
+			t.Errorf("Wrong exit code, expected: %v, got: %v", tc.ExpectedExit, exitStatus)
+		}
+	}
+}

cmd/validator/commands/root.go

@@ -0,0 +1,25 @@
+package cmd
+
+import (
+	"fmt"
+	"os"
+
+	validator "github.com/Boeing/config-file-validator/cmd/validator"
+	"github.com/spf13/cobra"
+)
+
+// rootCmd command configuration and setup
+var rootCmd = &cobra.Command{
+	Use:   "validator",
+	Short: "Cross Platform tool to validate configuration files",
+	Run: func(cmd *cobra.Command, args []string) {
+		os.Exit(validator.ExecRoot(cmd))
+	},
+}
+
+func Execute() {
+	if err := rootCmd.Execute(); err != nil {
+		fmt.Fprintln(os.Stderr, err)
+		os.Exit(1)
+	}
+}

version.go → cmd/validator/commands/version.go

@@ -1,6 +1,10 @@
-package configfilevalidator
+package cmd
 
-import "fmt"
+import (
+	"fmt"
+
+	"github.com/spf13/cobra"
+)
 
 // Version information set by link flags during build. We fall back to these sane
 // default values when not provided
@@ -18,9 +22,28 @@ func (v Version) String() string {
 	return fmt.Sprintf("validator version %v", v.Version)
 }
 
+// SetVersion set the version
+func SetVersion(v string) {
+	version = v
+}
+
 // GetVersion returns the version information
 func GetVersion() Version {
 	return Version{
 		Version: version,
 	}
 }
+
+func init() {
+	rootCmd.AddCommand(versionCmd)
+}
+
+// versionCmd command configuration and setup
+var versionCmd = &cobra.Command{
+	Use:   "version",
+	Short: "Version prints the release version of validator",
+	Long:  ``,
+	Run: func(cmd *cobra.Command, args []string) {
+		fmt.Println(GetVersion())
+	},
+}

cmd/validator/commands/version_test.go

@@ -0,0 +1,33 @@
+package cmd
+
+import (
+	"testing"
+
+	cmd "github.com/Boeing/config-file-validator/cmd/validator"
+	"github.com/spf13/cobra"
+)
+
+func TestFlagVersion(t *testing.T) {
+	var exitStatus int
+	expectedExit := 0
+	root := &cobra.Command{
+		Use: "root",
+		Run: func(c *cobra.Command, args []string) {
+			exitStatus = cmd.ExecRoot(c)
+		}}
+
+	SetVersion("testing")
+	root.AddCommand(versionCmd)
+
+	args := []string{"version"}
+
+	_, err := ExecuteTestHelper(t, root, args...)
+	if err != nil {
+		t.Error(err)
+	}
+
+	if expectedExit != exitStatus {
+		t.Errorf("Wrong exit code, expected: %v, got: %v", expectedExit, exitStatus)
+	}
+
+}