test: enable linting for tests

Enable linting for the test directory. A number of changes was made so
all tests conform the current rules used by lib and src directories. The
only exception for tests is that unreachable (dead) code is allowed.

test-fs-non-number-arguments-throw had to be excluded from the changes
because of a weird issue on Windows CI.

PR-URL: https://github.com/nodejs/io.js/pull/1721
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

1 files changed, 5 insertions, 4 deletions

diff --git a/test/parallel/test-https-strict.js b/test/parallel/test-https-strict.js
index b2620fa88b..7ab5c3ee82 100644
--- a/test/parallel/test-https-strict.js
+++ b/test/parallel/test-https-strict.js
@@ -1,3 +1,4 @@
+'use strict';
 // disable strict server certificate validation by the client
 process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0';
 
@@ -100,7 +101,7 @@ function listening() {
     if (listenWait === 0) {
       allListening();
     }
-  }
+  };
 }
 
 function makeReq(path, port, error, host, ca) {
@@ -127,7 +128,7 @@ function makeReq(path, port, error, host, ca) {
   }
 
   if (host) {
-    options.headers = { host: host }
+    options.headers = { host: host };
   }
   var req = https.get(options);
   expectResponseCount++;
@@ -136,7 +137,7 @@ function makeReq(path, port, error, host, ca) {
       : port === port3 ? server3
       : null;
 
-  if (!server) throw new Error('invalid port: '+port);
+  if (!server) throw new Error('invalid port: ' + port);
   server.expectCount++;
 
   req.on('response', function(res) {
@@ -150,7 +151,7 @@ function makeReq(path, port, error, host, ca) {
       server3.close();
     }
     res.resume();
-  })
+  });
 }
 
 function allListening() {