fix: do not explicitly depend on jest assertion utils

[SimenB]

What:

Replaced the direct usage of jest-matcher-utils and jest-diff with using the same functions from this.utils injected into each matcher.

Why:

I noticed I had old jest-diff in my node_modules after upgrading to v26 and decided to fix at least one of the entries 🙂

This change allows jest-dom to not depend on explicit versions of these dependencies, instead getting the utils from expect during runtime.

How:

Search and replace, followed by making the linter happy 😀

Checklist:

• Documentation
• Tests
• Updated Type Definitions
• Ready to be merged
  

[codecov]

Codecov Report

Merging #250 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #250   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           23        23           
  Lines          315       315           
  Branches        72        72           
=========================================
  Hits           315       315           

Impacted Files	Coverage Δ
src/to-be-checked.js	100.00% <ø> (ø)
src/to-be-disabled.js	100.00% <ø> (ø)
src/to-be-empty-dom-element.js	100.00% <ø> (ø)
src/to-be-empty.js	100.00% <ø> (ø)
src/to-be-in-the-dom.js	100.00% <ø> (ø)
src/to-be-invalid.js	100.00% <ø> (ø)
src/to-be-partially-checked.js	100.00% <ø> (ø)
src/to-be-required.js	100.00% <ø> (ø)
src/to-be-visible.js	100.00% <ø> (ø)
src/to-contain-element.js	100.00% <ø> (ø)
... and 12 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 92176e1...64fff15. Read the comment docs.

[eps1lon]

This change allows jest-dom to not depend on explicit versions of these dependencies, instead getting the utils from expect during runtime.

So we'd need a peer dependency on jest, no?

[SimenB]

Not, it's expect that injects these, not Jest. You already call expect.extend without a peer dep on it. And you already depend on e.g. this.isNot so in that regard this PR changes nothing. This maybe couple the matchers more to expect so they cannot be plugged into jasmine if that's a concern? expect itself can be used as assertion library for any runner

[SimenB]

@gnapse any holdup here?

[gnapse]

Nope, my bad. Will merge soon.

[gnapse]

@SimenB there have been a couple of new matchers since you proposed this change. I can take care of it later if you don't beat me to it.

[SimenB]

@gnapse updated

[eps1lon]

@SimenB Thanks!

[gnapse]

🎉 This PR is included in version 5.11.4 🎉

The release is available on:

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

Your semantic-release bot 📦🚀