Function TestShellJoinArgs

pkg/workflow/shell_test.go:157–198

(t *testing.T)

}

func TestShellJoinArgs(t *testing.T) {
	tests := []struct {
		name     string
		input    []string
		expected string
	}{
		{
			name:     "simple arguments",
			input:    []string{"git", "status"},
			expected: "git status",
		},
		{
			name:     "arguments with special characters",
			input:    []string{"--allow-tool", "shell(git add:*)", "--allow-tool", "shell(git commit:*)"},
			expected: "--allow-tool 'shell(git add:*)' --allow-tool 'shell(git commit:*)'",
		},
		{
			name:     "mixed arguments",
			input:    []string{"copilot", "--add-dir", "/tmp/gh-aw/", "--allow-tool", "shell(*.txt)"},
			expected: "copilot --add-dir /tmp/gh-aw/ --allow-tool 'shell(*.txt)'",
		},
		{
			name:     "prompt with pre-quoted instruction is now properly escaped by shellJoinArgs",
			input:    []string{"copilot", "--add-dir", "/tmp/gh-aw/", "--prompt", "\"$INSTRUCTION\""},
			expected: "copilot --add-dir /tmp/gh-aw/ --prompt '\"$INSTRUCTION\"'",
		},
		{
			name:     "allow-domains with GitHub Actions expression uses double quotes",
			input:    []string{"--allow-domains", "${{ env.MCP_ENV == 'staging' && env.MCP_URL_STAGING || env.MCP_URL_PROD }},errors.code.visualstudio.com"},
			expected: `--allow-domains "${{ env.MCP_ENV == 'staging' && env.MCP_URL_STAGING || env.MCP_URL_PROD }},errors.code.visualstudio.com"`,
		},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			result := shellJoinArgs(tt.input)
			if result != tt.expected {
				t.Errorf("shellJoinArgs(%q) = %q, expected %q", tt.input, result, tt.expected)
			}
		})
	}
}

func TestBuildDockerCommandWithExpandableVars(t *testing.T) {
	tests := []struct {