You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
nodeEqual() fails, because it expects className={undefined} property in the other component (<div/>). This is happening here https://github.com/airbnb/enzyme/blob/master/src/Utils.js#L94. Tested using the latest enzyme available on npm (2.5.1).
I think this case should be covered in enzyme by ignoring null and undefined props, which is what React does too. If everyone agrees, I can submit a PR to fix it ;)
The text was updated successfully, but these errors were encountered:
This is a continuation of #151, which fixed the skipping of nodes, but the same issue exists for props.
Here's an example to demonstrate the problem:
nodeEqual()
fails, because it expectsclassName={undefined}
property in the other component (<div/>
). This is happening here https://github.com/airbnb/enzyme/blob/master/src/Utils.js#L94. Tested using the latest enzyme available on npm (2.5.1
).I think this case should be covered in enzyme by ignoring
null
andundefined
props, which is what React does too. If everyone agrees, I can submit a PR to fix it ;)The text was updated successfully, but these errors were encountered: