Return value for number 0

[chaijs]

Return value for number 0

[keithamus]

This is important because many tests assert the difference between +0/-0. We should update Chai to reflect this change.

[flaambe]

And one more question, it seems custom functions don't work, or I have to pass some parameters?

5) utilities
       inspect with custom stylize-calling inspect()s:

      AssertionError: expected '{ outer: { inspect: [Function inspect] } }' to equal '{ outer: Object content }'
      + expected - actual

      -{ outer: { inspect: [Function inspect] } }
      +{ outer: Object content }

[keithamus]

This is a change in the loupe API to avoid surprising behaviours for calls that have an inspect that gets called when it is not desirable. The behaviour is opt in now, so you need to pass customInspect: true, e.g loupe(obj, { customInspect: true }). We can do that inside Chai to avoid breaking change, and turn it off in Chai 5.

[flaambe]

I tried to pass this parameter, but it seems the execution does not reach this line:

loupe/index.js

Line 142 in 0c75417

if (customInspect && value) {

so the result is the same.

[keithamus]

Ahh yeah its because of this above:

loupe/index.js

Lines 129 to 133 in 0c75417

	const proto = value ? Object.getPrototypeOf(value) : false
	// If it's a plain Object then use Loupe's inspector
	if (proto === Object.prototype || proto === null) {
	return inspectObject(value, options)
	}

We should verify this behaviour works well in Loupe and release a patch. The block I linked above should add !customInspect && as an extra conditional, IMO.

[github-actions]

🎉 This issue has been resolved in version 2.0.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀