Ma77Ball opened a new pull request, #4999:
URL: https://github.com/apache/texera/pull/4999
### What changes were proposed in this PR?
Replace the `throw err` / `throw reason` inside the
`process.on("uncaughtException")` and `process.on("unhandledRejection")`
handlers in `frontend/src/jsdom-svg-polyfill.ts` with `console.error(...)`.
Re-throwing in `uncaughtException` aborts the Node process, crashing the
Vitest worker mid-run and leaving the runner hanging until the CI timeout.
Logging instead lets Vitest's own error handling fail the offending spec
without killing sibling tests in the same worker.
### Any related issues, documentation, or discussions?
Closes: #4998
### How was this PR tested?
Ran `yarn ng test --watch=false` locally — full suite passes cleanly. CI
runs on the branch that no longer hangs.
### Was this PR authored or co-authored using generative AI tooling?
Discovered with Claude Opus 4.7 in compliance with ASF
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
