stream: do not throw multiple callback errors in writable

Align ERR_MULTIPLE_CALLBACK in Writable with the rest
of error handling as well as how the error is implemented
in Transform.

PR-URL: #30614
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>

Author: ronag

lib/_stream_writable.js

@@ -477,8 +477,10 @@ function onwrite(stream, er) {
   const sync = state.sync;
   const cb = state.writecb;
 
-  if (typeof cb !== 'function')
-    throw new ERR_MULTIPLE_CALLBACK();
+  if (typeof cb !== 'function') {
+    errorOrDestroy(stream, new ERR_MULTIPLE_CALLBACK());
+    return;
+  }
 
   state.writing = false;
   state.writecb = null;

test/parallel/test-stream-writable-callback-twice.js

@@ -0,0 +1,14 @@
+'use strict';
+const common = require('../common');
+const { Writable } = require('stream');
+const stream = new Writable({
+  write(chunk, enc, cb) { cb(); cb(); }
+});
+
+stream.on('error', common.expectsError({
+  type: Error,
+  message: 'Callback called multiple times',
+  code: 'ERR_MULTIPLE_CALLBACK'
+}));
+
+stream.write('foo');