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

Make nodeToHostNode handle arrays #2001

Merged
merged 2 commits into from Feb 4, 2019

Conversation

Projects
None yet
2 participants
@madicap
Copy link
Contributor

madicap commented Feb 4, 2019

In order to resolve Issue #1799 and return expected .text() and .html() results for non-wrapped Fragments, nodeToHostNode in the react adapters needed to be able to handle arrays. This PR does that, changing .text() and .html() to deal with an array of nodes while maintaining backwards compatibility for adapters returning single nodes.

Fixes #1799.

madicap added some commits Jan 31, 2019

@ljharb

ljharb approved these changes Feb 4, 2019

@ljharb ljharb added the MINOR label Feb 4, 2019

@ljharb ljharb merged commit bcb6fa9 into master Feb 4, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@ljharb ljharb deleted the mac--html-text-fragment-fixes branch Feb 4, 2019

ljharb added a commit that referenced this pull request Feb 5, 2019

[enzyme-adapter-react-16] v1.9.0
 - [New] Make nodeToHostNode handle arrays (#2001)
 - [Dev Deps] update `eslint`, `eslint-plugin-import`, `eslint-plugin-jsx-a11y`, `eslint-plugin-react`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment