stream: expect _destroy to call the callback in thennables

Update _destroy to only call onDestroy in case of errors,
waiting for the user implementation to call onDestroy on
the success path of async functions as well.

Fixes: nodejs#40377

lib/internal/streams/destroy.js

@@ -19,6 +19,7 @@ const {
 
 const kDestroy = Symbol('kDestroy');
 const kConstruct = Symbol('kConstruct');
+function nop() {}
 
 function checkError(err, w, r) {
   if (err) {
@@ -112,9 +113,7 @@ function _destroy(self, err, cb) {
       if (typeof then === 'function') {
         then.call(
           result,
-          function() {
-            process.nextTick(onDestroy, null);
-          },
+          nop,
           function(err) {
             process.nextTick(onDestroy, err);
           });

test/parallel/test-stream-construct-async-error.js

@@ -33,6 +33,7 @@ const assert = require('assert');
     async _destroy(err, cb) {
       // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
+      cb(err);
     }
   }