fix: catch synchronous error

[juanrgm]

Summary

Synchronous (non-promise) errors cannot be caught by ErrorBoundary.

https://playground.solidjs.com/anonymous/98e1258d-67c8-4c53-b9cb-4bfa029d451e

import { render } from "solid-js/web";
import { ErrorBoundary, createResource } from "solid-js";

function Counter() {
  const [res] = createResource(() => {
    if (1) throw new Error(); // this error is not captured by ErrorBoundary
    return 1;
  });

  return (
    <ErrorBoundary fallback={(error) => <>{error.toString()}</>}>
      {res()}
    </ErrorBoundary>
  );
}

render(() => <Counter />, document.getElementById("app")!);

Using an async function as fetcher it works:

https://playground.solidjs.com/anonymous/0e5e5348-57ef-41b3-aebf-66f8567122c8

How did you test this change?

An unit test was added to the commit.

[changeset-bot]

🦋 Changeset detected

Latest commit: 1441227

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
solid-js	Patch
test-integration	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coveralls]

Pull Request Test Coverage Report for Build 14923115814

Details

• 15 of 15 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.6%) to 85.752%

---

Totals
Change from base Build 14767323724:	0.6%
Covered Lines:	2333
Relevant Lines:	2652

---

💛 - Coveralls

[ryansolid]

Thanks