Skip to content

feat(react-17,18-tests): bootstrap r17,r18 integration tests #391

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

Merged
merged 2 commits into from
Jun 25, 2025

Conversation

Hotell
Copy link
Contributor

@Hotell Hotell commented May 19, 2025

similarly to microsoft/fluentui#34247 bootstraps integration test agains react v17 and r18 (to get proper type checks data set, which is currently missing)

NOTE: already exposes some issues with React 17 https://github.com/microsoft/fluentui-contrib/actions/runs/15853543512/job/44692987043?pr=391


to make this work, we introduce additional features ( will be done as separate PRs for granularity )

@Hotell Hotell force-pushed the integration/react/17 branch 11 times, most recently from c798509 to e1716f5 Compare May 19, 2025 16:11
@Hotell Hotell changed the title feat(react-17-tests): boostrap r17 integration tests feat(react-17-tests): bootstrap r17 integration tests May 19, 2025
@Hotell Hotell force-pushed the integration/react/17 branch 6 times, most recently from 6749896 to 91eadd6 Compare May 20, 2025 09:55
@Hotell Hotell changed the title feat(react-17-tests): bootstrap r17 integration tests feat(react-17,18-tests): bootstrap r17,r18 integration tests May 20, 2025
@Hotell Hotell force-pushed the integration/react/17 branch from 91eadd6 to 4215eca Compare June 24, 2025 14:37
* @param {string} code The code to transform
* @returns {string} The transformed code
*/
function transformRenderHookUsage(code) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

necessary to transform test sources to use react 17 related valid apis.

in case of react 17 and RTL with RTL react-hooks we need to transform import {renderHooks} from '@testing-library/react' to import {renderHooks} from '@testing-library/react-hooks' after transpilation to vanilla js happened.

@Hotell Hotell marked this pull request as ready for review June 24, 2025 14:53
@Hotell Hotell requested a review from a team as a code owner June 24, 2025 14:53
@Hotell Hotell requested a review from dmytrokirpa June 24, 2025 14:53
@Hotell Hotell merged commit 74d601d into microsoft:main Jun 25, 2025
4 of 5 checks passed
@Hotell Hotell deleted the integration/react/17 branch June 25, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants