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 a small test for ReactTestUtils to find rendered component with type in document #24368

Merged
merged 1 commit into from
Feb 17, 2023

Conversation

Biki-das
Copy link
Contributor

@Biki-das Biki-das commented Apr 13, 2022

I tried to write test for the ReactTestUtils to find rendered component with type in document

Tests before this PR

Windows PowerShell 4_13_2022 11_35_24 PM

Tests after this PR

Windows PowerShell 4_13_2022 11_35_30 PM

@Biki-das Biki-das changed the title Test :Added a small test for working of Simulate with renderIntoDocument Test :Added a small test for ReactTestUtils to find rendered component with type in document Apr 13, 2022
@sizebot
Copy link

sizebot commented Apr 13, 2022

Comparing: 168da8d...ddc77ac

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 131.43 kB 131.43 kB = 42.05 kB 42.05 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 136.70 kB 136.70 kB = 43.64 kB 43.64 kB
facebook-www/ReactDOM-prod.classic.js = 435.12 kB 435.12 kB = 79.88 kB 79.88 kB
facebook-www/ReactDOM-prod.modern.js = 420.12 kB 420.12 kB = 77.51 kB 77.51 kB
facebook-www/ReactDOMForked-prod.classic.js = 435.12 kB 435.12 kB = 79.88 kB 79.88 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against ddc77ac

@Biki-das
Copy link
Contributor Author

@gaearon could you. Review?

@Biki-das Biki-das changed the title Test :Added a small test for ReactTestUtils to find rendered component with type in document Test :Add a small test for ReactTestUtils to find rendered component with type in document Apr 14, 2022
Copy link
Member

@rickhanlonii rickhanlonii left a comment

Choose a reason for hiding this comment

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

Seems good to me, surprised that we don't have a test like this already.

@Biki-das
Copy link
Contributor Author

Seems good to me, surprised that we don't have a test like this already.

Also added some tests one month back to the react ART package too 😀
#23195

@Biki-das Biki-das requested a review from rickhanlonii April 18, 2022 07:08
@Biki-das
Copy link
Contributor Author

hey @rickhanlonii is it good to go , or there are some nits?

@Biki-das
Copy link
Contributor Author

@gaearon friendly ping :-)

@worldwidebaby
Copy link

See #21309

@Biki-das
Copy link
Contributor Author

@lunaruan could you review?

@badajack
Copy link

hey @sebmarkbage could you review!

@Biki-das
Copy link
Contributor Author

hey @eps1lon seems like this can go forward as it's already been approved looking forward to your thoughts

@eps1lon eps1lon merged commit 4fcc918 into facebook:main Feb 17, 2023
github-actions bot pushed a commit that referenced this pull request Feb 17, 2023
…with type in document (#24368)

I tried to write test for the ReactTestUtils to find rendered component
with type in document

Tests before this PR

![Windows PowerShell 4_13_2022 11_35_24
PM](https://user-images.githubusercontent.com/72331432/163243620-40eb753c-4136-4793-a628-efcf9e004562.png)

Tests after this PR

![Windows PowerShell 4_13_2022 11_35_30
PM](https://user-images.githubusercontent.com/72331432/163244704-cd17f0e3-7289-4794-895a-be03753e46de.png)

DiffTrain build for [4fcc918](4fcc918)
[View git log for this commit](https://github.com/facebook/react/commits/4fcc9184ac2662186cbf1470005bf2b7c67dd40b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants