This compares the objects properly in toMatchElement

[puneetar]

This compares the objects properly, rather than just equating the reference

[blainekasten]

Thanks for the PR!
But I don't believe this is the right fix. Ref #186

@see enzymejs/enzyme#1476

[puneetar]

Oh, I see. We are comparing the debug strings.
In some cases, I notice that the debug strings don't match but using the equals function on the enzyme wrapper returns true. Don't know why that happens.

[finnigantime]

This seems reasonable to me, but in the case where the expect fails, it's nice to show the diff in the debug strings to make it obvious what the tree diff is. That was the major advantage of comparing the debug strings. If you could use .equals() to compare but show the debug string diff (or some easily human-readable diff) in the case of expect failure, you'd have the best of both worlds. In either case, I think we want enzymejs/enzyme#1476 to get fixed.

[blainekasten]

@puneetar can you give me examples of cases you are describing?

[blainekasten]

This also won't work because actual is a string. You are wanting to do actualEnzymeWrapper.equals(expectedWrapper)

[blainekasten]

I'm closing this as it seems like the original PR is not responding to this, nor is the implementation correct. Feel free to keep talking here or open a new pr/issue.