(fix) - handle hooks that throw during cleanup

[JoviDeCroock]

When a hook throws an error during cleanup we want to run all the other cleanups so we can avoid memory leaks and end in a consistent UI.

For this we'll need to always clear the hook._cleanup function so that we don't call it twice. During unmount we want to ensure we run through all hooks and even if we error we defer that to only bubble up the latest error.

fixes #3329

[coveralls]

Coverage increased (+0.001%) to 99.625% when pulling 51d3402 on handle-hook-cleanup-errors into e0e1704 on master.

[github-actions]

Size Change: +47 B (0%)

Total Size: 42.1 kB

Filename	Size	Change
hooks/dist/hooks.js	1.14 kB	+14 B (1%)
hooks/dist/hooks.module.js	1.16 kB	+14 B (1%)
hooks/dist/hooks.umd.js	1.22 kB	+19 B (1%)

ℹ️ View Unchanged

Filename	Size	Change
compat/dist/compat.js	3.44 kB	0 B
compat/dist/compat.module.js	3.42 kB	0 B
compat/dist/compat.umd.js	3.49 kB	0 B
debug/dist/debug.js	2.99 kB	0 B
debug/dist/debug.module.js	2.98 kB	0 B
debug/dist/debug.umd.js	3.07 kB	0 B
devtools/dist/devtools.js	231 B	0 B
devtools/dist/devtools.module.js	240 B	0 B
devtools/dist/devtools.umd.js	307 B	0 B
dist/preact.js	3.97 kB	0 B
dist/preact.min.js	4 kB	0 B
dist/preact.module.js	3.99 kB	0 B
dist/preact.umd.js	4.03 kB	0 B
jsx-runtime/dist/jsxRuntime.js	317 B	0 B
jsx-runtime/dist/jsxRuntime.module.js	327 B	0 B
jsx-runtime/dist/jsxRuntime.umd.js	395 B	0 B
test-utils/dist/testUtils.js	437 B	0 B
test-utils/dist/testUtils.module.js	439 B	0 B
test-utils/dist/testUtils.umd.js	515 B	0 B

_{compressed-size-action}

[developit]

LGTM with silly nit