tests: improve test coverage for flags

Bearer · Dec 18, 2023 · 378965c · 378965c
1 parent af9cd82
commit 378965c
Show file tree

Hide file tree

Showing 5 changed files with 371 additions and 55 deletions.
diff --git a/internal/flag/options.go b/internal/flag/options.go
@@ -18,6 +18,8 @@ import (
 
 type Flags []flagtypes.FlagGroup
 
+const envPrefix = "bearer"
+
 var ErrInvalidScannerReportCombination = errors.New("invalid scanner argument; privacy report requires sast scanner")
 
 type flagGroupBase struct {
@@ -46,6 +48,26 @@ func addFlag(cmd *cobra.Command, flag *flagtypes.Flag) {
 	}
 }
 
+func BindViper(flag *flagtypes.Flag) error {
+	arguments := append(
+		[]string{
+			flag.ConfigName,
+			strings.ToUpper(
+				strings.Join(
+					[]string{
+						envPrefix,
+						strings.ReplaceAll(flag.Name, "-", "_"),
+					},
+					"_",
+				),
+			),
+		},
+		flag.EnvironmentVariables...,
+	)
+
+	return viper.BindEnv(arguments...)
+}
+
 func bind(cmd *cobra.Command, flag *flagtypes.Flag) error {
 	if flag == nil {
 		return nil
@@ -59,19 +81,7 @@ func bind(cmd *cobra.Command, flag *flagtypes.Flag) error {
 		return err
 	}
 
-	viper.AutomaticEnv()
-	viper.SetEnvPrefix("bearer")
-	viper.SetEnvKeyReplacer(strings.NewReplacer("-", "_", ".", "_"))
-	arguments := append(
-		[]string{flag.ConfigName},
-		flag.EnvironmentVariables...,
-	)
-
-	if err := viper.BindEnv(arguments...); err != nil {
-		return err
-	}
-
-	return nil
+	return BindViper(flag)
 }
 
 func argsToMap(flag *flagtypes.Flag) map[string]bool {

diff --git a/internal/flag/options_test.go b/internal/flag/options_test.go
@@ -2,32 +2,18 @@ package flag
 
 import (
 	"testing"
-
-	flagtypes "github.com/bearer/bearer/internal/flag/types"
-	"github.com/spf13/viper"
-	"github.com/stretchr/testify/assert"
 )
 
 func Test_getStringSlice(t *testing.T) {
-	type env struct {
-		key   string
-		value string
-	}
-	tests := []struct {
-		name      string
-		flag      *flagtypes.Flag
-		flagValue interface{}
-		env       env
-		want      []string
-	}{
+	testCases := []TestCase{
 		{
-			name:      "happy path. Empty value",
+			name:      "Empty value",
 			flag:      ScannerFlag,
 			flagValue: "",
 			want:      nil,
 		},
 		{
-			name:      "happy path. String value",
+			name:      "String value",
 			flag:      ScannerFlag,
 			flagValue: "sast,secrets",
 			want: []string{
@@ -36,7 +22,7 @@ func Test_getStringSlice(t *testing.T) {
 			},
 		},
 		{
-			name: "happy path. Slice value",
+			name: "Slice value",
 			flag: ScannerFlag,
 			flagValue: []string{
 				"sast",
@@ -48,11 +34,11 @@ func Test_getStringSlice(t *testing.T) {
 			},
 		},
 		{
-			name: "happy path. Env value",
+			name: "Env value",
 			flag: ScannerFlag,
-			env: env{
+			env: Env{
 				key:   "BEARER_SCANNER",
-				value: "secrets,sast",
+				value: "sast,secrets",
 			},
 			want: []string{
 				string(ScannerSAST),
@@ -61,21 +47,5 @@ func Test_getStringSlice(t *testing.T) {
 		},
 	}
 
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			if tt.env.key == "" {
-				viper.Set(tt.flag.ConfigName, tt.flagValue)
-			} else {
-				// err := viper.BindEnv(tt.flag.ConfigName, tt.env.key)
-				// assert.NoError(t, err)
-
-				t.Setenv(tt.env.key, tt.env.value)
-			}
-
-			sl := getStringSlice(tt.flag)
-			assert.Equal(t, tt.want, sl)
-
-			viper.Reset()
-		})
-	}
+	RunFlagTests(testCases, t)
 }