Improve Table component API

.changeset/silent-nails-hide.md

@@ -2,6 +2,4 @@
 '@sumup/circuit-ui': major
 ---
 
-Update the Table component's onSortBy method signature.
-
-The signature of the Table's custom onSortBy method changed. The `nextDirection` argument moved to the third position (`(index, nextDirection, rows)` 👉 `(index, rows, nextDirection)`) and can now be `undefined` (instead of `null` in the previous implementation).
+Changed the signature of the Table's custom onSortBy method. The `nextDirection` argument moved to the third position (`(index, nextDirection, rows)` 👉 `(index, rows, nextDirection)`) and can now be `undefined` (instead of `null` in the previous implementation).

packages/circuit-ui/components/Table/Table.spec.tsx

@@ -130,12 +130,15 @@ describe('Table', () => {
     it('should call the row click callback', () => {
       const onRowClickMock = jest.fn();
       const index = 0;
-      const { getAllByTestId } = render(
+      const { getAllByRole } = render(
         <Table onRowClick={onRowClickMock} headers={headers} rows={rows} />,
       );
 
+      const rowElements = getAllByRole('row');
+
       act(() => {
-        userEvent.click(getAllByTestId('table-row')[0]);
+        // rowElements[0] is the hidden first row
+        userEvent.click(rowElements[1]);
       });
 
       expect(onRowClickMock).toHaveBeenCalledTimes(1);
@@ -190,12 +193,14 @@ describe('Table', () => {
         const onSortByMock = jest.fn();
         const index = 0;
         const nextDirection = 'ascending';
-        const { getAllByTestId } = render(
+        const { getAllByRole } = render(
           <Table onSortBy={onSortByMock} headers={headers} rows={rows} />,
         );
 
+        const headerElements = getAllByRole('columnheader');
+
         act(() => {
-          userEvent.click(getAllByTestId('table-header')[0]);
+          userEvent.click(headerElements[0]);
         });
 
         expect(onSortByMock).toHaveBeenCalledTimes(1);