test: refactored to remove unnecessary variables

PR-URL: #17553 Reviewed-By: Jon Moss <me@jonathanmoss.me> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
nodejs · Dec 20, 2017 · bb96831 · bb96831
1 parent 08220a3
commit bb96831
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/test/parallel/test-net-connect-options-fd.js b/test/parallel/test-net-connect-options-fd.js
@@ -83,14 +83,13 @@ const forAllClients = (cb) => common.mustCall(cb, CLIENT_VARIANTS);
       path: serverPath
     });
     const getConnectCb = (index) => common.mustCall(function clientOnConnect() {
-      const client = this;
       // Test if it's wrapping an existing fd
       assert(handleMap.has(index));
       const oldHandle = handleMap.get(index);
       assert.strictEqual(oldHandle.fd, this._handle.fd);
-      client.write(String(oldHandle.fd));
+      this.write(String(oldHandle.fd));
       console.error(`[Pipe]Sending data through fd ${oldHandle.fd}`);
-      client.on('error', function(err) {
+      this.on('error', function(err) {
         console.error(err);
         assert.fail(null, null, `[Pipe Client]${err}`);
       });

diff --git a/test/parallel/test-net-connect-options-path.js b/test/parallel/test-net-connect-options-path.js
@@ -20,9 +20,8 @@ const CLIENT_VARIANTS = 12;
   }, CLIENT_VARIANTS))
   .listen(serverPath, common.mustCall(function() {
     const getConnectCb = () => common.mustCall(function() {
-      const client = this;
-      client.end();
-      client.on('close', common.mustCall(function() {
+      this.end();
+      this.on('close', common.mustCall(function() {
         counter++;
         if (counter === CLIENT_VARIANTS) {
           server.close();