test,tools: limit lint tolerance of gc global

Lint rules permitted the `gc` global in any test file. This change limits it to just the files that need it.
nodejs · Trott · Apr 21, 2016 · Apr 28, 2016 · Apr 28, 2016 · c91423fa9e266a1a400fd7e923c757f9a0667fa7
commit c91423fa9e266a1a400fd7e923c757f9a0667fa7
diff --git a/test/.eslintrc b/test/.eslintrc
@@ -3,6 +3,3 @@
 rules:
   ## common module is mandatory in tests
   required-modules: [2, "common"]
-
-globals:
-  gc: false
diff --git a/test/addons/buffer-free-callback/test.js b/test/addons/buffer-free-callback/test.js
@@ -1,5 +1,6 @@
 'use strict';
 // Flags: --expose-gc
+/* global gc:false */
 
 require('../../common');
 var binding = require('./build/Release/binding');

diff --git a/test/common.js b/test/common.js
@@ -275,7 +275,7 @@ var knownGlobals = [setTimeout,
                     global];
 
 if (global.gc) {
-  knownGlobals.push(gc);
+  knownGlobals.push(global.gc);
 }
 
 if (global.DTRACE_HTTP_SERVER_RESPONSE) {

diff --git a/test/gc/.eslintrc b/test/gc/.eslintrc
@@ -0,0 +1,2 @@
+globals:
+  gc: false
diff --git a/test/parallel/test-http-parser-bad-ref.js b/test/parallel/test-http-parser-bad-ref.js
@@ -3,6 +3,7 @@
 // problem.
 
 // Flags: --expose_gc
+/* global gc:false */
 
 require('../common');
 var assert = require('assert');

diff --git a/test/parallel/test-vm-create-and-run-in-context.js b/test/parallel/test-vm-create-and-run-in-context.js
@@ -1,5 +1,6 @@
 'use strict';
 // Flags: --expose-gc
+/* global gc:false */
 require('../common');
 var assert = require('assert');
 

diff --git a/test/parallel/test-vm-run-in-new-context.js b/test/parallel/test-vm-run-in-new-context.js
@@ -1,5 +1,6 @@
 'use strict';
 // Flags: --expose-gc
+/* global gc:false */
 
 const common = require('../common');
 const assert = require('assert');

diff --git a/test/pummel/test-net-connect-memleak.js b/test/pummel/test-net-connect-memleak.js
@@ -1,5 +1,6 @@
 'use strict';
 // Flags: --expose-gc
+/* global gc:false */
 
 var common = require('../common');
 var assert = require('assert');

diff --git a/test/pummel/test-regress-GH-814.js b/test/pummel/test-regress-GH-814.js
@@ -1,5 +1,6 @@
 'use strict';
 // Flags: --expose_gc
+/* global gc:false */
 
 function newBuffer(size, value) {
   var buffer = Buffer.allocUnsafe(size);

diff --git a/test/pummel/test-regress-GH-814_2.js b/test/pummel/test-regress-GH-814_2.js
@@ -1,5 +1,6 @@
 'use strict';
 // Flags: --expose_gc
+/* global gc:false */
 
 var common = require('../common');
 

diff --git a/test/pummel/test-tls-connect-memleak.js b/test/pummel/test-tls-connect-memleak.js
@@ -1,5 +1,6 @@
 'use strict';
 // Flags: --expose-gc
+/* global gc:false */
 
 var common = require('../common');
 var assert = require('assert');