fix(test): make sure tests aren't impacted by user config

Modify readConfig to check whether running inside a test, and if so, to
not process the user config file (~/.config/cocli/config.yaml) or the
environment variables.

The config file passed with --config will still be processed in case
a test wants provide one.

Signed-off-by: setrofim <setrofim@gmail.com>

Author: setrofim

cmd/corimSubmit_test.go

@@ -18,6 +18,7 @@ func Test_CorimSubmitCmd_bad_server_url(t *testing.T) {
 	ctrl := gomock.NewController(t)
 	defer ctrl.Finish()
 
+
 	ms := mock_deps.NewMockISubmitter(ctrl)
 	cmd := NewCorimSubmitCmd(ms)
 

cmd/output.cbor

@@ -11,6 +11,7 @@ import (
 
 	"github.com/spf13/afero"
 	"github.com/spf13/cobra"
+	"github.com/spf13/pflag"
 	"github.com/spf13/viper"
 	"github.com/veraison/apiclient/auth"
 	_ "github.com/veraison/corim/profiles/tdx"
@@ -84,10 +85,15 @@ func initConfig() {
 }
 
 func readConfig(path string) (*viper.Viper, error) {
+	// We want to make sure that tests run in a controlled enviroment and
+	// can't be influenced by user's config file (if it exits) or user-set
+	// enviroment variables.
+	insideTest := pflag.Lookup("test.v") != nil
+
 	v := viper.GetViper()
 	if path != "" {
 		v.SetConfigFile(path)
-	} else {
+	} else if !insideTest {
 		wd, err := os.Getwd()
 		if err != nil {
 			return nil, err
@@ -102,8 +108,10 @@ func readConfig(path string) (*viper.Viper, error) {
 		v.SetConfigName("config")
 	}
 
-	v.SetEnvPrefix("cocli")
-	v.AutomaticEnv()
+	if !insideTest {
+		v.SetEnvPrefix("cocli")
+		v.AutomaticEnv()
+	}
 
 	err := v.ReadInConfig()
 	if errors.As(err, &viper.ConfigFileNotFoundError{}) {