Fix tests related to double console calls

mui · Jun 13, 2021 · 6c5e504 · 6c5e504
1 parent 417f531
commit 6c5e504
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 5 deletions.
diff --git a/packages/material-ui/src/InputBase/InputBase.test.js b/packages/material-ui/src/InputBase/InputBase.test.js
@@ -483,9 +483,12 @@ describe('<InputBase />', () => {
               </div>
             </FormControl>,
           );
-        }).toErrorDev(
+        }).toErrorDev([
           'Material-UI: There are multiple `InputBase` components inside a FormControl.\nThis creates visual inconsistencies, only use one `InputBase`.',
-        );
+          // React 18 Strict Effects run mount effects twice
+          React.version.startsWith('18') &&
+            'Material-UI: There are multiple `InputBase` components inside a FormControl.\nThis creates visual inconsistencies, only use one `InputBase`.',
+        ]);
       });
 
       it('should not warn if only one input is rendered', () => {

diff --git a/packages/material-ui/src/ListItem/ListItem.test.js b/packages/material-ui/src/ListItem/ListItem.test.js
@@ -189,9 +189,12 @@ describe('<ListItem />', () => {
       it('should warn (but not error) with autoFocus with a function component with no content', () => {
         expect(() => {
           render(<ListItem component={NoContent} autoFocus />);
-        }).toErrorDev(
+        }).toErrorDev([
           'Material-UI: Unable to set focus to a ListItem whose component has not been rendered.',
-        );
+          // React 18 Strict Effects run mount effects twice
+          React.version.startsWith('18') &&
+            'Material-UI: Unable to set focus to a ListItem whose component has not been rendered.',
+        ]);
       });
     });
   });

diff --git a/packages/material-ui/src/Select/Select.test.js b/packages/material-ui/src/Select/Select.test.js
@@ -876,6 +876,10 @@ describe('<Select />', () => {
         }).toErrorDev([
           'Material-UI: The `value` prop must be an array',
           'The above error occurred in the <ForwardRef(SelectInput)> component',
+          // React 18 Strict Effects run mount effects twice
+          React.version.startsWith('18') && 'Material-UI: The `value` prop must be an array',
+          React.version.startsWith('18') &&
+            'The above error occurred in the <ForwardRef(SelectInput)> component',
         ]);
         const {
           current: { errors },

diff --git a/packages/material-ui/src/Tabs/Tabs.test.js b/packages/material-ui/src/Tabs/Tabs.test.js
@@ -322,8 +322,11 @@ describe('<Tabs />', () => {
           );
         }).toErrorDev([
           'You can provide one of the following values: 1, 3',
-          // StrictMode renders twice
+          // React 18 Strict Effects run mount effects twice
+          React.version.startsWith('18') && 'You can provide one of the following values: 1, 3',
           'You can provide one of the following values: 1, 3',
+          // React 18 Strict Effects run mount effects twice
+          React.version.startsWith('18') && 'You can provide one of the following values: 1, 3',
           'You can provide one of the following values: 1, 3',
           'You can provide one of the following values: 1, 3',
         ]);