tests: add unit tests to the code that loads config/flags

The config is also no longer dumped at startup. It used to do something
like:

dump, err := config.Dump()

I don't think there is a point in dumping the whole content of a file...
it's super verbose and not very useful. It should at least be gated
behind a --debug flag. So I propose to remove it for now.

Author: maelvls

cmd/agent.go

@@ -3,10 +3,9 @@ package cmd
 import (
 	"fmt"
 	"io/ioutil"
-	"os"
-	"time"
 
 	"github.com/jetstack/preflight/pkg/agent"
+	"github.com/jetstack/preflight/pkg/agent/config"
 	"github.com/jetstack/preflight/pkg/logs"
 	"github.com/jetstack/preflight/pkg/permissions"
 	"github.com/spf13/cobra"
@@ -37,16 +36,16 @@ var agentRBACCmd = &cobra.Command{
 	Long:  `Print RBAC string by reading GVRs`,
 	Run: func(cmd *cobra.Command, args []string) {
 
-		b, err := ioutil.ReadFile(agent.ConfigFilePath)
+		b, err := ioutil.ReadFile(agent.AgentCmdFlags.ConfigFilePath)
 		if err != nil {
 			logs.Log.Fatalf("Failed to read config file: %s", err)
 		}
-		config, err := agent.ParseConfig(b, false)
+		cfg, err := config.ParseConfig(b, false)
 		if err != nil {
 			logs.Log.Fatalf("Failed to parse config file: %s", err)
 		}
 
-		out := permissions.GenerateFullManifest(config.DataGatherers)
+		out := permissions.GenerateFullManifest(cfg.DataGatherers)
 		fmt.Print(out)
 	},
 }
@@ -55,119 +54,5 @@ func init() {
 	rootCmd.AddCommand(agentCmd)
 	agentCmd.AddCommand(agentInfoCmd)
 	agentCmd.AddCommand(agentRBACCmd)
-	agentCmd.PersistentFlags().StringVarP(
-		&agent.ConfigFilePath,
-		"agent-config-file",
-		"c",
-		"./agent.yaml",
-		"Config file location, default is `agent.yaml` in the current working directory.",
-	)
-	agentCmd.PersistentFlags().DurationVarP(
-		&agent.Period,
-		"period",
-		"p",
-		0,
-		"Override time between scans in the configuration file (given as XhYmZs).",
-	)
-	agentCmd.PersistentFlags().StringVarP(
-		&agent.CredentialsPath,
-		"credentials-file",
-		"k",
-		"",
-		"Location of the credentials file. For OAuth2 based authentication.",
-	)
-	agentCmd.PersistentFlags().BoolVarP(
-		&agent.VenafiCloudMode,
-		"venafi-cloud",
-		"",
-		false,
-		"Runs agent with parsing config (and credentials file if provided) in Venafi Cloud format if true.",
-	)
-	agentCmd.PersistentFlags().StringVarP(
-		&agent.ClientID,
-		"client-id",
-		"",
-		"",
-		"Venafi Cloud Service Account client ID. If you use this flag you don't need to use --venafi-cloud as it will assume you are authenticating against Venafi Cloud. Using this removes the need to use a credentials file with Venafi Cloud mode.",
-	)
-	agentCmd.PersistentFlags().StringVarP(
-		&agent.PrivateKeyPath,
-		"private-key-path",
-		"",
-		"/etc/venafi/agent/key/privatekey.pem",
-		"Venafi Cloud Service Account private key path.",
-	)
-	agentCmd.PersistentFlags().BoolVarP(
-		&agent.OneShot,
-		"one-shot",
-		"",
-		false,
-		"Runs agent a single time if true, or continously if false",
-	)
-	agentCmd.PersistentFlags().StringVarP(
-		&agent.OutputPath,
-		"output-path",
-		"",
-		"",
-		"Output file path, if used, it will write data to a local file instead of uploading to the preflight server",
-	)
-	agentCmd.PersistentFlags().StringVarP(
-		&agent.InputPath,
-		"input-path",
-		"",
-		"",
-		"Input file path, if used, it will read data from a local file instead of gathering data from clusters",
-	)
-	agentCmd.PersistentFlags().DurationVarP(
-		&agent.BackoffMaxTime,
-		"backoff-max-time",
-		"",
-		10*time.Minute,
-		"Max time for retrying failed data gatherers (given as XhYmZs).",
-	)
-	agentCmd.PersistentFlags().BoolVarP(
-		&agent.StrictMode,
-		"strict",
-		"",
-		false,
-		"Runs agent in strict mode. No retry attempts will be made for a missing data gatherer's data.",
-	)
-	agentCmd.PersistentFlags().StringVar(
-		&agent.APIToken,
-		"api-token",
-		os.Getenv("API_TOKEN"),
-		"Token used for authentication when API tokens are in use on the backend",
-	)
-	agentCmd.PersistentFlags().StringVar(
-		&agent.VenConnName,
-		"venafi-connection",
-		"",
-		"Name of the VenafiConnection to be used. Using this flag will enable the VenafiConnection mode.",
-	)
-	agentCmd.PersistentFlags().StringVar(
-		&agent.VenConnNS,
-		"venafi-connection-namespace",
-		"",
-		"Namespace of the VenafiConnection to be used. It is only useful when the VenafiConnection isn't in the same namespace as the agent. The field `allowReferencesFrom` must be present on the cross-namespace VenafiConnection for the agent to use it.",
-	)
-	agentCmd.PersistentFlags().StringVar(
-		&agent.InstallNS,
-		"install-namespace",
-		"",
-		"Namespace in which the agent is running. Only needed when running the agent outside of Kubernetes. Used for testing purposes.",
-	)
-	agentCmd.PersistentFlags().BoolVarP(
-		&agent.Profiling,
-		"enable-pprof",
-		"",
-		false,
-		"Enables the pprof profiling server on the agent (port: 6060).",
-	)
-	agentCmd.PersistentFlags().BoolVarP(
-		&agent.Prometheus,
-		"enable-metrics",
-		"",
-		false,
-		"Enables Prometheus metrics server on the agent (port: 8081).",
-	)
+	config.InitAgentCmdFlags(agentCmd, &agent.AgentCmdFlags)
 }