-
Notifications
You must be signed in to change notification settings - Fork 515
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
test(menu-select): common test suite #5874
Conversation
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 6e1d785:
|
5f288da
to
7b29ece
Compare
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.
Can you add a test for cssClasses
?
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.
Can't find other CTS that tests this, when I change it it doesn't seem to be taken into account for vue
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.
Not in CTS, in the JS widget tests. You can look at hits or current refinements for example. We should have done it for all CTS migrated widgets but it seems we forgot a few on the way when I searched for adds custom CSS classes
😅 .
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.
Ah yes, done 👍
Summary
There weren't many logic tests to remove, probably because they're normally tested in the connector (so
connectMenu
)Also obviously had to skip testing for React as it doesn't have
MenuSelect