@linzjs/step-ag-grid 7.3.1 → 7.3.2

package/package.json

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

package/src/components/gridForm/GridFormTextArea.tsx

@@ -50,7 +50,7 @@ export const GridFormTextArea = <RowType extends GridBaseRow>(props: GridFormTex
   });
 
   return popoverWrapper(
-    <div style={{ display: "flex", flexDirection: "row", width: props.width ?? 240 }}>
+    <div className={"subComponent"} style={{ display: "flex", flexDirection: "row", width: props.width ?? 240 }}>
       <TextAreaInput
         value={value}
         onChange={(e) => setValue(e.target.value)}

package/src/components/gridForm/GridFormTextInput.tsx

@@ -50,7 +50,10 @@ export const GridFormTextInput = <RowType extends GridBaseRow>(props: GridFormTe
   });
 
   return popoverWrapper(
-    <div style={{ display: "flex", flexDirection: "row", width: props.width ?? 240 }} className={"FormTest"}>
+    <div
+      style={{ display: "flex", flexDirection: "row", width: props.width ?? 240 }}
+      className={"FormTest subComponent"}
+    >
       <TextInputFormatted
         value={value}
         onChange={(e) => setValue(e.target.value)}

package/src/contexts/GridContext.tsx

@@ -15,7 +15,7 @@ export interface GridContextType {
   selectRowsByIdWithFlash: (rowIds?: number[]) => void;
   flashRows: (rowIds?: number[]) => void;
   flashRowsDiff: (updateFn: () => Promise<any>) => Promise<void>;
-  ensureRowVisible: (id: number) => void;
+  ensureRowVisible: (id: number | string) => boolean;
   ensureSelectedRowIsVisible: () => void;
   sizeColumnsToFit: () => void;
   stopEditing: () => void;
@@ -67,6 +67,7 @@ export const GridContext = createContext<GridContextType>({
   },
   ensureRowVisible: () => {
     console.error("no context provider for ensureRowVisible");
+    return true;
   },
   ensureSelectedRowIsVisible: () => {
     console.error("no context provider for ensureSelectedRowIsVisible");

package/src/contexts/GridContextProvider.tsx

@@ -200,10 +200,12 @@ export const GridContextProvider = (props: GridContextProps): ReactElement => {
     );
   };
 
-  const ensureRowVisible = (id: number): void => {
-    gridApiOp((gridApi) => {
+  const ensureRowVisible = (id: number | string): boolean => {
+    return gridApiOp((gridApi) => {
       const node = gridApi.getRowNode(`${id}`);
-      node && gridApi.ensureNodeVisible(node);
+      if (!node) return false;
+      gridApi.ensureNodeVisible(node);
+      return true;
     });
   };
 

package/src/utils/testUtil.ts

@@ -42,10 +42,13 @@ export const cellContains = async (
   text: string | RegExp,
   within?: HTMLElement,
 ) => {
-  return await waitFor(async () => {
-    const row = await findRow(rowId, within);
-    return await findQuick({ tagName: `[col-id='${colId}']`, text }, row);
-  });
+  return await waitFor(
+    async () => {
+      const row = await findRow(rowId, within);
+      return await findQuick({ tagName: `[col-id='${colId}']`, text }, row);
+    },
+    { timeout: 10000 },
+  );
 };
 
 export const selectCell = async (rowId: string | number, colId: string, within?: HTMLElement): Promise<void> => {
@@ -129,12 +132,14 @@ export const clickMultiSelectOption = async (value: string): Promise<void> => {
 export const typeOtherInput = async (value: string): Promise<void> => {
   const openMenu = await findOpenMenu();
   const otherInput = await findQuick({ classes: ".subComponent", child: { tagName: "input[type='text']" } }, openMenu);
+  userEvent.clear(otherInput);
   userEvent.type(otherInput, value);
 };
 
 export const typeOtherTextArea = async (value: string): Promise<void> => {
   const openMenu = await findOpenMenu();
   const otherTextArea = await findQuick({ classes: ".subComponent", child: { tagName: "textarea" } }, openMenu);
+  userEvent.clear(otherTextArea);
   userEvent.type(otherTextArea, value);
 };