[GR-33869] Backport: Do not print exceptions from Context.close() when terminating the process.

PullRequest: js/2160

Author: gilles-duboscq

graal-nodejs/mx.graal-nodejs/com.oracle.truffle.trufflenode/src/com/oracle/truffle/trufflenode/GraalJSAccess.java

@@ -3109,10 +3109,9 @@ public Object prepareStackTrace(JSRealm realm, DynamicObject error, DynamicObjec
     private void exit(int status) {
         try {
             evaluator.close();
-        } catch (Throwable t) {
-            t.printStackTrace();
+        } finally {
+            System.exit(status);
         }
-        System.exit(status);
     }
 
     public void isolateEnterPolyglotEngine(long callback, long isolate, long param1, long param2, long args, long execArgs) {

graal-nodejs/test/graal/unit/spawn.js

@@ -113,4 +113,11 @@ describe('Spawn', function () {
             assert.strictEqual(result.status, 1);
         });
     }
+    it('should finish gracefully when the process is terminated from an inner context', function () {
+        var code = `require('vm').runInNewContext('process.exit()', { process: process })`;
+        var result = spawnSync(process.execPath, ['-e', code]);
+        assert.strictEqual(result.stdout.toString(), '');
+        assert.strictEqual(result.stderr.toString(), '');
+        assert.strictEqual(result.status, 0);
+    });
 });