From 345ff3ce510e9bff7b6ffe0421799feec97128d1 Mon Sep 17 00:00:00 2001 From: iskore Date: Sat, 3 Feb 2018 21:52:07 -0500 Subject: [PATCH] test: added tests for setEncoding error check added test to ensure setEncoding inside socket connection would throw an error Fixes: nodejs#18118 Ref: nodejs#18178 --- .../test-http-socket-encoding-error.js | 23 +++++++++++++++++++ 1 file changed, 23 insertions(+) create mode 100644 test/parallel/test-http-socket-encoding-error.js diff --git a/test/parallel/test-http-socket-encoding-error.js b/test/parallel/test-http-socket-encoding-error.js new file mode 100644 index 00000000000000..cfb96f4f81bc2e --- /dev/null +++ b/test/parallel/test-http-socket-encoding-error.js @@ -0,0 +1,23 @@ +'use strict'; + +const common = require('../common'); +const http = require('http'); + +const server = http.createServer().listen(0, connectToServer); + +server.on('connection', (socket) => { + common.expectsError(() => socket.setEncoding(''), + { + code: 'ERR_METHOD_NOT_IMPLEMENTED', + type: Error + }); + + socket.end(); +}); + +function connectToServer() { + const client = new http.Agent().createConnection(this.address().port, () => { + client.end(); + }) + .on('end', () => server.close()); +}