@agilant/toga-blox 1.0.40 → 1.0.41

package/dist/components/TableRow/TableRow.d.ts

@@ -1,15 +1,36 @@
+import React from "react";
+import { Row } from "react-table";
 import { DataWithUUID } from "../TableCell";
-import TableRowProps from ".";
-interface TableRowProps {
-    row: any;
-    prepareRow: (row: any) => void;
+/**
+ * The props for your reusable TableRow component.
+ *
+ * @template T - extends DataWithUUID
+ */
+export interface TableRowProps<T extends DataWithUUID> {
+    /** The row object from react-table */
+    row: Row<T>;
+    /** Function from react-table to prepare the row. */
+    prepareRow: (row: Row<T>) => void;
+    /** Optionally mark this row as active, comparing row.index to activeIndex. */
     activeIndex?: number;
+    /** Class to apply if the row is active. */
     activeRowColor?: string;
+    /** Globally enable/disable "trim" on cells. */
     globalTrimActive?: boolean;
+    /** A UUID for this row. */
     rowUuid?: string;
+    /** Column "inputs" or config array, if relevant to your TableCell. */
     columnInputs?: string[];
+    /** Classes for row hover styling. Defaults to "hover:bg-red-100 hover:cursor-pointer" */
     rowHoverClasses?: string;
-    onRowClick?: (index: number, rowUuid: string, event: React.MouseEvent) => void;
+    /**
+     * Called when the row is clicked.
+     *
+     * @param index - the row.index from react-table
+     * @param rowUuid - the row's UUID
+     * @param event - the click event on the <tr>
+     */
+    onRowClick?: (index: number, rowUuid: string, event: React.MouseEvent<HTMLTableRowElement, MouseEvent>) => void;
 }
-declare const TableRow: <T extends DataWithUUID>({ row, prepareRow, activeIndex, globalTrimActive, rowUuid, columnInputs, activeRowColor, rowHoverClasses, onRowClick, }: TableRowProps) => import("react/jsx-runtime").JSX.Element;
+declare const TableRow: <T extends DataWithUUID>({ row, prepareRow, activeIndex, globalTrimActive, rowUuid, columnInputs, activeRowColor, rowHoverClasses, onRowClick, }: TableRowProps<T>) => import("react/jsx-runtime").JSX.Element;
 export default TableRow;

package/dist/components/TableRow/TableRow.js

@@ -9,9 +9,7 @@ const TableRow = ({ row, prepareRow, activeIndex, globalTrimActive, rowUuid, col
         rowClasses += ` activeRow ${activeRowColor}`;
     }
     return (_jsx("tr", { "data-testid": "table-row", className: `border-b border-b-navy-200 ${rowHoverClasses} ${rowClasses}`, ...(row.getRowProps ? row.getRowProps() : {}), onClick: (event) => {
-            if (onRowClick) {
-                onRowClick(row.index, rowUuid || "", event);
-            }
+            onRowClick?.(row.index, rowUuid || "", event);
         }, children: row.cells.map((cell, index) => {
             const isLastCell = index === row.cells.length - 1;
             const cellProps = cell.getCellProps();

package/dist/components/TableRow/TableRow.test.js

@@ -4,16 +4,21 @@ import { render, screen, fireEvent } from "@testing-library/react";
 import TableRow from "./TableRow";
 describe("TableRow - Branch Coverage Tests", () => {
     const mockPrepareRow = vi.fn();
-    // Basic row object
     const baseMockRow = {
+        // REQUIRED BY Row<T>:
+        id: "mock-row-id",
         index: 0,
-        getRowProps: vi.fn(() => ({})),
+        original: { uuid: "mock-uuid" },
+        subRows: [],
         cells: [
             {
                 getCellProps: vi.fn(() => ({ key: "cell-0" })),
                 render: vi.fn(() => "Cell 0"),
             },
         ],
+        allCells: [],
+        values: {},
+        getRowProps: vi.fn(() => ({ key: "row-props-0" })),
     };
     it("covers isActive = true (row is active)", () => {
         render(_jsx("table", { children: _jsx("tbody", { children: _jsx(TableRow, { row: { ...baseMockRow, index: 5 }, prepareRow: mockPrepareRow, activeIndex: 5 }) }) }));
@@ -27,18 +32,20 @@ describe("TableRow - Branch Coverage Tests", () => {
         const mockOnRowClick = vi.fn();
         render(_jsx("table", { children: _jsx("tbody", { children: _jsx(TableRow, { row: baseMockRow, prepareRow: mockPrepareRow, onRowClick: mockOnRowClick, rowUuid: "some-uuid" }) }) }));
         fireEvent.click(screen.getByTestId("table-row"));
-        // Because rowUuid is defined, we expect 'some-uuid'
-        expect(mockOnRowClick).toHaveBeenCalledWith(baseMockRow.index, "some-uuid", expect.anything() // Synthetic event
+        expect(mockOnRowClick).toHaveBeenCalledWith(baseMockRow.index, // 0
+        "some-uuid", expect.anything() // Synthetic event
         );
     });
     it("covers onRowClick = defined + rowUuid = undefined (so calls empty string)", () => {
         const mockOnRowClick = vi.fn();
         render(_jsx("table", { children: _jsx("tbody", { children: _jsx(TableRow, { row: baseMockRow, prepareRow: mockPrepareRow, onRowClick: mockOnRowClick }) }) }));
         fireEvent.click(screen.getByTestId("table-row"));
-        expect(mockOnRowClick).toHaveBeenCalledWith(baseMockRow.index, "", expect.anything());
+        expect(mockOnRowClick).toHaveBeenCalledWith(baseMockRow.index, // 0
+        "", expect.anything());
     });
     it("covers onRowClick = NOT defined", () => {
         render(_jsx("table", { children: _jsx("tbody", { children: _jsx(TableRow, { row: baseMockRow, prepareRow: mockPrepareRow }) }) }));
+        // onRowClick not provided; just ensure no crash
         fireEvent.click(screen.getByTestId("table-row"));
     });
 });

package/dist/components/TableRow/types.d.ts

@@ -8,4 +8,5 @@ export interface TableRowProps<T extends object> {
     columnInputs?: string[];
     rowHoverClasses?: string;
     addRowToClickedRows?: (rowUuid: string) => void;
+    onRowClick?: (index: number, rowUuid: string, event: React.MouseEvent<HTMLTableRowElement, MouseEvent>) => void;
 }

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@agilant/toga-blox",
 	"private": false,
-	"version": "1.0.40",
+	"version": "1.0.41",
 	"description": "",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",