fix(objectContaining): Fixed a bug where ObjectContaining matched with non-object values.

[mohammednumaan]

Summary

The objectContaining method matches with non-object values, which is inconsistent. Thanks to @flovouin for finding this bug!

This PR fixes: #15420

This PR does the following:

• Adds a conditional/check to ensure the argument passed to expect is an object.
• Write new unit tests to ensure that new code does not break and works as intended.
• Removes invalid test cases for the above bug.
• Adds a few comments to clarify some bits of code.

Test plan

This is the new code I wrote for the objectContaining implementation:

This is the new test cases I wrote for verifying the above added code works:

To run the tests, I ran the following command:

yarn jest /packages/expect/src/__tests__/

The test results:

[netlify]

✅ Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	e9ee49b
🔍 Latest deploy log	https://app.netlify.com/sites/jestjs/deploys/6789ed672900530008289bae
😎 Deploy Preview	https://deploy-preview-15462--jestjs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[github-actions]

This pull request 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.