Skip to content
Prev Previous commit
Next Next commit
Remove createMount from Menu tests
  • Loading branch information
aarongarciah committed Jun 21, 2024
commit 017bee81d7fa996362764bf202fa8f31046fa0f2
36 changes: 14 additions & 22 deletions packages/mui-material/src/Menu/Menu.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,20 +3,18 @@ import { spy } from 'sinon';
import { expect } from 'chai';
import {
createRenderer,
createMount,
screen,
fireEvent,
strictModeDoubleLoggingSuppressed,
} from '@mui/internal-test-utils';
import Menu, { menuClasses as classes } from '@mui/material/Menu';
import Popover from '@mui/material/Popover';
import { createTheme, ThemeProvider } from '@mui/material/styles';
import { MenuPaper } from './Menu';
import describeConformance from '../../test/describeConformance';
import { paperClasses } from '../Paper';

describe('<Menu />', () => {
const { render } = createRenderer({ clock: 'fake' });
const mount = createMount();

describeConformance(<Menu anchorEl={() => document.createElement('div')} open />, () => ({
classes,
Expand Down Expand Up @@ -141,8 +139,9 @@ describe('<Menu />', () => {
describe('prop: PaperProps', () => {
it('should be passed to the paper component', () => {
const customElevation = 12;
const customClasses = { rounded: { borderRadius: 12 } };
const wrapper = mount(
const customClasses = { rounded: 'custom-rounded' };

render(
<Menu
anchorEl={document.createElement('div')}
open
Expand All @@ -154,8 +153,8 @@ describe('<Menu />', () => {
/>,
);

expect(wrapper.find(MenuPaper).props().elevation).to.equal(customElevation);
expect(wrapper.find(MenuPaper).props().classes).to.contain(customClasses);
expect(screen.getByTestId('paper')).to.have.class(paperClasses.elevation12);
expect(screen.getByTestId('paper')).to.have.class(customClasses.rounded);
});
});

Expand Down Expand Up @@ -381,27 +380,20 @@ describe('<Menu />', () => {
});
});

describe('paper', () => {
it('should use MenuPaper component', () => {
const wrapper = mount(
<Menu anchorEl={document.createElement('div')} open>
<div />
</Menu>,
);

expect(wrapper.find(MenuPaper)).to.have.length(1);
});
});

describe('slots', () => {
it('should merge slots with existing values', () => {
const wrapper = mount(
<Menu slots={{ root: 'span' }} anchorEl={document.createElement('div')} open>
render(
<Menu
slots={{ root: 'span' }}
slotProps={{ paper: { 'data-testid': 'paper' } }}
anchorEl={document.createElement('div')}
open
>
<div />
</Menu>,
);

expect(wrapper.find(MenuPaper)).to.have.length(1);
expect(screen.getByTestId('paper')).to.have.length(1);
});
});
});