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, 6 insertions, 3 deletions

diff --git a/test/parallel/test-repl-reset-event.js b/test/parallel/test-repl-reset-event.js
index eee31e24be..e6157956d4 100644
--- a/test/parallel/test-repl-reset-event.js
+++ b/test/parallel/test-repl-reset-event.js
@@ -1,3 +1,4 @@
+'use strict';
 var common = require('../common');
 common.globalCheck = false;
 
@@ -7,7 +8,7 @@ var Stream = require('stream');
 
 // create a dummy stream that does nothing
 var dummy = new Stream();
-dummy.write = dummy.pause = dummy.resume = function(){};
+dummy.write = dummy.pause = dummy.resume = function() {};
 dummy.readable = dummy.writable = true;
 
 function testReset(cb) {
@@ -20,7 +21,8 @@ function testReset(cb) {
   r.on('reset', function(context) {
     assert(!!context, 'REPL did not emit a context with reset event');
     assert.equal(context, r.context, 'REPL emitted incorrect context');
-    assert.equal(context.foo, undefined, 'REPL emitted the previous context, and is not using global as context');
+    assert.equal(context.foo, undefined, 'REPL emitted the previous context' +
+                 ', and is not using global as context');
     context.foo = 42;
     cb();
   });
@@ -35,7 +37,8 @@ function testResetGlobal(cb) {
   });
   r.context.foo = 42;
   r.on('reset', function(context) {
-    assert.equal(context.foo, 42, '"foo" property is missing from REPL using global as context');
+    assert.equal(context.foo, 42,
+                 '"foo" property is missing from REPL using global as context');
     cb();
   });
   r.resetContext();