test: define SharedArrayBuffer as a known global

cjihrig · MylesBorins · commit 3afeaaa93458 · 2018-05-22T15:24:54.000-04:00
This commit defines SharedArrayBuffer as a global for all tests, rather than adding comments to individual tests. PR-URL: #20849 Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Yuta Hiroto <hello@hiroppy.me> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
diff --git a/test/.eslintrc.yaml b/test/.eslintrc.yaml
@@ -51,3 +51,4 @@ globals:
   WebAssembly: false
   BigInt64Array: false
   BigUint64Array: false
+  SharedArrayBuffer: false
diff --git a/test/parallel/test-buffer-sharedarraybuffer.js b/test/parallel/test-buffer-sharedarraybuffer.js
@@ -1,4 +1,3 @@
-/* global SharedArrayBuffer */
 'use strict';
 
 require('../common');
diff --git a/test/parallel/test-util-format-shared-arraybuffer.js b/test/parallel/test-util-format-shared-arraybuffer.js
@@ -2,7 +2,5 @@
 require('../common');
 const assert = require('assert');
 const util = require('util');
-
-/* global SharedArrayBuffer */
 const sab = new SharedArrayBuffer(4);
 assert.strictEqual(util.format(sab), 'SharedArrayBuffer { byteLength: 4 }');
diff --git a/test/parallel/test-util-types.js b/test/parallel/test-util-types.js
@@ -1,5 +1,4 @@
 // Flags: --harmony-bigint --experimental-vm-modules
-/* global SharedArrayBuffer */
 'use strict';
 const common = require('../common');
 const fixtures = require('../common/fixtures');

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		`-/* global SharedArrayBuffer */`
`2`	`1`	`'use strict';`
`3`	`2`
`4`	`3`	`require('../common');`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`// Flags: --harmony-bigint --experimental-vm-modules`
`2`		`-/* global SharedArrayBuffer */`
`3`	`2`	`'use strict';`
`4`	`3`	`const common = require('../common');`
`5`	`4`	`const fixtures = require('../common/fixtures');`