Browser tests are inconsistently failing due to dynamic `istanbul` import about fuels-ts HOT 4 CLOSED

The message may be misleading, as the logs the following errors are present:

Module "child_process" has been externalized for browser compatibility. Cannot access "child_process.spawn" in client code. See https://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
Module "child_process" has been externalized for browser compatibility. Cannot access "child_process.exec" in client code. See https://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.

This appears to have been introduced in the following PR (based upon the CI failures):

• #2378

I can imagine that there is failure path that is causing child_process to be called in the browser env, not 💯 sure where yet.

cc @nedsalk

from fuels-ts.

Indeed child_process will not work in browser. When bundling, every time vite identifies that dep it'll give that warning. However, we should be dynamically importing it everywhere it is actually needed in tests. I'd expect it to throw on that line otherwise, like it did in the past before we had a dynamic import.

We could try the v8 coverage provider.

from fuels-ts.

Bumping priority to p0 as it seems to affect everything else. cc @nedsalk

from fuels-ts.

I see that some of the CI jobs are now passing upon re-running the jobs, so the PRs in my description may become outdated.

from fuels-ts.