From c81e90eb370b1e7506c0449157ba026c29b34af8 Mon Sep 17 00:00:00 2001
From: Robert Knight <robertknight@gmail.com>
Date: Mon, 20 May 2024 12:51:48 +0100
Subject: [PATCH] Ensure all `<tr>` elements have `tbody`, `thead` or `tfoot`
 parents

The latest Preact release warns about `<tr>` elements that have a `<table>` as
their direct parent. See https://github.com/preactjs/preact/pull/4375.
---
 src/components/data/test/TableCell-test.js | 20 ++++++++++++--------
 src/components/data/test/TableRow-test.js  | 20 ++++++++++++--------
 2 files changed, 24 insertions(+), 16 deletions(-)

diff --git a/src/components/data/test/TableCell-test.js b/src/components/data/test/TableCell-test.js
index 0f53d689e..b2d13fddb 100644
--- a/src/components/data/test/TableCell-test.js
+++ b/src/components/data/test/TableCell-test.js
@@ -13,11 +13,13 @@ describe('TableCell', () => {
     return mount(
       <TableContext.Provider value={tableContextValue}>
         <table>
-          <TableSectionContext.Provider value={contextValue}>
-            <tr>
-              <TableCell {...props}>{props.children ?? 'Content'}</TableCell>
-            </tr>
-          </TableSectionContext.Provider>
+          <tbody>
+            <TableSectionContext.Provider value={contextValue}>
+              <tr>
+                <TableCell {...props}>{props.children ?? 'Content'}</TableCell>
+              </tr>
+            </TableSectionContext.Provider>
+          </tbody>
         </table>
       </TableContext.Provider>,
     );
@@ -28,9 +30,11 @@ describe('TableCell', () => {
   const contentFn = (Component, props = {}) => {
     return mount(
       <table>
-        <tr>
-          <Component {...props} />
-        </tr>
+        <tbody>
+          <tr>
+            <Component {...props} />
+          </tr>
+        </tbody>
       </table>,
     );
   };
diff --git a/src/components/data/test/TableRow-test.js b/src/components/data/test/TableRow-test.js
index e5a421caf..7605ff5ac 100644
--- a/src/components/data/test/TableRow-test.js
+++ b/src/components/data/test/TableRow-test.js
@@ -13,11 +13,13 @@ describe('TableRow', () => {
     return mount(
       <TableContext.Provider value={tableContextValue}>
         <table>
-          <TableSectionContext.Provider value={sectionContextValue}>
-            <TableRow {...props}>
-              <td>Cell content</td>
-            </TableRow>
-          </TableSectionContext.Provider>
+          <tbody>
+            <TableSectionContext.Provider value={sectionContextValue}>
+              <TableRow {...props}>
+                <td>Cell content</td>
+              </TableRow>
+            </TableSectionContext.Provider>
+          </tbody>
         </table>
       </TableContext.Provider>,
     );
@@ -28,9 +30,11 @@ describe('TableRow', () => {
   const contentFn = (Component, props = {}) => {
     return mount(
       <table>
-        <Component {...props}>
-          <td>Cell content</td>
-        </Component>
+        <tbody>
+          <Component {...props}>
+            <td>Cell content</td>
+          </Component>
+        </tbody>
       </table>,
     );
   };