expect(null).toEqual(undefined) failing since 1.13.1

[agundermann]

This was causing me some issues with rackt/history.

It looks like this test passed before 1.13.1 with deep-equals but fails with is-equal.

I don't know how you want this case to be handled, but since it's a minor version bump, I assume it wasn't intentional. Otherwise, feel free to close.

[mjackson]

I naively assumed that is-equal would handle all the most common cases similarly to deep-equal but I was mistaken. The break wasn't intentional, but now that we've worked out most of the kinks let's just add a test to make our intention clear and close.

[ljharb]

null and undefined aren't equal - isn't toEqual like ===?

[mjackson]

Yes, it is. We should just add a test here to make that clear and close.

[ljharb]

👍

[mjackson]

Wait, I'm not sure what I was saying here. toEqual should be like ==. toBe is for ===.

[ljharb]

oh right, good point