@instructure/ui-menu 10.19.2-snapshot-3 → 10.19.2-snapshot-4

package/src/Menu/MenuItem/__new-tests__/MenuItem.test.tsx

@@ -1,161 +0,0 @@
-/*
- * The MIT License (MIT)
- *
- * Copyright (c) 2015 - present Instructure, Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal
- * in the Software without restriction, including without limitation the rights
- * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
- * copies of the Software, and to permit persons to whom the Software is
- * furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in all
- * copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
- * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
- * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
- * SOFTWARE.
- */
-
-import { fireEvent, render, screen } from '@testing-library/react'
-import { vi } from 'vitest'
-import '@testing-library/jest-dom'
-
-import { MenuItem } from '../index'
-import type { MenuItemProps } from '../props'
-
-interface ExtendedMenuItemProps extends MenuItemProps {
-  to: string
-}
-
-const ExtendedMenuItem: React.FC<ExtendedMenuItemProps> = ({ ...props }) => {
-  return <MenuItem {...props} />
-}
-
-describe('<MenuItem />', () => {
-  it('should render', () => {
-    render(<MenuItem>Menu Item Text</MenuItem>)
-    const menuItem = screen.getByText('Menu Item Text')
-
-    expect(menuItem).toBeInTheDocument()
-  })
-
-  it('should render as a link when an href is provided', () => {
-    render(<MenuItem href="example.html">Menu Item Text</MenuItem>)
-
-    const menuItem = screen.getByRole('menuitem')
-
-    expect(menuItem).toBeInTheDocument()
-    expect(menuItem).toHaveAttribute('href', 'example.html')
-  })
-
-  it('should render as a link when a to is provided', () => {
-    render(<ExtendedMenuItem to="/example">Hello</ExtendedMenuItem>)
-
-    const menuItem = screen.getByRole('menuitem')
-
-    expect(menuItem).toHaveAttribute('to', '/example')
-  })
-
-  it('should call onSelect after click', () => {
-    const onSelect = vi.fn()
-    render(
-      <MenuItem onSelect={onSelect} value="foo">
-        Hello
-      </MenuItem>
-    )
-    const menuItem = screen.getByRole('menuitem')
-
-    fireEvent.click(menuItem)
-
-    expect(onSelect).toHaveBeenCalledWith(
-      expect.any(Object),
-      'foo',
-      true,
-      expect.any(Object)
-    )
-  })
-
-  it('should call onClick after click', () => {
-    const onClick = vi.fn()
-    render(
-      <MenuItem onClick={onClick} value="foo">
-        Hello
-      </MenuItem>
-    )
-    const menuItem = screen.getByRole('menuitem')
-
-    fireEvent.click(menuItem)
-
-    expect(onClick).toHaveBeenCalled()
-  })
-
-  it('should set the tabIndex attribute', () => {
-    render(<MenuItem>Hello</MenuItem>)
-    const menuItem = screen.getByRole('menuitem')
-
-    expect(menuItem).toHaveAttribute('tabIndex', '-1')
-  })
-
-  it('should set the aria-controls attribute', () => {
-    render(<MenuItem controls="testId">Hello</MenuItem>)
-    const menuItem = screen.getByRole('menuitem')
-
-    expect(menuItem).toHaveAttribute('aria-controls', 'testId')
-  })
-
-  it('should set the aria-disabled attribute', () => {
-    render(<MenuItem disabled>Hello</MenuItem>)
-    const menuItem = screen.getByRole('menuitem')
-
-    expect(menuItem).toHaveAttribute('aria-disabled', 'true')
-  })
-
-  it('should set the aria-checked attribute when defaultSelected prop is true', () => {
-    render(
-      <MenuItem type="checkbox" defaultSelected>
-        Hello
-      </MenuItem>
-    )
-    const menuItem = screen.getByRole('menuitemcheckbox')
-
-    expect(menuItem).toHaveAttribute('aria-checked', 'true')
-  })
-
-  it('should set the aria-checked attribute when selected prop is true', () => {
-    render(
-      <MenuItem type="checkbox" selected onSelect={vi.fn()}>
-        Hello
-      </MenuItem>
-    )
-    const menuItem = screen.getByRole('menuitemcheckbox')
-
-    expect(menuItem).toHaveAttribute('aria-checked', 'true')
-  })
-
-  it('should default to the "menuitem" role', () => {
-    const { container } = render(<MenuItem>Menu Item Text</MenuItem>)
-    const menuItem = container.querySelector("span[class$='-menuItem']")
-
-    expect(menuItem).toHaveAttribute('role', 'menuitem')
-  })
-
-  it('should set the role to "menuitemcheckbox" when the type is "checkbox"', () => {
-    const { container } = render(<MenuItem type="checkbox">Hello</MenuItem>)
-    const menuItem = container.querySelector("span[class$='-menuItem']")
-
-    expect(menuItem).toHaveAttribute('role', 'menuitemcheckbox')
-  })
-
-  it('should set the role to "menuitemradio" when the type is "radio"', () => {
-    const { container } = render(<MenuItem type="radio">Hello</MenuItem>)
-    const menuItem = container.querySelector("span[class$='-menuItem']")
-
-    expect(menuItem).toHaveAttribute('role', 'menuitemradio')
-  })
-})

package/src/Menu/MenuItemGroup/__new-tests__/MenuItemGroup.test.tsx

@@ -1,187 +0,0 @@
-/*
- * The MIT License (MIT)
- *
- * Copyright (c) 2015 - present Instructure, Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal
- * in the Software without restriction, including without limitation the rights
- * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
- * copies of the Software, and to permit persons to whom the Software is
- * furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in all
- * copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
- * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
- * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
- * SOFTWARE.
- */
-
-import { fireEvent, render, screen } from '@testing-library/react'
-import { vi } from 'vitest'
-import '@testing-library/jest-dom'
-
-import { MenuItem } from '../../MenuItem'
-import { MenuItemSeparator } from '../../MenuItemSeparator'
-import { MenuItemGroup } from '../index'
-
-describe('<MenuItemGroup />', () => {
-  it('should render', () => {
-    const { container } = render(
-      <MenuItemGroup label="Menu Label">
-        <MenuItem>Item Text 1</MenuItem>
-        <MenuItem>Item Text 2</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const group = container.querySelector("[class*='menuItemGroup']")
-    expect(group).toBeInTheDocument()
-    expect(group).toHaveTextContent('Menu Label')
-    expect(group).toHaveTextContent('Item Text 1')
-    expect(group).toHaveTextContent('Item Text 2')
-  })
-
-  it('should default to children with type "radio"', () => {
-    render(
-      <MenuItemGroup label="Select one">
-        <MenuItem>Foo</MenuItem>
-        <MenuItem>Bar</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const menuItems = screen.getAllByRole('menuitemradio')
-
-    expect(menuItems).toHaveLength(2)
-  })
-
-  it('should render children with type "checkbox" if allowMultiple is true', () => {
-    render(
-      <MenuItemGroup label="Select a few" allowMultiple>
-        <MenuItem>Foo</MenuItem>
-        <MenuItem>Bar</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const menuItems = screen.getAllByRole('menuitemcheckbox')
-
-    expect(menuItems).toHaveLength(2)
-  })
-
-  it('should set aria-disabled', () => {
-    render(
-      <MenuItemGroup label="Select one" disabled>
-        <MenuItem>Foo</MenuItem>
-        <MenuItem>Bar</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const menuItems = screen.getAllByRole('menuitemradio')
-
-    expect(menuItems).toHaveLength(2)
-    expect(menuItems[0]).toHaveAttribute('aria-disabled', 'true')
-    expect(menuItems[1]).toHaveAttribute('aria-disabled', 'true')
-  })
-
-  it('should set selected from defaultSelected prop', () => {
-    render(
-      <MenuItemGroup label="Select one" defaultSelected={[1]}>
-        <MenuItem>Foo</MenuItem>
-        <MenuItem>Bar</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const menuItems = screen.getAllByRole('menuitemradio')
-
-    expect(menuItems).toHaveLength(2)
-    expect(menuItems[0]).toHaveAttribute('aria-checked', 'false')
-    expect(menuItems[1]).toHaveAttribute('aria-checked', 'true')
-  })
-
-  it('should set selected from selected prop', () => {
-    render(
-      <MenuItemGroup label="Select one" onSelect={vi.fn()} selected={[1]}>
-        <MenuItem>Foo</MenuItem>
-        <MenuItem>Bar</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const menuItems = screen.getAllByRole('menuitemradio')
-
-    expect(menuItems).toHaveLength(2)
-    expect(menuItems[0]).toHaveAttribute('aria-checked', 'false')
-    expect(menuItems[1]).toHaveAttribute('aria-checked', 'true')
-  })
-
-  it('should set selected from children', () => {
-    render(
-      <MenuItemGroup label="Select a few" allowMultiple>
-        <MenuItem key="foo" defaultSelected>
-          Foo
-        </MenuItem>
-        <MenuItem key="bar" selected onSelect={vi.fn()}>
-          Bar
-        </MenuItem>
-      </MenuItemGroup>
-    )
-    const menuItems = screen.getAllByRole('menuitemcheckbox')
-
-    expect(menuItems).toHaveLength(2)
-    expect(menuItems[0]).toHaveAttribute('aria-checked', 'true')
-    expect(menuItems[1]).toHaveAttribute('aria-checked', 'true')
-  })
-
-  it('should honor the allowMultiple prop (defaults to false)', () => {
-    render(
-      <MenuItemGroup label="Select one">
-        <MenuItem defaultSelected>Foo</MenuItem>
-        <MenuItem selected onSelect={vi.fn()}>
-          Bar
-        </MenuItem>
-      </MenuItemGroup>
-    )
-    const menuItems = screen.getAllByRole('menuitemradio')
-
-    expect(menuItems).toHaveLength(2)
-    expect(menuItems[0]).toHaveAttribute('aria-checked', 'true')
-    expect(menuItems[1]).toHaveAttribute('aria-checked', 'false')
-  })
-
-  it('calls onSelect when items are selected', () => {
-    const onSelect = vi.fn()
-    render(
-      <MenuItemGroup label="Select one" onSelect={onSelect} selected={[1]}>
-        <MenuItem>Item 1</MenuItem>
-        <MenuItem>Item 2</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const menuItem = screen.getByText('Item 1')
-
-    fireEvent.click(menuItem)
-
-    expect(onSelect).toHaveBeenCalled()
-    expect(onSelect.mock.calls[0][1]).toEqual([0])
-    expect(onSelect.mock.calls[0][2]).toEqual(true)
-  })
-
-  it('does not call onSelect when disabled', () => {
-    const onSelect = vi.fn()
-    render(
-      <MenuItemGroup label="Select one" onSelect={onSelect} disabled>
-        <MenuItem>Item 1</MenuItem>
-        <MenuItem>Item 2</MenuItem>
-        <MenuItemSeparator />
-      </MenuItemGroup>
-    )
-    const menuItem = screen.getByText('Item 1')
-
-    fireEvent.click(menuItem)
-
-    expect(onSelect).not.toHaveBeenCalled()
-  })
-})

package/src/Menu/__new-tests__/Menu.test.tsx

@@ -1,358 +0,0 @@
-/*
- * The MIT License (MIT)
- *
- * Copyright (c) 2015 - present Instructure, Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal
- * in the Software without restriction, including without limitation the rights
- * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
- * copies of the Software, and to permit persons to whom the Software is
- * furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in all
- * copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
- * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
- * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
- * SOFTWARE.
- */
-
-import { render, screen, fireEvent } from '@testing-library/react'
-import { vi } from 'vitest'
-import { userEvent } from '@testing-library/user-event'
-import '@testing-library/jest-dom'
-
-import { runAxeCheck } from '@instructure/ui-axe-check'
-import { Popover } from '@instructure/ui-popover'
-import { Menu, MenuItem, MenuItemSeparator } from '../index'
-
-describe('<Menu />', () => {
-  let consoleWarningMock: ReturnType<typeof vi.spyOn>
-  let consoleErrorMock: ReturnType<typeof vi.spyOn>
-
-  beforeEach(() => {
-    // Mocking console to prevent test output pollution and expect for messages
-    consoleWarningMock = vi
-      .spyOn(console, 'warn')
-      .mockImplementation(() => {}) as any
-    consoleErrorMock = vi
-      .spyOn(console, 'error')
-      .mockImplementation(() => {}) as any
-  })
-
-  afterEach(() => {
-    consoleWarningMock.mockRestore()
-    consoleErrorMock.mockRestore()
-  })
-
-  describe('without a trigger', () => {
-    it('should render', () => {
-      render(
-        <Menu label="Menu-label-text">
-          <MenuItem>Menu Item Text</MenuItem>
-        </Menu>
-      )
-      const menu = screen.getByRole('menu')
-
-      expect(menu).toBeInTheDocument()
-      expect(menu).toHaveTextContent('Menu Item Text')
-      expect(menu).toHaveAttribute('aria-label', 'Menu-label-text')
-    })
-
-    it('should meet a11y standards', async () => {
-      const { container } = render(
-        <Menu label="Menu-label-text">
-          <MenuItem>Menu Item Text</MenuItem>
-        </Menu>
-      )
-
-      const axeCheck = await runAxeCheck(container)
-
-      expect(axeCheck).toBe(true)
-    })
-
-    it('should meet standards when menu is closed', async () => {
-      const { container } = render(
-        <Menu trigger={<button>More</button>}>
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-      const axeCheck = await runAxeCheck(container)
-
-      expect(axeCheck).toBe(true)
-    })
-
-    it('should meet standards when menu is open', async () => {
-      const { container } = render(
-        <Menu trigger={<button>More</button>} defaultShow>
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-      const axeCheck = await runAxeCheck(container)
-
-      expect(axeCheck).toBe(true)
-    })
-
-    it('should call onSelect when menu item is selected', async () => {
-      const onSelect = vi.fn()
-
-      const { getByText } = render(
-        <Menu label="Settings" onSelect={onSelect}>
-          <MenuItem value="Test">Test Item</MenuItem>
-        </Menu>
-      )
-      const item = getByText('Test Item')
-
-      await userEvent.click(item)
-
-      expect(onSelect).toHaveBeenCalled()
-      expect(onSelect.mock.calls[0][1]).toEqual('Test')
-    })
-
-    it('should not call onSelect when disabled', () => {
-      const onSelect = vi.fn()
-
-      const { getByText } = render(
-        <Menu label="Settings" onSelect={onSelect} disabled>
-          <MenuItem value="Account">Account</MenuItem>
-        </Menu>
-      )
-      const itemText = getByText('Account')
-      const menu = screen.getByRole('menu')
-      const menuItem = screen.getByRole('menuitem')
-
-      expect(itemText).toBeInTheDocument()
-      expect(menu).toHaveAttribute('aria-disabled', 'true')
-      expect(menuItem).toHaveAttribute('aria-disabled', 'true')
-    })
-
-    it('should provide a menu ref', () => {
-      const menuRef = vi.fn()
-
-      render(
-        <Menu label="Settings" menuRef={menuRef}>
-          <MenuItem value="Account">Account</MenuItem>
-        </Menu>
-      )
-      const menu = screen.getByRole('menu')
-
-      expect(menuRef).toHaveBeenLastCalledWith(menu)
-    })
-
-    it('should set aria attributes properly', () => {
-      render(
-        <Menu disabled label="Settings">
-          <MenuItem value="Account">Account</MenuItem>
-        </Menu>
-      )
-      const menu = screen.getByRole('menu')
-
-      expect(menu).toHaveAttribute('aria-disabled', 'true')
-      expect(menu).toHaveAttribute('aria-label', 'Settings')
-    })
-  })
-
-  describe('with a trigger', () => {
-    it('should render into a mountNode', () => {
-      const mountNode = document.createElement('div')
-
-      document.body.appendChild(mountNode)
-
-      render(
-        <Menu
-          defaultShow
-          mountNode={mountNode}
-          label="Settings"
-          trigger={<button>Settings</button>}
-        >
-          <MenuItem>Account</MenuItem>
-        </Menu>
-      )
-
-      expect(mountNode).toHaveTextContent('Account')
-
-      document.body.removeChild(mountNode)
-    })
-
-    it('should set aria attributes properly', () => {
-      render(
-        <Menu trigger={<button>Settings</button>} defaultShow>
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-          <MenuItem type="radio" defaultChecked>
-            Default (Grid view)
-          </MenuItem>
-          <MenuItem type="radio">Individual (List view)</MenuItem>
-          <MenuItem type="checkbox" defaultChecked>
-            Include Anchor Standards
-          </MenuItem>
-          <MenuItemSeparator />
-          <MenuItem>Open grading history...</MenuItem>
-        </Menu>
-      )
-      const menu = screen.getByRole('menu')
-      const trigger = screen.getByRole('button', { name: 'Settings' })
-
-      expect(menu).toBeInTheDocument()
-      expect(menu).toHaveAttribute(
-        'aria-labelledby',
-        trigger.getAttribute('id')
-      )
-    })
-
-    it('should call onFocus on focus', () => {
-      const onFocus = vi.fn()
-
-      const { getByRole } = render(
-        <Menu trigger={<button>More</button>} onFocus={onFocus}>
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-      const triggerButton = getByRole('button', { name: 'More' })
-
-      fireEvent.focus(triggerButton)
-
-      expect(onFocus).toHaveBeenCalled()
-    })
-
-    it('should render when show and onToggle props are set', () => {
-      const { getByRole, getAllByRole, queryByText } = render(
-        <Menu trigger={<button>More</button>} show onToggle={() => {}}>
-          <MenuItem>Test1</MenuItem>
-          <MenuItem disabled>Test2</MenuItem>
-        </Menu>
-      )
-      const menuItems = getAllByRole('menuitem')
-      const triggerButton = getByRole('button', { name: 'More' })
-
-      const menuItem1 = queryByText('Test1')
-      const menuItem2 = queryByText('Test2')
-
-      expect(triggerButton).toBeInTheDocument()
-      expect(menuItems).toHaveLength(2)
-      expect(menuItem1).toBeInTheDocument()
-      expect(menuItem2).toBeInTheDocument()
-    })
-
-    it('should not show by default', () => {
-      const { queryByText, getByRole } = render(
-        <Menu trigger={<button>More</button>}>
-          <MenuItem>Test1</MenuItem>
-          <MenuItem disabled>Test2</MenuItem>
-        </Menu>
-      )
-      const triggerButton = getByRole('button', { name: 'More' })
-
-      const menuItem1 = queryByText('Test1')
-      const menuItem2 = queryByText('Test2')
-
-      expect(triggerButton).toBeInTheDocument()
-      expect(menuItem1).not.toBeInTheDocument()
-      expect(menuItem2).not.toBeInTheDocument()
-    })
-
-    it('should accept a default show', () => {
-      const { queryByText, getByRole } = render(
-        <Menu trigger={<button>More</button>} defaultShow>
-          <MenuItem>Test1</MenuItem>
-          <MenuItem disabled>Test2</MenuItem>
-        </Menu>
-      )
-      const triggerButton = getByRole('button', { name: 'More' })
-
-      const menuItem1 = queryByText('Test1')
-      const menuItem2 = queryByText('Test2')
-
-      expect(triggerButton).toBeInTheDocument()
-      expect(menuItem1).toBeInTheDocument()
-      expect(menuItem2).toBeInTheDocument()
-    })
-
-    it('should provide a menu ref', () => {
-      const menuRef = vi.fn()
-
-      render(
-        <Menu trigger={<button>More</button>} defaultShow menuRef={menuRef}>
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-      const menu = screen.getByRole('menu')
-
-      expect(menuRef).toHaveBeenCalledWith(menu)
-    })
-
-    it('should provide a popoverRef ref', () => {
-      const popoverRef = vi.fn()
-
-      render(
-        <Menu
-          trigger={<button>More</button>}
-          defaultShow
-          popoverRef={popoverRef}
-        >
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-
-      expect(popoverRef).toHaveBeenCalled()
-    })
-
-    it('should call onToggle on click', () => {
-      const onToggle = vi.fn()
-
-      render(
-        <Menu trigger={<button>More</button>} onToggle={onToggle}>
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-      const trigger = screen.getByRole('button')
-
-      fireEvent.click(trigger)
-
-      expect(onToggle).toHaveBeenCalled()
-    })
-
-    it('should have an aria-haspopup attribute', () => {
-      render(
-        <Menu trigger={<button>More</button>}>
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-      const trigger = screen.getByRole('button')
-
-      expect(trigger).toHaveAttribute('aria-haspopup')
-    })
-
-    it('should pass positionContainerDisplay prop to Popover', () => {
-      let popoverRef: Popover | null = null
-      render(
-        <Menu
-          trigger={<button>More</button>}
-          popoverRef={(e) => {
-            popoverRef = e
-          }}
-          positionContainerDisplay="block"
-        >
-          <MenuItem>Learning Mastery</MenuItem>
-          <MenuItem disabled>Gradebook</MenuItem>
-        </Menu>
-      )
-
-      const popoverProps = popoverRef!.props
-
-      expect(popoverProps.positionContainerDisplay).toBe('block')
-    })
-  })
-})