Rename issue-specific tests to generic names

- Rename TestIssue71NestedPerformance to TestNestedArrayPerformance
- Rename BenchmarkIssue71Nested100 to BenchmarkNestedArrayDecode100
- Rename BenchmarkIssue71Nested1000 to BenchmarkNestedArrayDecode1000
- Remove issue-specific comments and references
- Improve code style consistency with other tests in the file
- Add concise documentation explaining test purpose and race detector behavior
- Add [race=true/false] prefix to timing error messages for clarity

Author: Jecoms

decoder_test.go

@@ -1937,9 +1937,9 @@ func TestDecoder_InvalidSliceIndex(t *testing.T) {
 	Equal(t, v2.PostIds[1], "2")
 }
 
-// Issue #71: Nested structure decoding performance
-// https://github.com/go-playground/form/issues/71
-func TestIssue71NestedPerformance(t *testing.T) {
+// TestNestedArrayPerformance verifies decoding performance for nested structures.
+//   - Uses more lenient thresholds when race detector is enabled, since it can slow down execution.
+func TestNestedArrayPerformance(t *testing.T) {
 	type NestedBar struct {
 		Bazs   []string          `form:"bazs"`
 		Lookup map[string]string `form:"lookup"`
@@ -1955,43 +1955,34 @@ func TestIssue71NestedPerformance(t *testing.T) {
 
 	decoder := NewDecoder()
 
-	// Adjust thresholds based on race detector
-	// Race detector adds 5-10x overhead, especially on older Go versions
-	// Using smaller counts (10, 50, 200) since we know the bug scales exponentially.
-	// Without the fix, even 200 values would take 10+ seconds.
-	var thresholds []struct {
+	var thresholdTests []struct {
 		numValues int
 		maxTime   time.Duration
 	}
-	
+
 	if raceEnabled {
-		// Lenient thresholds for race detector mode (CI)
-		t.Log("Using lenient thresholds (race detector enabled)")
-		thresholds = []struct {
+		thresholdTests = []struct {
 			numValues int
 			maxTime   time.Duration
 		}{
 			{10, 50 * time.Millisecond},
-			{50, 500 * time.Millisecond},  // Without fix: ~5s, with fix: ~50-100ms
-			{200, 5 * time.Second},        // Without fix: ~80s+, with fix: ~500ms-2s
+			{50, 500 * time.Millisecond},
+			{200, 5 * time.Second},
 		}
 	} else {
-		// Strict thresholds for normal mode (local dev)
-		t.Log("Using strict thresholds (race detector disabled)")
-		thresholds = []struct {
+		thresholdTests = []struct {
 			numValues int
 			maxTime   time.Duration
 		}{
 			{10, 10 * time.Millisecond},
-			{50, 50 * time.Millisecond},   // Without fix: ~1s, with fix: ~5-10ms
-			{200, 500 * time.Millisecond}, // Without fix: ~16s, with fix: ~50-100ms
+			{50, 50 * time.Millisecond},
+			{200, 500 * time.Millisecond},
 		}
 	}
 
-	for _, tt := range thresholds {
+	for _, tt := range thresholdTests {
 		urlValues := make(url.Values)
 
-		// Generate test data with nested structure
 		for i := 0; i < tt.numValues; i++ {
 			urlValues.Add(fmt.Sprintf("foos[0].bars[%d].bazs", i), fmt.Sprintf("value%d", i))
 			urlValues.Add(fmt.Sprintf("foos[0].bars[%d].lookup[A]", i), fmt.Sprintf("lookupA%d", i))
@@ -2006,24 +1997,21 @@ func TestIssue71NestedPerformance(t *testing.T) {
 			t.Errorf("Decode error for %d values: %v", tt.numValues, err)
 		}
 
-		// Verify correct decoding
 		if len(req.Foos) != 1 {
 			t.Errorf("Expected 1 Foo, got %d", len(req.Foos))
 		}
 		if len(req.Foos[0].Bars) != tt.numValues {
 			t.Errorf("Expected %d Bars, got %d", tt.numValues, len(req.Foos[0].Bars))
 		}
 
-		t.Logf("%6d decoded values took: %v", tt.numValues, elapsed)
-
 		if elapsed > tt.maxTime {
-			t.Errorf("Decoding %d values took %v, expected less than %v (performance regression?)",
-				tt.numValues, elapsed, tt.maxTime)
+			t.Errorf("[race=%t] Decoding %d values took %v, expected less than %v",
+				raceEnabled, tt.numValues, elapsed, tt.maxTime)
 		}
 	}
 }
 
-func BenchmarkIssue71Nested100(b *testing.B) {
+func BenchmarkNestedArrayDecode100(b *testing.B) {
 	type NestedBar struct {
 		Bazs   []string          `form:"bazs"`
 		Lookup map[string]string `form:"lookup"`
@@ -2054,7 +2042,7 @@ func BenchmarkIssue71Nested100(b *testing.B) {
 	}
 }
 
-func BenchmarkIssue71Nested1000(b *testing.B) {
+func BenchmarkNestedArrayDecode1000(b *testing.B) {
 	type NestedBar struct {
 		Bazs   []string          `form:"bazs"`
 		Lookup map[string]string `form:"lookup"`