Remove react-test-renderer

michaldudak · michaldudak · commit a612ad1b199d · 2024-07-03T11:23:09.000+02:00
diff --git a/packages/mui-base/src/AlertDialog/Backdrop/AlertDialogBackdrop.test.tsx b/packages/mui-base/src/AlertDialog/Backdrop/AlertDialogBackdrop.test.tsx
@@ -15,7 +15,6 @@ describe('<AlertDialog.Backdrop />', () => {
         </AlertDialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has role="presentation"', async () => {
diff --git a/packages/mui-base/src/AlertDialog/Close/AlertDialogClose.test.tsx b/packages/mui-base/src/AlertDialog/Close/AlertDialogClose.test.tsx
@@ -15,6 +15,5 @@ describe('<AlertDialog.Close />', () => {
         </AlertDialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/AlertDialog/Description/AlertDialogDescription.test.tsx b/packages/mui-base/src/AlertDialog/Description/AlertDialogDescription.test.tsx
@@ -15,6 +15,5 @@ describe('<AlertDialog.Description />', () => {
         </AlertDialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/AlertDialog/Popup/AlertDialogPopup.test.tsx b/packages/mui-base/src/AlertDialog/Popup/AlertDialogPopup.test.tsx
@@ -16,7 +16,6 @@ describe('<AlertDialog.Popup />', () => {
         </AlertDialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('should have role="alertdialog"', async () => {
diff --git a/packages/mui-base/src/AlertDialog/Title/AlertDialogTitle.test.tsx b/packages/mui-base/src/AlertDialog/Title/AlertDialogTitle.test.tsx
@@ -15,6 +15,5 @@ describe('<AlertDialog.Title />', () => {
         </AlertDialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/AlertDialog/Trigger/AlertDialogTrigger.test.tsx b/packages/mui-base/src/AlertDialog/Trigger/AlertDialogTrigger.test.tsx
@@ -15,6 +15,5 @@ describe('<AlertDialog.Trigger />', () => {
         </AlertDialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Checkbox/Indicator/CheckboxIndicator.test.tsx b/packages/mui-base/src/Checkbox/Indicator/CheckboxIndicator.test.tsx
@@ -24,7 +24,6 @@ describe('<Checkbox.Indicator />', () => {
         <CheckboxContext.Provider value={testContext}>{node}</CheckboxContext.Provider>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('should not render indicator by default', () => {
diff --git a/packages/mui-base/src/Dialog/Backdrop/DialogBackdrop.test.tsx b/packages/mui-base/src/Dialog/Backdrop/DialogBackdrop.test.tsx
@@ -16,7 +16,6 @@ describe('<Dialog.Backdrop />', () => {
         </Dialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has role="presentation"', () => {
diff --git a/packages/mui-base/src/Dialog/Close/DialogClose.test.tsx b/packages/mui-base/src/Dialog/Close/DialogClose.test.tsx
@@ -15,6 +15,5 @@ describe('<Dialog.Close />', () => {
         </Dialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Dialog/Description/DialogDescription.test.tsx b/packages/mui-base/src/Dialog/Description/DialogDescription.test.tsx
@@ -15,6 +15,5 @@ describe('<Dialog.Description />', () => {
         </Dialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Dialog/Popup/DialogPopup.test.tsx b/packages/mui-base/src/Dialog/Popup/DialogPopup.test.tsx
@@ -15,7 +15,6 @@ describe('<Dialog.Popup />', () => {
         </Dialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   describe('prop: keepMounted', () => {
diff --git a/packages/mui-base/src/Dialog/Title/DialogTitle.test.tsx b/packages/mui-base/src/Dialog/Title/DialogTitle.test.tsx
@@ -15,6 +15,5 @@ describe('<Dialog.Title />', () => {
         </Dialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Dialog/Trigger/DialogTrigger.test.tsx b/packages/mui-base/src/Dialog/Trigger/DialogTrigger.test.tsx
@@ -15,6 +15,5 @@ describe('<Dialog.Trigger />', () => {
         </Dialog.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/NumberField/Decrement/NumberFieldDecrement.test.tsx b/packages/mui-base/src/NumberField/Decrement/NumberFieldDecrement.test.tsx
@@ -28,7 +28,6 @@ describe('<NumberField.Decrement />', () => {
         <NumberFieldContext.Provider value={testContext}>{node}</NumberFieldContext.Provider>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has decrease label', () => {
diff --git a/packages/mui-base/src/NumberField/Group/NumberFieldGroup.test.tsx b/packages/mui-base/src/NumberField/Group/NumberFieldGroup.test.tsx
@@ -27,7 +27,6 @@ describe('<NumberField.Group />', () => {
         <NumberFieldContext.Provider value={testContext}>{node}</NumberFieldContext.Provider>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has role prop', () => {
diff --git a/packages/mui-base/src/NumberField/Increment/NumberFieldIncrement.test.tsx b/packages/mui-base/src/NumberField/Increment/NumberFieldIncrement.test.tsx
@@ -28,7 +28,6 @@ describe('<NumberField.Increment />', () => {
         <NumberFieldContext.Provider value={testContext}>{node}</NumberFieldContext.Provider>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has increase label', () => {
diff --git a/packages/mui-base/src/NumberField/Input/NumberFieldInput.test.tsx b/packages/mui-base/src/NumberField/Input/NumberFieldInput.test.tsx
@@ -27,7 +27,6 @@ describe('<NumberField.Input />', () => {
         <NumberFieldContext.Provider value={testContext}>{node}</NumberFieldContext.Provider>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has textbox role', () => {
diff --git a/packages/mui-base/src/NumberField/ScrubArea/NumberFieldScrubArea.test.tsx b/packages/mui-base/src/NumberField/ScrubArea/NumberFieldScrubArea.test.tsx
@@ -36,7 +36,6 @@ describe('<NumberField.ScrubArea />', () => {
         <NumberFieldContext.Provider value={testContext}>{node}</NumberFieldContext.Provider>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has presentation role', () => {
diff --git a/packages/mui-base/src/NumberField/ScrubAreaCursor/NumberFieldScrubAreaCursor.test.tsx b/packages/mui-base/src/NumberField/ScrubAreaCursor/NumberFieldScrubAreaCursor.test.tsx
@@ -34,7 +34,6 @@ describe('<NumberField.ScrubAreaCursor />', () => {
         <NumberFieldContext.Provider value={testContext}>{node}</NumberFieldContext.Provider>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('has presentation role', () => {
diff --git a/packages/mui-base/src/Popover/Arrow/PopoverArrow.test.tsx b/packages/mui-base/src/Popover/Arrow/PopoverArrow.test.tsx
@@ -18,6 +18,5 @@ describe('<Popover.Arrow />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Popover/Backdrop/PopoverBackdrop.test.tsx b/packages/mui-base/src/Popover/Backdrop/PopoverBackdrop.test.tsx
@@ -16,6 +16,5 @@ describe('<Popover.Backdrop />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Popover/Close/PopoverClose.test.tsx b/packages/mui-base/src/Popover/Close/PopoverClose.test.tsx
@@ -20,7 +20,6 @@ describe('<Popover.Close />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('should close popover when clicked', async () => {
diff --git a/packages/mui-base/src/Popover/Description/PopoverDescription.test.tsx b/packages/mui-base/src/Popover/Description/PopoverDescription.test.tsx
@@ -20,7 +20,6 @@ describe('<Popover.Description />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('describes the popup element with its id', async () => {
diff --git a/packages/mui-base/src/Popover/Popup/PopoverPopup.test.tsx b/packages/mui-base/src/Popover/Popup/PopoverPopup.test.tsx
@@ -18,7 +18,6 @@ describe('<Popover.Popup />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('should render the children', async () => {
diff --git a/packages/mui-base/src/Popover/Positioner/PopoverPositioner.test.tsx b/packages/mui-base/src/Popover/Positioner/PopoverPositioner.test.tsx
@@ -16,6 +16,6 @@ describe('<Popover.Positioner />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
+    skip: [''],
   }));
 });
diff --git a/packages/mui-base/src/Popover/Title/PopoverTitle.test.tsx b/packages/mui-base/src/Popover/Title/PopoverTitle.test.tsx
@@ -20,7 +20,6 @@ describe('<Popover.Title />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('labels the popup element with its id', async () => {
diff --git a/packages/mui-base/src/Popover/Trigger/PopoverTrigger.test.tsx b/packages/mui-base/src/Popover/Trigger/PopoverTrigger.test.tsx
@@ -16,6 +16,5 @@ describe('<Popover.Trigger />', () => {
         </Popover.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Switch/Thumb/SwitchThumb.test.tsx b/packages/mui-base/src/Switch/Thumb/SwitchThumb.test.tsx
@@ -20,6 +20,5 @@ describe('<Switch.Thumb />', () => {
     render: (node) => {
       return render(<SwitchContext.Provider value={testContext}>{node}</SwitchContext.Provider>);
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Tabs/Tab/Tab.test.tsx b/packages/mui-base/src/Tabs/Tab/Tab.test.tsx
@@ -50,8 +50,5 @@ describe('<Tabs.Tab />', () => {
       return { container, ...other };
     },
     refInstanceof: window.HTMLButtonElement,
-    skip: [
-      'reactTestRenderer', // Need to be wrapped with TabsContext
-    ],
   }));
 });
diff --git a/packages/mui-base/src/Tabs/TabIndicator/TabIndicator.test.tsx b/packages/mui-base/src/Tabs/TabIndicator/TabIndicator.test.tsx
@@ -31,9 +31,6 @@ describe('<Tabs.Indicator />', () => {
     },
     refInstanceof: window.HTMLSpanElement,
     testRenderPropWith: 'div',
-    skip: [
-      'reactTestRenderer', // Need to be wrapped with TabsContext
-    ],
   }));
 
   describe('rendering', () => {
diff --git a/packages/mui-base/src/Tabs/TabPanel/TabPanel.test.tsx b/packages/mui-base/src/Tabs/TabPanel/TabPanel.test.tsx
@@ -31,8 +31,5 @@ describe('<Tabs.Panel />', () => {
       return { container, ...other };
     },
     refInstanceof: window.HTMLDivElement,
-    skip: [
-      'reactTestRenderer', // Need to be wrapped with TabsContext
-    ],
   }));
 });
diff --git a/packages/mui-base/src/Tabs/TabsList/TabsList.test.tsx b/packages/mui-base/src/Tabs/TabsList/TabsList.test.tsx
@@ -31,9 +31,6 @@ describe('<Tabs.List />', () => {
       return { container, ...other };
     },
     refInstanceof: window.HTMLDivElement,
-    skip: [
-      'reactTestRenderer', // Need to be wrapped with TabsContext
-    ],
   }));
 
   describe('accessibility attributes', () => {
diff --git a/packages/mui-base/src/Tooltip/Arrow/TooltipArrow.test.tsx b/packages/mui-base/src/Tooltip/Arrow/TooltipArrow.test.tsx
@@ -17,6 +17,5 @@ describe('<Tooltip.Arrow />', () => {
         </Tooltip.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Tooltip/Popup/TooltipPopup.test.tsx b/packages/mui-base/src/Tooltip/Popup/TooltipPopup.test.tsx
@@ -17,7 +17,6 @@ describe('<Tooltip.Popup />', () => {
         </Tooltip.Root>,
       );
     },
-    skip: ['reactTestRenderer'],
   }));
 
   it('should render the children', () => {
diff --git a/packages/mui-base/src/Tooltip/Positioner/TooltipPositioner.test.tsx b/packages/mui-base/src/Tooltip/Positioner/TooltipPositioner.test.tsx
@@ -12,6 +12,5 @@ describe('<Tooltip.Positioner />', () => {
     render(node) {
       return render(<Tooltip.Root open>{node}</Tooltip.Root>);
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/Tooltip/Trigger/TooltipTrigger.test.tsx b/packages/mui-base/src/Tooltip/Trigger/TooltipTrigger.test.tsx
@@ -12,6 +12,5 @@ describe('<Tooltip.Trigger />', () => {
     render(node) {
       return render(<Tooltip.Root>{node}</Tooltip.Root>);
     },
-    skip: ['reactTestRenderer'],
   }));
 });
diff --git a/packages/mui-base/src/legacy/Menu/Menu.test.tsx b/packages/mui-base/src/legacy/Menu/Menu.test.tsx
@@ -59,7 +59,7 @@ describe('<Menu />', () => {
         expectedClassName: menuClasses.listbox,
       },
     },
-    skip: ['reactTestRenderer', 'componentProp', 'slotsProp'],
+    skip: ['componentProp', 'slotsProp'],
   }));
 
   describe('after initialization', () => {
diff --git a/packages/mui-base/src/legacy/MenuButton/MenuButton.test.tsx b/packages/mui-base/src/legacy/MenuButton/MenuButton.test.tsx
@@ -37,7 +37,7 @@ describe('<MenuButton />', () => {
         testWithElement: null,
       },
     },
-    skip: ['componentProp', 'reactTestRenderer'],
+    skip: ['componentProp'],
   }));
 
   describe('prop: disabled', () => {
diff --git a/packages/mui-base/src/legacy/MenuItem/MenuItem.test.tsx b/packages/mui-base/src/legacy/MenuItem/MenuItem.test.tsx
@@ -37,9 +37,6 @@ describe('<MenuItem />', () => {
         expectedClassName: menuItemClasses.root,
       },
     },
-    skip: [
-      'componentProp',
-      'reactTestRenderer', // Need to be wrapped in MenuContext
-    ],
+    skip: ['componentProp'],
   }));
 });
diff --git a/packages/mui-base/src/legacy/Modal/Modal.test.tsx b/packages/mui-base/src/legacy/Modal/Modal.test.tsx
@@ -29,10 +29,7 @@ describe('<Modal />', () => {
           expectedClassName: classes.root,
         },
       },
-      skip: [
-        'componentProp',
-        'reactTestRenderer', // portal https://github.com/facebook/react/issues/11565
-      ],
+      skip: ['componentProp'],
     }),
   );
 
diff --git a/packages/mui-base/src/legacy/Option/Option.test.tsx b/packages/mui-base/src/legacy/Option/Option.test.tsx
@@ -38,9 +38,6 @@ describe('<Option />', () => {
         expectedClassName: optionClasses.root,
       },
     },
-    skip: [
-      'componentProp',
-      'reactTestRenderer', // Need to be wrapped in SelectContext
-    ],
+    skip: ['componentProp'],
   }));
 });
diff --git a/packages/mui-base/src/legacy/Popper/Popper.test.tsx b/packages/mui-base/src/legacy/Popper/Popper.test.tsx
@@ -23,11 +23,7 @@ describe('<Popper />', () => {
     inheritComponent: 'div',
     render,
     refInstanceof: window.HTMLDivElement,
-    skip: [
-      // https://github.com/facebook/react/issues/11565
-      'reactTestRenderer',
-      'componentProp',
-    ],
+    skip: ['componentProp'],
     slots: {
       root: {
         expectedClassName: popperClasses.root,
diff --git a/packages/mui-base/src/legacy/Select/Select.test.tsx b/packages/mui-base/src/legacy/Select/Select.test.tsx
@@ -60,7 +60,7 @@ describe('<Select />', () => {
         testWithElement: 'span',
       },
     },
-    skip: ['componentProp', 'reactTestRenderer'],
+    skip: ['componentProp'],
   }));
 
   describe('selected option rendering', () => {
diff --git a/packages/mui-base/src/legacy/Unstable_Popup/Popup.test.tsx b/packages/mui-base/src/legacy/Unstable_Popup/Popup.test.tsx
@@ -68,11 +68,7 @@ describe('<Popup />', () => {
       return result;
     },
     refInstanceof: window.HTMLDivElement,
-    skip: [
-      // https://github.com/facebook/react/issues/11565
-      'reactTestRenderer',
-      'componentProp',
-    ],
+    skip: ['componentProp'],
     slots: {
       root: {
         expectedClassName: popupClasses.root,
diff --git a/packages/mui-base/test/describeConformance.tsx b/packages/mui-base/test/describeConformance.tsx
@@ -4,7 +4,6 @@ import {
   MuiRenderResult,
   RenderOptions,
   createDescribe,
-  testReactTestRenderer,
 } from '@mui/internal-test-utils';
 import { testPropForwarding } from './conformanceTests/propForwarding';
 import { testRefForwarding } from './conformanceTests/refForwarding';
@@ -23,7 +22,6 @@ export interface BaseUiConformanceTestsOptions
 
 const fullSuite = {
   propsSpread: testPropForwarding,
-  reactTestRenderer: testReactTestRenderer,
   refForwarding: testRefForwarding,
   renderProp: testRenderProp,
   className: testClassName,
diff --git a/packages/mui-base/test/describeConformanceUnstyled.tsx b/packages/mui-base/test/describeConformanceUnstyled.tsx
@@ -10,7 +10,6 @@ import {
   describeRef,
   randomStringValue,
   testComponentProp,
-  testReactTestRenderer,
 } from '@mui/internal-test-utils';
 import { ClassNameConfigurator } from '@base_ui/react/legacy/utils/ClassNameConfigurator';
 
@@ -405,7 +404,6 @@ const fullSuite = {
   slotPropsCallbacks: testSlotPropsCallbacks,
   mergeClassName: testClassName,
   propsSpread: testPropForwarding,
-  reactTestRenderer: testReactTestRenderer,
   refForwarding: describeRef,
   ownerStatePropagation: testOwnerStatePropagation,
   disableClassGeneration: testDisablingClassGeneration,
diff --git a/renovate.json b/renovate.json
