feat(flat-table, flat-table-row): components can now be wrapped, subRows no longer need to be arrays FE-6050

[edleeks87]

fix #6219

• some tests were refactored to RTL as they now need to be async to allow the update to occur before the assertion

Proposed behaviour

Adds data tags to FlatTableCheckbox

Refactors FlatTable and sub-components to allow consumers greater flexibility with respect to
wrapping the components by removing code that relied on iterating over children and cloning.

Refactors FlatTableRow so that it is no longer required that an array is passed to the subRows
prop. Adds FlatTableRowContext and SubRowProvider as well as adding data-sticky-align attribute to FlatTableRowHeader and id attribute to all row children to support expandable and sticky column functionality.

Current behaviour

Components rely heavily on children iteration and using cloning to pass props down to achieve desired functionality.

Checklist

• Commits follow our style guide
• Related issues linked in commit messages if required
• Unit tests added or updated if required
• Storybook added or updated if required
• Typescript d.ts file added or updated if required

QA

• Tested in CodeSandbox/storybook
• Add new Cypress test coverage if required
• Carbon implementation matches Design System/designs
• UI Tests GitHub check reviewed if required

Additional context

Testing instructions

The following CodeSandbox is an example of the broken behaviour.
You can see the new behaviour by looking at the version in the comment by codesandbox[bot].

https://codesandbox.io/s/romantic-kilby-mhhzsp?file=/src/App.js
https://codesandbox.io/s/table-forked-lq389j?file=/src/Table.js -- children mutation

[cypress]

Passing run #17051 ↗︎

0

5492

41

0

0

Details:

feat(flat-table, flat-table-row): components can now be wrapped, subRows no long...
Project: carbon	Commit: bbb9d5a3b8
Status: Passed	Duration: 06:17 💡
Started: Jul 27, 2023 2:16 PM	Ended: Jul 27, 2023 2:23 PM

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit ea27a57:

Sandbox	Source
carbon-quickstart	Configuration
carbon-quickstart-typescript	Configuration
carbon-quickstart	PR
flamboyant-breeze-nt38yw	PR
table (forked)	PR

[robinzigmond]

comments submitted at halfway point to avoid losing them overnight - will complete the review tomorrow

EDIT: now complete

[carbonci]

🎉 This PR is included in version 120.1.0 🎉

The release is available on:

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

Your semantic-release bot 📦🚀