-
Notifications
You must be signed in to change notification settings - Fork 292
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
Fix clickable tests (add interaction tests for focus and few behaviour tests) #609
Conversation
role={role} | ||
onClick={disabled ? undefined : onClick} | ||
id={id} | ||
onKeyDown={disabled ? undefined : onKeyDown} | ||
tabIndex={tabIndex} | ||
tabIndex={disabled ? -1 : tabIndex} |
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.
fix bug - we allowed focus on disabled clickable element
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.
package-lock shouldn't have any changes - something's probably wrong @hadasfa
export const statesPlaySuite = interactionSuite({ | ||
tests: [states_onClickTabFocusElementTest], | ||
afterEach: async () => { | ||
debugger; |
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.
😱
await pressNavigationKey(NAVIGATIONS_COMMANDS.TAB); | ||
expect(document.activeElement).toEqual(clickableElement); | ||
await pressNavigationKey(NAVIGATIONS_COMMANDS.TAB); | ||
console.log(document.activeElement); |
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.
whoops
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.
don't forget the package-lock :D
…to tests/hadas/interaction-tests
No description provided.