expect('null').toContain(null) should fail

[jeysal]

In #8008 (comment), we discovered that expect('asdf null asdf').toContain(null) passes and agreed that it shouldn't. This breaking change should be included in the next major. @SimenB

[pedrottimark]

Let’s also remember #8008 (comment)

Breaking change to require .exec method in addition to .test method for RegExp

EDIT also for .toThrow and .toThrowError see #8033 (comment)

Or to get to the root of the problem, see #8033 (comment)

[pedrottimark]

Please critique 3 possible matcher error messages. Or suggest your own Goldilocks message :)

Baby bear is direct parallel with other error messages:

Mama bear makes the conditional requirement explicit:

Papa bear also displays received value because the message refers to it:

[jeysal]

I vote 3 ("Papa bear").
This is the only really helpful message in case received turns out to be a string when the user actually thought it was an array and intended to use the array version of toContain.

Also, TIL where the term Goldilocks zone originates 😅

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.