You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This issue is to implement visual regression testing (VRT) for the following components by adding baseline Percy snapshot(s) inside a *-test.e2e.js file. More info and high level steps to complete can be found in the umbrella issue, #8911
At first glance this appears to be a large issue, though I think the complexity can be reduced significantly by rendering subsets of table "types" like what we have in the storybook. This way we can cover a large portion of data table functionality in fewer snapshots. I don't know that we need to individually snapshot every component.
default
with checkmark columns
with overflowmenu
with batch actions
with expansion
with filtering
with selection
with sorting
Though here is the full component list that these snapshots need to cover in some way:
DataTable
DataTableSkeleton
Table
TableActionList
TableBatchAction
TableBatchActions
TableBody
TableCell
TableContainer
TableExpandedRow
TableExpandHeader
TableExpandRow
TableHead
TableHeader
TableRow
TableSelectAll
TableSelectRow
TableToolbar
TableToolbarAction
TableToolbarContent
TableToolbarMenu
TableToolbarSearch
The text was updated successfully, but these errors were encountered:
This issue is to implement visual regression testing (VRT) for the following components by adding baseline Percy snapshot(s) inside a
*-test.e2e.js
file. More info and high level steps to complete can be found in the umbrella issue, #8911At first glance this appears to be a large issue, though I think the complexity can be reduced significantly by rendering subsets of table "types" like what we have in the storybook. This way we can cover a large portion of data table functionality in fewer snapshots. I don't know that we need to individually snapshot every component.
Though here is the full component list that these snapshots need to cover in some way:
The text was updated successfully, but these errors were encountered: