@linzjs/step-ag-grid 7.11.0 → 7.11.1

package/package.json

@@ -2,7 +2,7 @@
   "name": "@linzjs/step-ag-grid",
   "repository": "github:linz/step-ag-grid.git",
   "license": "MIT",
-  "version": "7.11.0",
+  "version": "7.11.1",
   "keywords": [
     "aggrid",
     "ag-grid",

package/src/components/Grid.tsx

@@ -285,7 +285,12 @@ export const Grid = (params: GridProps): JSX.Element => {
   return (
     <div
       data-testid={params["data-testid"]}
-      className={clsx("Grid-container", "ag-theme-alpine", staleGrid && "Grid-sortIsStale")}
+      className={clsx(
+        "Grid-container",
+        "ag-theme-alpine",
+        staleGrid && "Grid-sortIsStale",
+        gridReady && params.rowData && "Grid-ready",
+      )}
     >
       {params.quickFilter && (
         <div className="Grid-quickFilter">

package/src/utils/testUtil.ts

@@ -1,7 +1,6 @@
 import { act, waitFor, within } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 import { findQuick, getAllQuick, getMatcher, getQuick, queryQuick } from "./testQuick";
-import { wait } from "./util";
 
 export const countRows = async (within?: HTMLElement): Promise<number> => {
   return getAllQuick({ tagName: `div[row-id]:not(:empty)` }, within).length;
@@ -22,9 +21,15 @@ const _selectRow = async (
 ): Promise<void> => {
   await act(async () => {
     const row = await findRow(rowId, within);
-    const isSelected = !row.className.includes("ag-row-selected");
+    const isSelected = row.className.includes("ag-row-selected");
     if (select === "toggle" || (select === "select" && !isSelected) || (select === "deselect" && isSelected)) {
-      userEvent.click(row);
+      const cell = await findCell(rowId, "selection", within);
+      userEvent.click(cell);
+      await waitFor(async () => {
+        const row = await findRow(rowId, within);
+        const nowSelected = row.className.includes("ag-row-selected");
+        if (nowSelected == isSelected) throw `Row ${rowId} won't select`;
+      });
     }
   });
 };
@@ -63,11 +68,12 @@ export const selectCell = async (rowId: string | number, colId: string, within?:
 };
 
 export const editCell = async (rowId: number | string, colId: string, within?: HTMLElement): Promise<void> => {
+  await selectRow(rowId, within);
   await act(async () => {
     const cell = await findCell(rowId, colId, within);
     userEvent.dblClick(cell);
   });
-  await findOpenMenu();
+  await waitFor(findOpenMenu);
 };
 
 const findOpenMenu = async (): Promise<HTMLElement> => findQuick({ classes: ".szh-menu--state-open" });
@@ -89,14 +95,14 @@ export const findMenuOption = async (menuOptionText: string | RegExp): Promise<H
       }
       return menuOption;
     },
-    { timeout: 10000 },
+    { timeout: 5000 },
   );
 };
 
 export const clickMenuOption = async (menuOptionText: string | RegExp): Promise<void> => {
+  const menuOption = await findMenuOption(menuOptionText);
   await act(async () => {
-    const menuOption = await findMenuOption(menuOptionText);
-    menuOption && userEvent.click(menuOption);
+    userEvent.click(menuOption);
   });
 };
 
@@ -107,7 +113,6 @@ export const openAndClickMenuOption = async (
   within?: HTMLElement,
 ): Promise<void> => {
   await editCell(rowId, colId, within);
-  await wait(100);
   await clickMenuOption(menuOptionText);
 };