Rename consolidateFields to consolidateV320StatsFields for clarity

Signed-off-by: Sean Chittenden <sean.chittenden@crowdstrike.com>

Author: sean-

opensearchapi/api_cat-indices_consolidation_test.go

@@ -32,7 +32,7 @@ func TestCatIndicesConsolidateFields(t *testing.T) {
 		}
 
 		// Call consolidateFields method
-		resp.consolidateFields()
+		resp.consolidateV320StatsFields()
 
 		// Stable fields should remain unchanged
 		assert.Equal(t, 10, *resp.Indices[0].PrimarySearchStartreeQueryCurrent)
@@ -55,7 +55,7 @@ func TestCatIndicesConsolidateFields(t *testing.T) {
 			},
 		}
 
-		resp.consolidateFields()
+		resp.consolidateV320StatsFields()
 
 		// Should fallback to V32 values when stable fields are nil
 		assert.Equal(t, 20, *resp.Indices[0].PrimarySearchStartreeQueryCurrent)
@@ -81,7 +81,7 @@ func TestCatIndicesConsolidateFields(t *testing.T) {
 			},
 		}
 
-		resp.consolidateFields()
+		resp.consolidateV320StatsFields()
 
 		// Stable fields should take precedence
 		assert.Equal(t, 10, *resp.Indices[0].PrimarySearchStartreeQueryCurrent)
@@ -107,7 +107,7 @@ func TestCatIndicesConsolidateFields(t *testing.T) {
 			},
 		}
 
-		resp.consolidateFields()
+		resp.consolidateV320StatsFields()
 
 		// Should use V32 values when stable fields have zero/empty values
 		assert.Equal(t, 20, *resp.Indices[0].PrimarySearchStartreeQueryCurrent)
@@ -138,7 +138,7 @@ func TestCatIndicesConsolidateFields(t *testing.T) {
 			},
 		}
 
-		resp.consolidateFields()
+		resp.consolidateV320StatsFields()
 
 		// Check first index (stable fields)
 		assert.Equal(t, 10, *resp.Indices[0].PrimarySearchStartreeQueryCurrent)
@@ -165,7 +165,7 @@ func TestCatIndicesConsolidateFields(t *testing.T) {
 		}
 
 		// Should not panic
-		resp.consolidateFields()
+		resp.consolidateV320StatsFields()
 
 		// All consolidated fields should remain nil
 		assert.Nil(t, resp.Indices[0].PrimarySearchStartreeQueryCurrent)

opensearchapi/api_cat.go

@@ -131,7 +131,7 @@ func (c catClient) Indices(ctx context.Context, req *CatIndicesReq) (*CatIndices
 	}
 
 	// Consolidate version-specific fields to maintain backward compatibility
-	data.consolidateFields()
+	data.consolidateV320StatsFields()
 
 	return &data, nil
 }