diff --git a/packages/ibm-products/src/components/Datagrid/Datagrid.test.js b/packages/ibm-products/src/components/Datagrid/Datagrid.test.js index 026185e153..97ed284303 100644 --- a/packages/ibm-products/src/components/Datagrid/Datagrid.test.js +++ b/packages/ibm-products/src/components/Datagrid/Datagrid.test.js @@ -326,7 +326,7 @@ const EmptyUsage = ({ emptyStateType, ...rest } = {}) => { DatagridPagination, }); - return ; + return ; }; const TenThousandEntries = ({ ...rest } = {}) => { @@ -406,6 +406,8 @@ const ExpandedRow = ({ ...rest } = {}) => { data, ExpandedRowContentComponent: expansionRenderer, expandedContentHeight: 95, + expanderButtonTitleExpanded: 'Collapse row', + expanderButtonTitleCollapsed: 'Expand row', }, useExpandedRow ); @@ -609,6 +611,8 @@ const NestedTable = ({ ...rest } = {}) => { data, ExpandedRowContentComponent: expansionRenderer, expandedContentHeight: (nestedDatagridState.state.pageSize + 2) * 48 + 1, // +2 for header and pagination + expanderButtonTitleExpanded: 'Collapse row', + expanderButtonTitleCollapsed: 'Expand row', }, useExpandedRow ); @@ -1012,7 +1016,7 @@ describe(componentName, () => { }); it('renders a Batch Actions Table', async () => { - render(); + render(); const alertMock = jest.spyOn(window, 'alert'); @@ -1206,7 +1210,7 @@ describe(componentName, () => { }); it('Infinite Scroll', async () => { - render(); + render(); expect( screen @@ -1247,7 +1251,7 @@ describe(componentName, () => { }); it('With Pagination', async () => { - render(); + render(); expect( document.getElementById(`${carbon.prefix}-pagination-select-4`) @@ -1340,7 +1344,7 @@ describe(componentName, () => { } it('Is Hover On Row', async () => { - render(); + render(); completeHoverOperation(1); completeHoverOperation(5); @@ -1348,7 +1352,7 @@ describe(componentName, () => { //Disables Selected Rows it('Renders Disable Select Row', async () => { - render(); + render(); const alertMock = jest.spyOn(window, 'alert'); @@ -1538,7 +1542,7 @@ describe(componentName, () => { } it('Hide Select All', async () => { - render(); + render(); hideSelectAll(2); @@ -1599,7 +1603,7 @@ describe(componentName, () => { }); it('Nested Table', async () => { - render(); + render(); const firstRowExpander = screen.getAllByLabelText('Expand row')[0]; const firstRow = screen.getAllByRole('row')[1]; fireEvent.click(firstRowExpander); @@ -1673,7 +1677,7 @@ describe(componentName, () => { } it('Radio Select', async () => { - render(); + render(); radioSelectButton(1, 1); radioSelectButton(1, 4); @@ -1687,9 +1691,7 @@ describe(componentName, () => { it('Select Items In All Pages', async () => { const alertMock = jest.spyOn(window, 'alert'); - render( - - ); + render(); // check if 10 rows are rendered on initial load var numRows = screen .getByRole('table') @@ -1943,7 +1945,7 @@ describe(componentName, () => { }); it('Selectable Row', async () => { - render(); + render(); fireEvent.click( screen @@ -2036,12 +2038,7 @@ describe(componentName, () => { }, ]; const columns = [...columnsWithoutSticky, ...defaultHeader.slice(2)]; - render( - - ); + render(); const customizeColumnsButton = screen.getByLabelText('Customize columns'); fireEvent.click(customizeColumnsButton); @@ -2060,7 +2057,7 @@ describe(componentName, () => { }); it('Customizing Columns', async () => { - render(); + render(); const alertMock = jest.spyOn(window, 'alert'); @@ -2324,6 +2321,8 @@ describe(componentName, () => { }); const sharedFilterGridProps = { + expanderButtonTitleExpanded: 'Collapse row', + expanderButtonTitleCollapsed: 'Expand row', gridTitle: 'Data table title', gridDescription: 'Additional information if needed', useDenseHeader: false, diff --git a/packages/ibm-products/src/components/Datagrid/useRowExpander.js b/packages/ibm-products/src/components/Datagrid/useRowExpander.js index dab360876d..585adf4620 100644 --- a/packages/ibm-products/src/components/Datagrid/useRowExpander.js +++ b/packages/ibm-products/src/components/Datagrid/useRowExpander.js @@ -41,11 +41,10 @@ const useRowExpander = (hooks) => { row?.onClick?.(row, event); lastExpandedRowIndex.current = row.id; }, + title: null, }; - const { - expanderButtonTitleExpanded = 'Collapse row', - expanderButtonTitleCollapsed = 'Expand row', - } = tempState?.current || {}; + const { expanderButtonTitleExpanded, expanderButtonTitleCollapsed } = + tempState?.current || {}; const expanderTitle = row.isExpanded ? expanderButtonTitleExpanded : expanderButtonTitleCollapsed;