stream: fix misleading error message

The method to implement is `_write` not `_transform`. PR-URL: #18604 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Matteo Collina <[email protected]>
nodejs · Feb 21, 2018 · ba46103 · ba46103
1 parent 15db296
commit ba46103
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/lib/_stream_writable.js b/lib/_stream_writable.js
@@ -551,7 +551,7 @@ function clearBuffer(stream, state) {
 }
 
 Writable.prototype._write = function(chunk, encoding, cb) {
-  cb(new errors.Error('ERR_METHOD_NOT_IMPLEMENTED', '_transform'));
+  cb(new errors.Error('ERR_METHOD_NOT_IMPLEMENTED', '_write'));
 };
 
 Writable.prototype._writev = null;

diff --git a/test/parallel/test-stream-writable-constructor-set-methods.js b/test/parallel/test-stream-writable-constructor-set-methods.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 
 const Writable = require('stream').Writable;
@@ -26,6 +26,16 @@ w2.write(Buffer.from('blerg'));
 w2.write(Buffer.from('blerg'));
 w2.end();
 
+const w3 = new Writable();
+
+w3.on('error', common.expectsError({
+  type: Error,
+  code: 'ERR_METHOD_NOT_IMPLEMENTED',
+  message: 'The _write method is not implemented'
+}));
+
+w3.end(Buffer.from('blerg'));
+
 process.on('exit', function() {
   assert.strictEqual(w._write, _write);
   assert(_writeCalled);