feat(config): Allow avoiding reading default config file (#2227)

Signed-off-by: Dave Henderson <dhenderson@gmail.com>

1 files changed, 24 insertions, 4 deletions

diff --git a/internal/cmd/config_test.go b/internal/cmd/config_test.go
index ce59f027..760766fd 100644
--- a/internal/cmd/config_test.go
+++ b/internal/cmd/config_test.go
@@ -188,29 +188,49 @@ func TestPickConfigFile(t *testing.T) {
 	cmd.Flags().String("config", defaultConfigFile, "foo")
 
 	t.Run("default", func(t *testing.T) {
-		cf, req := pickConfigFile(cmd)
+		cf, req, skip := pickConfigFile(cmd)
 		assert.False(t, req)
+		assert.False(t, skip)
 		assert.Equal(t, defaultConfigFile, cf)
 	})
 
 	t.Run("GOMPLATE_CONFIG env var", func(t *testing.T) {
 		t.Setenv("GOMPLATE_CONFIG", "foo.yaml")
-		cf, req := pickConfigFile(cmd)
+		cf, req, skip := pickConfigFile(cmd)
 		assert.True(t, req)
+		assert.False(t, skip)
 		assert.Equal(t, "foo.yaml", cf)
 	})
 
 	t.Run("--config flag", func(t *testing.T) {
 		cmd.ParseFlags([]string{"--config", "config.file"})
-		cf, req := pickConfigFile(cmd)
+		cf, req, skip := pickConfigFile(cmd)
 		assert.True(t, req)
+		assert.False(t, skip)
 		assert.Equal(t, "config.file", cf)
 
 		t.Setenv("GOMPLATE_CONFIG", "ignored.yaml")
-		cf, req = pickConfigFile(cmd)
+		cf, req, skip = pickConfigFile(cmd)
 		assert.True(t, req)
+		assert.False(t, skip)
 		assert.Equal(t, "config.file", cf)
 	})
+
+	t.Run("--config flag with empty value should skip reading", func(t *testing.T) {
+		cmd.ParseFlags([]string{"--config", ""})
+		cf, req, skip := pickConfigFile(cmd)
+		assert.False(t, req)
+		assert.True(t, skip)
+		assert.Equal(t, "", cf)
+	})
+
+	t.Run("GOMPLATE_CONFIG env var with empty value should skip reading", func(t *testing.T) {
+		t.Setenv("GOMPLATE_CONFIG", "")
+		cf, req, skip := pickConfigFile(cmd)
+		assert.False(t, req)
+		assert.True(t, skip)
+		assert.Equal(t, "", cf)
+	})
 }
 
 func TestApplyEnvVars(t *testing.T) {