Move parse error checking to config file check function

cmd/functions.go

@@ -175,6 +175,12 @@ func checkConfigFile(requireToken bool) error {
 	if err != nil {
 		return fmt.Errorf("could not open the config file " + viper.ConfigFileUsed() + ". Have you called the login command? ")
 	}
+	// make sure the config file is parsable
+	err = viper.ReadInConfig()
+	if err != nil {
+		return fmt.Errorf("%w: could not parse the config file "+viper.ConfigFileUsed()+". Have you called the login command? ", err)
+	}
+
 	fmeflowUrl := viper.GetString("url")
 
 	// check the fme server URL is valid

cmd/login_v4_test.go

@@ -102,6 +102,7 @@ token: 5ba5e0fd15c2403bc8b2e3aa1dfb975ca2197fbe
 url: %s
 `, mainHttpServerLogin.URL),
 			},
+			omitConfig: true,
 		},
 		{
 			name:            "login with token",

cmd/root.go

@@ -102,10 +102,7 @@ func initConfig() {
 	//viper.AutomaticEnv() // read in environment variables that match
 
 	// If a config file is found, read it in.
-	err := viper.ReadInConfig()
-	if err != nil { // Handle errors reading the config file
-		cobra.CheckErr(err)
-	}
+	viper.ReadInConfig()
 
 }
 

cmd/testFunctions.go

@@ -111,7 +111,7 @@ func runTests(tcs []testCase, t *testing.T) {
 			}
 
 			// if a config file isn't specified, generate a random file and set the config file flag
-			if !configFlagExists(tc.args) {
+			if !configFlagExists(tc.args) && !tc.omitConfig {
 				f, err := os.CreateTemp("", "config-file*.yaml")
 				require.NoError(t, err)
 				defer os.Remove(f.Name()) // clean up