process: fix promise catching

Fixes: #30953

PR-URL: #30957
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: David Carlier <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>

Author: pd4d10

lib/internal/process/promises.js

@@ -123,7 +123,8 @@ function handledRejection(promise) {
       return;
     }
   }
-  setHasRejectionToWarn(false);
+  if (maybeUnhandledPromises.size === 0 && asyncHandledRejections.length === 0)
+    setHasRejectionToWarn(false);
 }
 
 const unhandledRejectionErrName = 'UnhandledPromiseRejectionWarning';

test/parallel/test-promises-unhandled-rejections.js

@@ -718,3 +718,15 @@ asyncTest(
     let timer = setTimeout(common.mustNotCall(), 10000);
   },
 );
+
+// https://github.com/nodejs/node/issues/30953
+asyncTest(
+  'Catching a promise should not take effect on previous promises',
+  function(done) {
+    onUnhandledSucceed(done, function(reason, promise) {
+      assert.strictEqual(reason, '1');
+    });
+    Promise.reject('1');
+    Promise.reject('2').catch(function() {});
+  }
+);