fix: Allow double and float types to be rollupable

packages/iris-grid/src/sidebar/RollupRows.test.tsx

@@ -0,0 +1,21 @@
+import { type dh as DhType } from '@deephaven/jsapi-types';
+import { TestUtils } from '@deephaven/test-utils';
+import RollupRows from './RollupRows';
+
+it('should allow all column types to be groupable', () => {
+  function testType(type: string, expected = true) {
+    const column = TestUtils.createMockProxy<DhType.Column>({ type });
+    expect(RollupRows.isGroupable(column)).toBe(expected);
+  }
+
+  testType('string');
+  testType('int');
+  testType('long');
+  testType('float');
+  testType('double');
+  testType('java.lang.String');
+  testType('java.lang.Integer');
+  testType('java.lang.Long');
+  testType('java.math.BigDecimal', false);
+  testType('java.math.BigInteger', false);
+});

packages/iris-grid/src/sidebar/RollupRows.tsx

@@ -115,7 +115,10 @@ class RollupRows extends Component<RollupRowsProps, RollupRowsState> {
   }
 
   static isGroupable(column: dh.Column): boolean {
-    return !TableUtils.isDecimalType(column.type);
+    return (
+      !TableUtils.isBigDecimalType(column.type) &&
+      !TableUtils.isBigIntegerType(column.type)
+    );
   }
 
   constructor(props: RollupRowsProps) {

tests/table-operations.spec.ts

@@ -470,6 +470,18 @@ test('rollup rows and aggregrate columns', async ({ page }) => {
     await expect(page.locator('.iris-grid-column')).toHaveScreenshot();
   });
 
+  await test.step('Rollup a double column', async () => {
+    const doubleColumn = page.getByRole('button', {
+      name: 'Double',
+      exact: true,
+    });
+    expect(doubleColumn).toBeTruthy();
+    await doubleColumn.dblclick();
+
+    await waitForLoadingDone(page);
+    await expect(page.locator('.iris-grid-column')).toHaveScreenshot();
+  });
+
   await test.step('Aggregate columns', async () => {
     await page.getByText('Constituents').click();
     await page.getByText('Non-Aggregated Columns').click();