Skip to content
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] Add conformance test suite for v5 #23798

Merged
merged 6 commits into from
Dec 2, 2020

Conversation

mnajdova
Copy link
Member

@mnajdova mnajdova commented Dec 1, 2020

This PR adds a conformance test suite for v5. It is used in the Slider tests.

@mui-pr-bot
Copy link

mui-pr-bot commented Dec 1, 2020

Details of bundle changes

Generated by 🚫 dangerJS against fbeeda7

@mnajdova mnajdova requested a review from eps1lon December 1, 2020 12:09
classes: {},
inheritComponent: 'span',
inheritComponent: SliderUnstyled,
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed also this test

componentsProp,
} = getOptions();

const wrapper = mount(React.cloneElement(element, { components: { Root: component } }));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it make sense to use react-testing-library for this new test instead of enzyme?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bit hard to change it only for this test, as the other suits are coming from the original describeConformance. I would suggest, we do it in one run after all components are migrated and we can safely remove the old test suite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants