Fix uncatchable error inside a promise (#810)

[laishere]

Fix issue #810

[saghul]

It would be good to have a test for this.

[laishere]

How to run all the tests? @saghul

I just ran

run-test262.exe tests.conf

Result: 0/29 errors, 3 excluded

Cannot run test262.conf.

BTW, I should add my test.js to tests directory right?

[saghul]

This wouldn't be a 262 test, so yeah adding it to the tests/ directory is the way.

[laishere]

I just realized that we need an interrupt handler to test it. So I can't do it in pure js. So would need a separate test or modify run-test262.c

[saghul]

Yes it would likely need to be its own C file with the simplified test.

[laishere]

Should I add Makefile and CMakeLists.txt target for this test? @saghul

[saghul]

CMake should be the only requirement, since it shouldn't have EXCLUDE_FROM_ALL

[saghul]

Looking good! Can you pl run the test from the CI files? Building it won't make it run.

[laishere]

Sure!

[laishere]

Updated.

[saghul]

Can you please add it to (some of) the windows targets?

[laishere]

No problem, I am working on it now!

[laishere]

Updated.

[saghul]

LGTM nice work! @bnoordhuis can you have a quick look?

[laishere]

Updated.