feat(component-testing): breaking: Add React rerender functionality #16038
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
agg23
requested review from
chrisbreiding and
kuceb
and removed request for
a team
|
Thanks for taking the time to open a PR!
|
|
Test summary
Run details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
lmiller1990
previously approved these changes
Apr 19, 2021
There was a problem hiding this comment.
Cool, looks good. Merge on CI ✅
I think we should also update the docs repo to include more information about this. I think we should have some pages about the specific CT integrations in the docs repo - just having a README in the npm package directory is not very discoverable.
We should work with the docs team on this.
…ypress into agg23/ReactRerender
elevatebart
approved these changes
Apr 20, 2021
tgriesser
added a commit
that referenced
this pull request
Apr 21, 2021
…press into tgriesser/chore/improve-ci * 'tgriesser/chore/improve-ci' of github.com:cypress-io/cypress: fix(deps): update dependency systeminformation to version 5.6.4 🌟 (#15819) docs: fix a typo of package name [skip ci] (#15783) chore: deduplicate yarn.lock (#15988) fix(webpack): allow load custom asset on windows (#16099) fix: Properly display unmount as a command (#16041) fix(component-testing): correct imports for relative paths in cypress.json (#16056) Fixed missing SearchInput and improved SpecList scrolling (#16090) docs: update react docs (#16055) chore: Design System Cleanup (#16077) feat(component-testing): breaking: Add React rerender functionality (#16038) fix(component-testing): Increased timeout to allow useEffect to trigger (#16091) chore: release @cypress/webpack-batteries-included-preprocessor-v2.2.1
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
User facing changelog
Breaking change: Introduces
rerenderfunctionality in@cypress/react. This is exposed through the thenable chain frommount:This is breaking as
mountpreviously returned a thenable with the renderedcomponent, rather than an object.Additional details
React has several mechanisms for internally preserving and updating state between rerenders/remounts. Exposing
rerenderoffers a mechanism to take an existing rendered component and update its props in the same way that a parent component changes the props of its children.How has the user experience changed?
rerendernow appears in the CommandLog.PR Tasks
cypress-documentation?