Update unmountOnExit behaviour docs and tests

mui · Jan 5, 2024 · 549b303 · 549b303
1 parent 23fc7c4
commit 549b303
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 2 deletions.
diff --git a/docs/data/material/components/accordion/accordion.md b/docs/data/material/components/accordion/accordion.md
@@ -53,7 +53,7 @@ accordions it might be a good idea to change this default behavior by enabling t
 `unmountOnExit` in `TransitionProps`:
 
 ```jsx
-<Accordion TransitionProps={{ unmountOnExit: true }} />
+<Accordion slotProps={{ transition: { unmountOnExit: true } }} />
 ```
 
 As with any performance optimization this is not a silver bullet.

diff --git a/packages/mui-material/src/Accordion/Accordion.test.js b/packages/mui-material/src/Accordion/Accordion.test.js
@@ -30,7 +30,7 @@ describe('<Accordion />', () => {
     testVariantProps: { variant: 'rounded' },
     slots: {
       transition: {
-        expectedClassName: '',
+        testWithElement: null,
       },
     },
     skip: ['componentProp', 'componentsProp', 'slotPropsCallback'],
@@ -227,4 +227,28 @@ describe('<Accordion />', () => {
       expect(getByTestId('transition-testid')).not.to.equal(null);
     });
   });
+
+  describe('details unmounting behavior', () => {
+    it('does not unmount by default', () => {
+      const { queryByTestId } = render(
+        <Accordion expanded={false}>
+          <AccordionSummary>Summary</AccordionSummary>
+          <div data-testid="details">Details</div>
+        </Accordion>,
+      );
+
+      expect(queryByTestId('details')).not.to.equal(null);
+    });
+
+    it('unmounts if opted in via slotProps.transition', () => {
+      const { queryByTestId } = render(
+        <Accordion expanded={false} slotProps={{ transition: { unmountOnExit: true } }}>
+          <AccordionSummary>Summary</AccordionSummary>
+          <div data-testid="details">Details</div>
+        </Accordion>,
+      );
+
+      expect(queryByTestId('details')).to.equal(null);
+    });
+  });
 });