Skip to content

Added SSR/hydration tests for modes, forwardRef, and Profiler#13195

Merged
bvaughn merged 3 commits intofacebook:masterfrom
bvaughn:ssr-hydration-special-component-tests
Jul 12, 2018
Merged

Added SSR/hydration tests for modes, forwardRef, and Profiler#13195
bvaughn merged 3 commits intofacebook:masterfrom
bvaughn:ssr-hydration-special-component-tests

Conversation

@bvaughn
Copy link
Copy Markdown
Contributor

@bvaughn bvaughn commented Jul 11, 2018

We didn't seem to have great coverage for these component types.

@bvaughn bvaughn requested a review from gaearon July 11, 2018 21:01
@bvaughn bvaughn changed the title Added partial renderer tests for modes, forwardRef, and Profiler Added SSR/hydration tests for modes, forwardRef, and Profiler Jul 11, 2018
@bvaughn bvaughn force-pushed the ssr-hydration-special-component-tests branch from fb16396 to 163365d Compare July 11, 2018 21:03
@bvaughn bvaughn force-pushed the ssr-hydration-special-component-tests branch from 163365d to ef480db Compare July 11, 2018 21:04
@@ -0,0 +1,56 @@
/**
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can we combine these two files into IntegrationSpecialTypes or something? We don't expect to add many more tests to it, and each test file has some overhead.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure.

@bvaughn bvaughn merged commit 58f3b29 into facebook:master Jul 12, 2018
@bvaughn bvaughn deleted the ssr-hydration-special-component-tests branch July 12, 2018 15:35
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.

3 participants