Function TestEngineConfigurationWithModel

pkg/workflow/engine_config_test.go:713–784

(t *testing.T)

}

func TestEngineConfigurationWithModel(t *testing.T) {
	tests := []struct {
		name           string
		engine         CodingAgentEngine
		engineConfig   *EngineConfig
		expectedModel  string
		expectedAPIKey string
	}{
		{
			name:   "Claude with model",
			engine: NewClaudeEngine(),
			engineConfig: &EngineConfig{
				ID:    "claude",
				Model: "claude-3-5-sonnet-20241022",
			},
			expectedModel:  "claude-3-5-sonnet-20241022",
			expectedAPIKey: "",
		},
		{
			name:   "Codex with model",
			engine: NewCodexEngine(),
			engineConfig: &EngineConfig{
				ID:    "codex",
				Model: "gpt-4o",
			},
			expectedModel:  "gpt-4o",
			expectedAPIKey: "",
		},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			workflowData := &WorkflowData{
				Name:         "test-workflow",
				EngineConfig: tt.engineConfig,
			}
			steps := tt.engine.GetExecutionSteps(workflowData, "test-log")

			if len(steps) == 0 {
				t.Fatalf("Expected at least one step, got none")
			}

			// Convert first step to YAML string for testing
			stepContent := strings.Join([]string(steps[0]), "\n")

			switch tt.engine.GetID() {
			case "claude":
				if tt.expectedModel != "" {
					// Claude passes model via native ANTHROPIC_MODEL env var
					expectedEnvLine := "ANTHROPIC_MODEL: " + tt.expectedModel
					if !strings.Contains(stepContent, expectedEnvLine) {
						t.Errorf("Expected step to contain env var for model %s, got step content:\n%s", tt.expectedModel, stepContent)
					}
					// Should NOT embed --model in the shell command
					if strings.Contains(stepContent, "--model "+tt.expectedModel) {
						t.Errorf("Model should not be embedded as --model flag, got step content:\n%s", stepContent)
					}
				}