test: make common.js mandatory via linting rule
authorRich Trott <rtrott@gmail.com>
Fri, 2 Oct 2015 04:38:16 +0000 (21:38 -0700)
committerJames M Snell <jasnell@gmail.com>
Thu, 8 Oct 2015 03:39:17 +0000 (20:39 -0700)
commit70c9e4337e032d4d9334725cb280d5674293b764
treea80fbba46064d70e46722f5bf3f201ee48839a2b
parentb7179562aa95400b5a88414ddd3e4815873eaabc
test: make common.js mandatory via linting rule

test/common.js contains code that detects global variable leaks.

This eslint rule checks that a module named `common` is loaded. It is
only applicable to files in the test directory. Tests that intentionally
leak variables can opt out with an eslint-disable comment.

PR-URL: https://github.com/nodejs/node/pull/3157
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
test/.eslintrc
test/common.js
test/parallel/test-domain-crypto.js
test/parallel/test-regression-object-prototype.js
test/parallel/test-repl-autolibs.js
tools/eslint-rules/required-modules.js [new file with mode: 0644]