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: #1721
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

Author: silverwind

.eslintignore

@@ -1 +1,4 @@
 lib/punycode.js
+test/fixtures
+test/**/node_modules
+test/parallel/test-fs-non-number-arguments-throw.js

.eslintrc

@@ -8,6 +8,7 @@ ecmaFeatures:
   octalLiterals: true
   binaryLiterals: true
   generators: true
+  forOf: true
 
 rules:
   # Possible Errors
@@ -54,6 +55,10 @@ rules:
   space-infix-ops: 2
   ## require spaces return, throw, case
   space-return-throw-case: 2
+  ## no space before function, eg. 'function()'
+  space-before-function-paren: [2, "never"]
+  ## require space before blocks, eg 'function() {'
+  space-before-blocks: [2, "always"]
   ## require parens for Constructor
   new-parens: 2
   ## max 80 length

Makefile

@@ -375,7 +375,7 @@ bench-idle:
 	./$(NODE_EXE) benchmark/idle_clients.js &
 
 jslint:
-	./$(NODE_EXE) tools/eslint/bin/eslint.js src lib --reset --quiet
+	./$(NODE_EXE) tools/eslint/bin/eslint.js src lib test --reset --quiet
 
 CPPLINT_EXCLUDE ?=
 CPPLINT_EXCLUDE += src/node_lttng.cc

test/.eslintrc

@@ -0,0 +1,5 @@
+## Test-specific linter rules
+
+rules:
+  ## allow unreachable code
+  no-unreachable: 0

test/addons/async-hello-world/test.js

@@ -1,15 +1,16 @@
+'use strict';
 var assert = require('assert');
 var binding = require('./build/Release/binding');
 var called = false;
 
-process.on('exit', function () {
+process.on('exit', function() {
   assert(called);
 });
 
-binding(5, function (err, val) {
+binding(5, function(err, val) {
   assert.equal(null, err);
   assert.equal(10, val);
-  process.nextTick(function () {
+  process.nextTick(function() {
     called = true;
   });
 });

test/addons/at-exit/test.js

@@ -1 +1,2 @@
+'use strict';
 var binding = require('./build/Release/binding');

test/addons/hello-world-function-export/test.js

@@ -1,3 +1,4 @@
+'use strict';
 var assert = require('assert');
 var binding = require('./build/Release/binding');
 assert.equal('world', binding());

test/addons/hello-world/test.js

@@ -1,3 +1,4 @@
+'use strict';
 var assert = require('assert');
 var binding = require('./build/Release/binding');
 assert.equal('world', binding.hello());

test/addons/repl-domain-abort/test.js

@@ -1,3 +1,4 @@
+'use strict';
 var assert = require('assert');
 var repl = require('repl');
 var stream = require('stream');

test/addons/smalloc-alloc/test.js

@@ -1,3 +1,4 @@
+'use strict';
 var assert = require('assert');
 var binding = require('./build/Release/binding');
 var obj = binding.alloc(16);