Migrate FinalizationRegistry tests to per-item callback API

tc39 · Apr 1, 2020 · 966fc11 · 966fc11
1 parent 4199a3b
commit 966fc11
Show file tree

Hide file tree

Showing 19 changed files with 49 additions and 986 deletions.
diff --git a/harness/async-gc.js b/harness/async-gc.js
@@ -21,10 +21,10 @@ function asyncGC(...targets) {
   targets = null;
 
   return Promise.resolve('tick').then(() => asyncGCDeref()).then(() => {
-    var names;
+    var names = [];
 
     // consume iterator to capture names
-    finalizationRegistry.cleanupSome(iter => { names = [...iter]; });
+    finalizationRegistry.cleanupSome(name => { names.push(name); });
 
     if (!names || names.length != length) {
       throw asyncGC.notCollected;

diff --git a/...s/FinalizationRegistry/FinalizationRegistryCleanupIteratorPrototype/Symbol.toStringTag.js b/...s/FinalizationRegistry/FinalizationRegistryCleanupIteratorPrototype/Symbol.toStringTag.js
diff --git a/...zationRegistry/FinalizationRegistryCleanupIteratorPrototype/next-job-not-active-throws.js b/...zationRegistry/FinalizationRegistryCleanupIteratorPrototype/next-job-not-active-throws.js
diff --git a/...uilt-ins/FinalizationRegistry/FinalizationRegistryCleanupIteratorPrototype/next-length.js b/...uilt-ins/FinalizationRegistry/FinalizationRegistryCleanupIteratorPrototype/next-length.js
diff --git a/...tionRegistry/FinalizationRegistryCleanupIteratorPrototype/next-missing-internal-throws.js b/...tionRegistry/FinalizationRegistryCleanupIteratorPrototype/next-missing-internal-throws.js
diff --git a/.../built-ins/FinalizationRegistry/FinalizationRegistryCleanupIteratorPrototype/next-name.js b/.../built-ins/FinalizationRegistry/FinalizationRegistryCleanupIteratorPrototype/next-name.js