fix: Allow double and float types to be rollupable (#2311)

- We had this restriction in the UI, but the engine can actually handle
it
- Tested with doubles, floats
- Tested with the following tables:
```python
from deephaven import empty_table

doubles = empty_table(1_000_000).update_view(["Group = i*10.4", "N = (i % 347)*0.1", "M = (i % 29)*10.2", "Value=i*2.4", "Weight=i*4.2"])
floats = empty_table(1_000_000).update_view(["Group = i*10.4", "N = (float)(i % 347)*0.1", "M = (float)(i % 29)*10.2"
, "Value=(float)i*2.4", "Weight=(float)i*4.2"])
```
- Updated e2e tests and unit tests
- Fixes #2295 
- For DH-18030

Author: mofojed

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

@@ -110,7 +110,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

@@ -462,6 +462,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();