test tidy up

Author: tx3stn

model/autofix_test.go

@@ -9,7 +9,6 @@ import (
 	"go.yaml.in/yaml/v4"
 )
 
-// TestAutoFixIntegration tests basic auto-fix functionality
 func TestAutoFixIntegration(t *testing.T) {
 	spec := `
 openapi: 3.0.0
@@ -47,17 +46,14 @@ info:
 
 	result := ApplyRulesToRuleSet(execution)
 
-	// Should have fixed results
 	assert.Greater(t, len(result.FixedResults), 0, "Should have fixed some violations")
 
-	// Fixed results should have AutoFixed flag set
 	for _, r := range result.FixedResults {
 		assert.True(t, r.AutoFixed)
 		assert.Equal(t, "empty-description-autofix", r.RuleId)
 	}
 }
 
-// TestAutoFixDisabled tests that autofix doesn't run when disabled
 func TestAutoFixDisabled(t *testing.T) {
 	spec := `
 openapi: 3.0.0
@@ -81,13 +77,10 @@ info:
 	}
 
 	result := ApplyRulesToRuleSet(execution)
-
-	// Should have regular results, no fixed results
 	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
@@ -140,17 +133,14 @@ paths:
 
 	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)