fix: add more exec test for empty env

Author: gabyx

pkg/exec/command-ctx-builder.go

@@ -104,7 +104,7 @@ func (c CmdContextBuilder) Env(env ...string) CmdContextBuilder {
 	// NOTE:
 	// If we add here something and the default is set (.env == nil -> means use
 	// os.Environ()) -> we need to add it back!
-	if env != nil && c.cmdCtx.env == nil {
+	if c.cmdCtx.env == nil && env != nil {
 		c.cmdCtx.env = os.Environ()
 	}
 

pkg/exec/command-ctx_test.go

@@ -211,6 +211,15 @@ func TestCommandCtxPipeStdErr(t *testing.T) {
 	assert.Contains(t, output, "this-is-not-existing")
 }
 
+func TestCommandCtxEnvEmpty(t *testing.T) {
+	envs := []string{}
+	ctx := NewCmdCtxBuilder().Env(envs...).Build()
+
+	out, err := ctx.Get("env")
+	require.NoError(t, err)
+	assert.Contains(t, out, "PATH")
+}
+
 func TestCommandCtxEnv(t *testing.T) {
 	envs := []string{"A=banana", "B=monkey", "C=monkey"}
 	ctx := NewCmdCtxBuilder().Env(envs...).Build()