process: remove unhandled rejection deprecation warning

doc/api/deprecations.md

@@ -430,9 +430,12 @@ changes:
   - version: v7.0.0
     pr-url: https://github.com/nodejs/node/pull/8217
     description: Runtime deprecation.
+  - version: REPLACEME
+    pr-url: REPLACEME
+    descripion: Deprecation revoked
 -->
 
-Type: Runtime
+Type: Deprecation revoked
 
 Unhandled promise rejections are deprecated. In the future, promise rejections
 that are not handled will terminate the Node.js process with a non-zero exit

lib/internal/process/promises.js

@@ -103,19 +103,6 @@ function emitWarning(uid, reason) {
   } catch {}
 
   process.emitWarning(warning);
-  emitDeprecationWarning();
-}
-
-let deprecationWarned = false;
-function emitDeprecationWarning() {
-  if (!deprecationWarned) {
-    deprecationWarned = true;
-    process.emitWarning(
-      'Unhandled promise rejections are deprecated. In the future, ' +
-      'promise rejections that are not handled will terminate the ' +
-      'Node.js process with a non-zero exit code.',
-      'DeprecationWarning', 'DEP0018');
-  }
 }
 
 // If this method returns true, at least one more tick need to be

test/message/unhandled_promise_trace_warnings.out

@@ -25,15 +25,6 @@
     at *
     at *
     at *
-(node:*) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
-    at *
-    at *
-    at *
-    at *
-    at *
-    at *
-    at *
-    at *
 (node:*) PromiseRejectionHandledWarning: Promise rejection was handled asynchronously (rejection id: 1)
     at handledRejection (internal/process/promises.js:*)
     at promiseRejectHandler (internal/process/promises.js:*)

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

@@ -3,11 +3,6 @@ const common = require('../common');
 
 common.disableCrashOnUnhandledRejection();
 
-const expectedDeprecationWarning = ['Unhandled promise rejections are ' +
-                                   'deprecated. In the future, promise ' +
-                                   'rejections that are not handled will ' +
-                                   'terminate the Node.js process with a ' +
-                                   'non-zero exit code.', 'DEP0018'];
 const expectedPromiseWarning = ['Unhandled promise rejection. ' +
   'This error originated either by throwing ' +
   'inside of an async function without a catch ' +
@@ -35,7 +30,6 @@ const thorny = new Proxy({}, {
 });
 
 common.expectWarning({
-  DeprecationWarning: expectedDeprecationWarning,
   UnhandledPromiseRejectionWarning: expectedPromiseWarning,
 });
 

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

@@ -4,22 +4,16 @@ const common = require('../common');
 common.disableCrashOnUnhandledRejection();
 
 const expectedValueWarning = ['Symbol()'];
-const expectedDeprecationWarning = ['Unhandled promise rejections are ' +
-                                   'deprecated. In the future, promise ' +
-                                   'rejections that are not handled will ' +
-                                   'terminate the Node.js process with a ' +
-                                   'non-zero exit code.', 'DEP0018'];
 const expectedPromiseWarning = ['Unhandled promise rejection. ' +
   'This error originated either by throwing ' +
   'inside of an async function without a catch ' +
   'block, or by rejecting a promise which was ' +
   'not handled with .catch(). (rejection id: 1)'];
 
 common.expectWarning({
-  DeprecationWarning: expectedDeprecationWarning,
   UnhandledPromiseRejectionWarning: [
     expectedValueWarning,
-    expectedPromiseWarning
+    expectedPromiseWarning,
   ],
 });
 

test/parallel/test-promises-warning-on-unhandled-rejection.js

@@ -27,14 +27,10 @@ process.on('warning', common.mustCall((warning) => {
       );
       break;
     case 2:
-      // One time deprecation warning, first unhandled rejection
-      assert.strictEqual(warning.name, 'DeprecationWarning');
-      break;
-    case 3:
       // Number rejection error displayed. Note it's been stringified
       assert.strictEqual(warning.message, '42');
       break;
-    case 4:
+    case 3:
       // Unhandled rejection warning (won't be handled next tick)
       assert.strictEqual(warning.name, 'UnhandledPromiseRejectionWarning');
       assert(
@@ -43,13 +39,13 @@ process.on('warning', common.mustCall((warning) => {
         'but did not. Had "' + warning.message + '" instead.'
       );
       break;
-    case 5:
+    case 4:
       // Rejection handled asynchronously.
       assert.strictEqual(warning.name, 'PromiseRejectionHandledWarning');
       assert(/Promise rejection was handled asynchronously/
         .test(warning.message));
   }
-}, 6));
+}, 5));
 
 const p = Promise.reject('This was rejected'); // Reject with a string
 setImmediate(common.mustCall(() => p.catch(() => { })));