-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
menu list unit tests #28857
menu list unit tests #28857
Conversation
…erbrady/fluentui-fork into web-components-v3
🕵 fluentui-web-components-v3 No visual regressions between this PR and main |
📊 Bundle size report🤖 This report was generated against c7b318221b2bd34b3b60c61c84859e9bc861b0a2 |
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: c7b318221b2bd34b3b60c61c84859e9bc861b0a2 (build) |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 400c424:
|
await expect(menuItems.nth(3)).toBeFocused(); | ||
}); | ||
|
||
test('should close the menu when pressing the escape key', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this is testing to see if the focus returns to the parent item after the Esc key is pressed. I would reword the test to make your assertion clearer.
await root.evaluate(node => { | ||
node.innerHTML = /* html */ ` | ||
<fluent-menu-list> | ||
<fluent-menu-item role="menuitemcheckbox"></fluent-menu-item> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
role should be: menuitemradio
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
w/ comment
be3c3cb
into
microsoft:web-components-v3
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
* radio init * styles radio * reverts branch * input spec init * cleans up spec * formatting * updates component name to text input * updates component name in spec * reverts dead file * adds menulist unit tests * menu-list tests: updates test names, fixes role attribute in radio test
Adds MenuList playwright tests