Some recent changes to harness/asyncHelpers.js regressed some harness tests; test/harness/asyncHelpers-throwsAsync-func-throws-sync.js and test/harness/asyncHelpers-throwsAsync-no-arg.js to be more specific. The cause of this was fa3d0246e74da7844843ebfb4c71453c616b923a and feb400c68598138b4e5a056c69b94bc1108d8a51, which made some changes that broke those two tests.
This PR fixes this by slightly modifying asyncHelpers.js so that it passes the two tests, and also adds a new test to ensure #4186 is still fixed even with the new changes.
Some recent changes to
harness/asyncHelpers.jsregressed some harness tests;test/harness/asyncHelpers-throwsAsync-func-throws-sync.jsandtest/harness/asyncHelpers-throwsAsync-no-arg.jsto be more specific. The cause of this was fa3d0246e74da7844843ebfb4c71453c616b923a and feb400c68598138b4e5a056c69b94bc1108d8a51, which made some changes that broke those two tests.This PR fixes this by slightly modifying
asyncHelpers.jsso that it passes the two tests, and also adds a new test to ensure #4186 is still fixed even with the new changes.