nodejs
diff --git a/Collapse file
‎doc/api/deprecations.md‎
Copy file name to clipboardExpand all lines: doc/api/deprecations.md
+4-1Lines changed: 4 additions & 1 deletion
Display the source diff
Display the rich diff b/Collapse file
‎doc/api/deprecations.md‎
Copy file name to clipboardExpand all lines: doc/api/deprecations.md
+4-1Lines changed: 4 additions & 1 deletion
Display the source diff
Display the rich diff
diff --git a/Collapse file
‎lib/internal/util.js‎
Copy file name to clipboardExpand all lines: lib/internal/util.js
+5-2Lines changed: 5 additions & 2 deletions b/Collapse file
‎lib/internal/util.js‎
Copy file name to clipboardExpand all lines: lib/internal/util.js
+5-2Lines changed: 5 additions & 2 deletions
diff --git a/Collapse file
‎test/parallel/test-util-promisify.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-util-promisify.js
+23Lines changed: 23 additions & 0 deletions b/Collapse file
‎test/parallel/test-util-promisify.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-util-promisify.js
+23Lines changed: 23 additions & 0 deletions
@@ -3387,12 +3387,15 @@ Consider using alternatives such as the [`mock`][] helper function.
 
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/49609
+    description: Runtime deprecation.
   - version: REPLACEME
     pr-url: https://github.com/nodejs/node/pull/49647
     description: Documentation-only deprecation.
 -->
 
-Type: Documentation-only
+Type: Runtime
 
 Calling [`util.promisify`][] on a function that returns a <Promise> will ignore
 the result of said promise, which can lead to unhandled promise rejections.
 
@@ -63,7 +63,7 @@ const {
   sleep: _sleep,
   toUSVString: _toUSVString,
 } = internalBinding('util');
-const { isNativeError } = internalBinding('types');
+const { isNativeError, isPromise } = internalBinding('types');
 const { getOptionValue } = require('internal/options');
 
 const noCrypto = !process.versions.openssl;
@@ -409,7 +409,10 @@ function promisify(original) {
           resolve(values[0]);
         }
       });
-      ReflectApply(original, this, args);
+      if (isPromise(ReflectApply(original, this, args))) {
+        process.emitWarning('Calling promisify on a function that returns a Promise is likely a mistake.',
+                            'DeprecationWarning', 'DEP0174');
+      }
     });
   }
 
 
@@ -7,6 +7,29 @@ const vm = require('vm');
 const { promisify } = require('util');
 const { customPromisifyArgs } = require('internal/util');
 
+{
+  const warningHandler = common.mustNotCall();
+  process.on('warning', warningHandler);
+  function foo() {}
+  foo.constructor = (async () => {}).constructor;
+  promisify(foo);
+  process.off('warning', warningHandler);
+}
+
+common.expectWarning(
+  'DeprecationWarning',
+  'Calling promisify on a function that returns a Promise is likely a mistake.',
+  'DEP0174');
+promisify(async (callback) => { callback(); })().then(common.mustCall(() => {
+  // We must add the second `expectWarning` call in the `.then` handler, when
+  // the first warning has already been triggered.
+  common.expectWarning(
+    'DeprecationWarning',
+    'Calling promisify on a function that returns a Promise is likely a mistake.',
+    'DEP0174');
+  promisify(async () => {})().then(common.mustNotCall());
+}));
+
 const stat = promisify(fs.stat);
 
 {