refactor(flat-table-checkbox): convert unit tests to RTL

[edleeks87]

Proposed behaviour

Converts FlatTableCheckbox, FlatTableHead, FlatTableBody, FlatTableBodyDraggable and Sort component unit tests to RTL

Current behaviour

FlatTableCheckbox, FlatTableHead, FlatTableBody, FlatTableBodyDraggable and Sort component unit tests are enzyme

Checklist

• Commits follow our style guide
• Unit tests added or updated if required
• Typescript d.ts file added or updated if required

QA

• Tested in provided StackBlitz sandbox/Storybook
• Add new Playwright test coverage if required
• Carbon implementation matches Design System/designs
• UI Tests GitHub check reviewed if required

Additional context

Testing instructions

[Parsium]

suggestion(non-blocking): for improved readability, what do you think of querying these elements directly rather than indexing an array? Not essential though.

Suggested change

	const dropTarget = tableRows[2];
	const elementToDrag = tableRows[0];
	const dropTarget = screen.getByRole("row", { name: "Row three" });
	const elementToDrag = screen.getByRole("row", { name: "Row one" })

[Parsium]

suggestion: what do you think of just checking the cell has the correct data attributes here? If we use the cell role, this should eliminate the need to check the cell is a <td> tag:

test("should render cell with expected data-* attributes", () => {
  render(
    <table>
      <tbody>
        <tr>
          <FlatTableCheckbox onChange={() => {}} data-role="ft-checkbox" />
        </tr>
      </tbody>
    </table>
  );
  const checkboxCell = screen.getByRole("cell");

  expect(checkboxCell).toHaveAttribute("data-role", "ft-checkbox");
  expect(checkboxCell).toHaveAttribute(
    "data-element",
    "flat-table-checkbox-cell"
  );
});

[Parsium]

suggestion: similarly to the cell test, what do you think of just testing that the data attributes are correct, if we utilise the columnheader role to find the <th> tag?:

[DipperTheDan]

Nothing more to add from me. Just address the comments left by @Parsium 👍🏻

[carbonci]

🎉 This PR is included in version 142.3.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀