fix: Do no aggregate results for CSV downloads from AG Grid raw records table (#36247)

superset-frontend/plugins/plugin-chart-ag-grid-table/src/buildQuery.ts

@@ -28,6 +28,7 @@ import {
   QueryFormOrderBy,
   QueryMode,
   QueryObject,
+  QueryObjectExtras,
   removeDuplicates,
   PostProcessingRule,
   BuildQuery,
@@ -322,7 +323,9 @@ const buildQuery: BuildQuery<TableChartFormData> = (
       };
 
       orderedColumns = reorderByColumnOrder(columns) as typeof columns;
-      orderedMetrics = reorderByColumnOrder(metrics || []) as typeof metrics;
+      orderedMetrics = metrics
+        ? (reorderByColumnOrder(metrics) as typeof metrics)
+        : metrics;
     }
 
     let queryObject = {
@@ -473,7 +476,7 @@ const buildQuery: BuildQuery<TableChartFormData> = (
                 ? `${queryObject.extras.having} AND ${havingClause}`
                 : havingClause,
             }),
-          };
+          } as QueryObjectExtras;
         }
       }
     }

superset-frontend/plugins/plugin-chart-ag-grid-table/test/buildQuery.test.ts

@@ -475,4 +475,28 @@ describe('plugin-chart-ag-grid-table', () => {
       expect(query.columns).toEqual(['state', 'city']);
     });
   });
+
+  describe('buildQuery - metrics handling in different query modes', () => {
+    test('should not include metrics in raw records mode', () => {
+      const query = buildQuery({
+        viz_type: VizType.Table,
+        datasource: '11__table',
+        query_mode: QueryMode.Raw,
+        all_columns: ['state', 'city'],
+      }).queries[0];
+
+      expect(query.metrics).toBeUndefined();
+    });
+
+    test('should set metrics to empty array in aggregate mode when no metrics specified', () => {
+      const query = buildQuery({
+        viz_type: VizType.Table,
+        datasource: '11__table',
+        query_mode: QueryMode.Aggregate,
+        groupby: ['state'],
+      }).queries[0];
+
+      expect(query.metrics).toEqual([]);
+    });
+  });
 });