add another test

Author: tx3stn

motor/autofix_integration_test.go

@@ -86,3 +86,73 @@ info:
 	assert.Greater(t, len(result.Results), 0)
 	assert.Equal(t, 0, len(result.FixedResults))
 }
+
+// TestAutoFixDoesNotAffectNonFixableViolations tests that autofix only affects fixable rules
+func TestAutoFixDoesNotAffectNonFixableViolations(t *testing.T) {
+	spec := `
+openapi: 3.0.0
+info:
+  title: Test API
+  version: 1.0.0
+  description: ""
+paths:
+  /test:
+    get:
+      summary: ""
+`
+
+	// Fixable rule
+	fixableRule := model.Rule{
+		Id:              "empty-description-fixable",
+		Message:         "Empty description found",
+		Given:           "$.info.description",
+		Severity:        model.SeverityWarn,
+		AutoFixFunction: "fixEmptyDescription",
+		Then: &model.RuleAction{Function: "truthy"},
+	}
+
+	// Non-fixable rule (no AutoFixFunction)
+	nonFixableRule := model.Rule{
+		Id:       "empty-summary-not-fixable",
+		Message:  "Empty summary found",
+		Given:    "$.paths..summary",
+		Severity: model.SeverityError,
+		Then: &model.RuleAction{Function: "truthy"},
+	}
+
+	emptyDescriptionFix := func(node *yaml.Node, document *yaml.Node, context *model.RuleFunctionContext) (*yaml.Node, error) {
+		if node.Value == "" {
+			node.Value = "TODO: Add description"
+		}
+		return node, nil
+	}
+
+	execution := &RuleSetExecution{
+		RuleSet: &rulesets.RuleSet{Rules: map[string]*model.Rule{
+			"empty-description-fixable":    &fixableRule,
+			"empty-summary-not-fixable":    &nonFixableRule,
+		}},
+		Spec:             []byte(spec),
+		SpecFileName:     "test.yaml",
+		ApplyAutoFixes:   true,
+		AutoFixFunctions: map[string]model.AutoFixFunction{"fixEmptyDescription": emptyDescriptionFix},
+	}
+
+	result := ApplyRulesToRuleSet(execution)
+
+	// Should have both fixed and unfixed results
+	assert.Greater(t, len(result.FixedResults), 0, "Should have fixed some violations")
+	assert.Greater(t, len(result.Results), 0, "Should have unfixed violations")
+
+	// Verify fixed results are only from fixable rule
+	for _, r := range result.FixedResults {
+		assert.True(t, r.AutoFixed)
+		assert.Equal(t, "empty-description-fixable", r.RuleId)
+	}
+
+	// Verify unfixed results are only from non-fixable rule
+	for _, r := range result.Results {
+		assert.False(t, r.AutoFixed)
+		assert.Equal(t, "empty-summary-not-fixable", r.RuleId)
+	}
+}