pkg: react-hooks-testing-library major

[ntucker]

BREAKING CHANGE:

• result.current, result.error is now undefined after suspense, rather than null
• interval will now default to 50ms in async utils
• timeout will now default to 1000ms in async utils
• suppressErrors has been removed from async utils
• Adjust types so that react renderer exports don't required extra generic parameter
• Importing from renderHook and act from @testing-library/react-hooks will now auto-detect which renderer to used based on the project's dependencies
  • peerDependencies are now optional to support different dependencies being required
  • This means there will be no warning if the dependency is not installed at all, but it will still warn if an incompatible version is installed
  • Auto-detection won't work with bundlers (e.g. Webpack). Please use as specific renderer import instead
    (see https://github.com/testing-library/react-hooks-testing-library/releases/tag/v5.0.0)

Motivation

Keep up with the latest

Solution

Upgrade react-hooks-testing-library from 3.2 to 5
Had to change all the checks for suspense to undefined from null.
Deprecated cleanup() as it already happens on unmount.
Added cleanup() to externalprovider in tests to reflect behavior of cacheprovider.
Updated redux docs to include calling manager's init

Waiting on testing-library/react-hooks-testing-library#554 to be able to use this version.

[ntucker]

Might have to consider forking or discontinuing use of this library as it doesn't seem to be maintained well.

[github-actions]

Size Change: +230 B (0%)

Total Size: 109 kB

Filename	Size	Change
packages/test/dist/index.cjs.js	3 kB	+230 B (7%)	🔍

ℹ️ View Unchanged

Filename	Size	Change
packages/core/dist/index.cjs.js	12.5 kB	0 B
packages/core/dist/index.umd.min.js	5.73 kB	0 B
packages/endpoint/dist/index.cjs.js	1.76 kB	0 B
packages/endpoint/dist/index.umd.min.js	1.29 kB	0 B
packages/hooks/dist/index.cjs.js	714 B	0 B
packages/hooks/dist/index.umd.min.js	429 B	0 B
packages/legacy/dist/index.cjs.js	579 B	0 B
packages/legacy/dist/index.umd.min.js	511 B	0 B
packages/normalizr/dist/normalizr.amd.js	10.3 kB	0 B
packages/normalizr/dist/normalizr.amd.min.js	5.59 kB	0 B
packages/normalizr/dist/normalizr.browser.js	10.3 kB	0 B
packages/normalizr/dist/normalizr.browser.min.js	5.59 kB	0 B
packages/normalizr/dist/normalizr.js	10.1 kB	0 B
packages/normalizr/dist/normalizr.min.js	5.59 kB	0 B
packages/normalizr/dist/normalizr.umd.js	10.4 kB	0 B
packages/normalizr/dist/normalizr.umd.min.js	5.67 kB	0 B
packages/rest-hooks/dist/index.cjs.js	7.67 kB	0 B
packages/rest-hooks/dist/index.umd.min.js	4.08 kB	0 B
packages/rest/dist/index.cjs.js	3.88 kB	0 B
packages/rest/dist/index.umd.min.js	2.14 kB	0 B
packages/use-enhanced-reducer/dist/index.cjs.js	1.08 kB	0 B
packages/use-enhanced-reducer/dist/index.umd.min.js	607 B	0 B

_{compressed-size-action}

[ljharb]

why ~ ?

[ntucker]

As outlined in a previous PR - there was an uncaught breaking change in a minor upgrade to this library. This lets us validate all minor upgrades before our users potentially get broken things.

[ljharb]

oof, gross