lib,test: handle new Iterator global

Refs: v8/v8@80fb767 PR-URL: #51362 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
nodejs · Mar 31, 2024 · 351495e · 351495e
1 parent cdc4437
commit 351495e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -65,6 +65,8 @@ rules:
       # disabled with --without-intl build flag.
     - name: Intl
       message: Use `const { Intl } = globalThis;` instead of the global.
+    - name: Iterator
+      message: Use `const { Iterator } = globalThis;` instead of the global.
     - name: MessageChannel
       message: Use `const { MessageChannel } = require('internal/worker/io');` instead of the global.
     - name: MessageEvent

diff --git a/test/common/globals.js b/test/common/globals.js
@@ -63,6 +63,7 @@ const intrinsics = new Set([
   'SharedArrayBuffer',
   'Atomics',
   'WebAssembly',
+  'Iterator',
 ]);
 
 if (global.gc) {

diff --git a/test/parallel/test-repl-tab-complete.js b/test/parallel/test-repl-tab-complete.js
@@ -632,6 +632,7 @@ const builtins = [
     'Int32Array',
     'Int8Array',
     ...(common.hasIntl ? ['Intl'] : []),
+    'Iterator',
     'inspector',
     'isFinite',
     'isNaN',