AsyncIterator, pt 3

Author: rwaldron

test/built-ins/AsyncIterator/prototype/asIndexedPairs/abrupt-iterator-get-next.js

@@ -7,41 +7,51 @@ description: >
 info: |
   %AsyncIterator.prototype%.asIndexedPairs ( )
 
-  %AsyncIterator.prototype%.asIndexedPairs is a built-in async generator function which, when called, performs the following prelude steps:
-
-    Let iterated be ? GetIteratorDirect(this value).
-
-  The body of %AsyncIterator.prototype%.asIndexedPairs is composed of the following steps:
-
+  Let iterated be ? GetIteratorDirect(this value).
+  Let closure be a new Abstract Closure with no parameters that captures iterated and performs the following steps when called:
     Let index be 0.
     Let lastValue be undefined.
     Repeat,
       Let next be ? Await(? IteratorNext(iterated, lastValue)).
       ...
 
+  IteratorNext ( iteratorRecord [ , value ] )
+
+    If value is not present, then
+      Let result be ? Call(iteratorRecord.[[NextMethod]], iteratorRecord.[[Iterator]]).
+    Else,
+      Let result be ? Call(iteratorRecord.[[NextMethod]], iteratorRecord.[[Iterator]], « value »).
+    If Type(result) is not Object, throw a TypeError exception.
+    Return result.
+
+
 includes: [iterators.js]
 features: [async-iteration, iterator-helpers]
 flags: [async]
 ---*/
+let nextGets = 0;
 class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
   get next() {
+    nextGets++;
     throw new Test262Error();
   }
 }
 
 (async () => {
-  let count = 0;
-  let iterator = new Test262AsyncIteratorAbrupt([0, 1, 2, 3]);
-  assert.sameValue(iterator.nextCalls, 0, 'The value of iterator.nextCalls is 0');
+  let tryCount = 0;
+  let catchCount = 0;
+  let iterator = new Test262AsyncIteratorAbrupt([1, 2]);
+  assert.sameValue(nextGets, 0, 'The value of `nextGets` is 0');
 
   try {
+    tryCount++;
     iterator.asIndexedPairs();
   } catch (e) {
-    count++;
+    catchCount++;
     assert.sameValue(e instanceof Test262Error, true, 'The result of evaluating `(e instanceof Test262Error)` is true');
   }
 
-  assert.sameValue(count, 1, 'The value of `count` is 1');
-  assert.sameValue(iterator.nextCalls, 0, 'The value of iterator.nextCalls is 0');
-  assert.sameValue(iterator.iterable.length, 4, 'The value of iterator.iterable.length is 4');
+  assert.sameValue(tryCount, 1, 'The value of `tryCount` is 1');
+  assert.sameValue(catchCount, 1, 'The value of `catchCount` is 1');
+  assert.sameValue(nextGets, 1, 'The value of `nextGets` is 1');
 })().then($DONE, $DONE);

test/built-ins/AsyncIterator/prototype/asIndexedPairs/abrupt-iterator-get-return.js

@@ -7,12 +7,8 @@ description: >
 info: |
   %AsyncIterator.prototype%.asIndexedPairs ( )
 
-  %AsyncIterator.prototype%.asIndexedPairs is a built-in async generator function which, when called, performs the following prelude steps:
-
-    Let iterated be ? GetIteratorDirect(this value).
-
-  The body of %AsyncIterator.prototype%.asIndexedPairs is composed of the following steps:
-
+  Let iterated be ? GetIteratorDirect(this value).
+  Let closure be a new Abstract Closure with no parameters that captures iterated and performs the following steps when called:
     Let index be 0.
     Let lastValue be undefined.
     Repeat,
@@ -23,14 +19,15 @@ info: |
       Set index to index + 1.
       Set lastValue to Yield(pair).
       IfAbruptCloseAsyncIterator(iterated, lastValue).
+  ...
 
 features: [async-iteration, iterator-helpers]
 flags: [async]
 ---*/
-let genCount = 0;
+let yieldCount = 0;
 
 async function* g() {
-  genCount++;
+  yieldCount++;
 }
 
 (async () => {
@@ -56,7 +53,7 @@ async function* g() {
     assert.sameValue(e instanceof Test262Error, true, 'The result of evaluating `(e instanceof Test262Error)` is true');
   }
 
-  assert.sameValue(genCount, 1, 'The value of `genCount` is 1');
+  assert.sameValue(yieldCount, 1, 'The value of `yieldCount` is 1');
   assert.sameValue(tryCount, 1, 'The value of `tryCount` is 1');
   assert.sameValue(catchCount, 1, 'The value of `catchCount` is 1');
   assert.sameValue(returnCount, 1, 'The value of `returnCount` is 1');

test/built-ins/AsyncIterator/prototype/asIndexedPairs/abrupt-iterator-next-done.js

@@ -7,28 +7,29 @@ description: >
 info: |
   %AsyncIterator.prototype%.asIndexedPairs ( )
 
-  %AsyncIterator.prototype%.asIndexedPairs is a built-in async generator function which, when called, performs the following prelude steps:
-
-    Let iterated be ? GetIteratorDirect(this value).
-
-  The body of %AsyncIterator.prototype%.asIndexedPairs is composed of the following steps:
-
+  Let iterated be ? GetIteratorDirect(this value).
+  Let closure be a new Abstract Closure with no parameters that captures iterated and performs the following steps when called:
     Let index be 0.
     Let lastValue be undefined.
     Repeat,
       Let next be ? Await(? IteratorNext(iterated, lastValue)).
       If ? IteratorComplete(next) is true, return undefined.
       ...
 
+  IteratorComplete ( iterResult )
+
+    Return ! ToBoolean(? Get(iterResult, "done")).
+
 includes: [iterators.js]
 features: [async-iteration, iterator-helpers]
 flags: [async]
 ---*/
 let doneCount = 0;
+let nextCalls = 0;
 
 class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
   async next() {
-    this.nextCalls++;
+    nextCalls++;
 
     return {
       get done() {
@@ -43,14 +44,12 @@ class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
 
 (async () => {
   let count = 0;
-  let iterator = new Test262AsyncIteratorAbrupt([0, 1, 2, 3]);
-  assert.sameValue(iterator.nextCalls, 0, 'The value of iterator.nextCalls is 0');
-  let indexedPairs = iterator.asIndexedPairs();
+  let iterator = new Test262AsyncIteratorAbrupt([1, 2]).asIndexedPairs();
 
   try {
     count++;
 
-    for await (const [i, v] of indexedPairs) {
+    for await (const [i, v] of iterator) {
       $DONE('for await body must not be reachable');
     }
   } catch (e) {
@@ -59,6 +58,5 @@ class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
   }
 
   assert.sameValue(doneCount, 1, 'The value of `doneCount` is 1');
-  assert.sameValue(iterator.nextCalls, 1, 'The value of iterator.nextCalls is 1');
-  assert.sameValue(iterator.iterable.length, 4, 'The value of iterator.iterable.length is 4');
+  assert.sameValue(nextCalls, 1, 'The value of `nextCalls` is 1');
 })().then($DONE, $DONE);

test/built-ins/AsyncIterator/prototype/asIndexedPairs/abrupt-iterator-next-not-object.js

@@ -0,0 +1,54 @@
+// Copyright (C) 2020 Rick Waldron. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+/*---
+esid: sec-asynciteratorprototype.asindexedpairs
+description: >
+  Returns abrupt when next accessor is abrupt.
+info: |
+  %AsyncIterator.prototype%.asIndexedPairs ( )
+
+  Let iterated be ? GetIteratorDirect(this value).
+  Let closure be a new Abstract Closure with no parameters that captures iterated and performs the following steps when called:
+    Let index be 0.
+    Let lastValue be undefined.
+    Repeat,
+      Let next be ? Await(? IteratorNext(iterated, lastValue)).
+      ...
+
+  IteratorNext ( iteratorRecord [ , value ] )
+
+    If value is not present, then
+      Let result be ? Call(iteratorRecord.[[NextMethod]], iteratorRecord.[[Iterator]]).
+    Else,
+      Let result be ? Call(iteratorRecord.[[NextMethod]], iteratorRecord.[[Iterator]], « value »).
+    If Type(result) is not Object, throw a TypeError exception.
+    Return result.
+
+
+includes: [iterators.js]
+features: [async-iteration, iterator-helpers]
+flags: [async]
+---*/
+let nextCalls = 0;
+class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
+  async next() {
+    nextCalls++;
+    return null;
+  }
+}
+
+(async () => {
+  let catchCount = 0;
+  let iterator = (new Test262AsyncIteratorAbrupt([1, 2])
+).asIndexedPairs();
+
+  try {
+    await iterator.next();
+  } catch (e) {
+    catchCount++;
+    assert.sameValue(e instanceof TypeError, true, 'The result of evaluating `(e instanceof TypeError)` is true');
+  }
+
+  assert.sameValue(catchCount, 1, 'The value of `catchCount` is 1');
+  assert.sameValue(nextCalls, 1, 'The value of `nextCalls` is 1');
+})().then($DONE, $DONE);

test/built-ins/AsyncIterator/prototype/asIndexedPairs/abrupt-iterator-next-value.js

@@ -7,12 +7,8 @@ description: >
 info: |
   %AsyncIterator.prototype%.asIndexedPairs ( )
 
-  %AsyncIterator.prototype%.asIndexedPairs is a built-in async generator function which, when called, performs the following prelude steps:
-
-    Let iterated be ? GetIteratorDirect(this value).
-
-  The body of %AsyncIterator.prototype%.asIndexedPairs is composed of the following steps:
-
+  Let iterated be ? GetIteratorDirect(this value).
+  Let closure be a new Abstract Closure with no parameters that captures iterated and performs the following steps when called:
     Let index be 0.
     Let lastValue be undefined.
     Repeat,
@@ -21,15 +17,20 @@ info: |
       Let value be ? IteratorValue(next).
       ...
 
+  IteratorValue ( iterResult )
+
+    Return ? Get(iterResult, "value").
+
 includes: [iterators.js]
 features: [async-iteration, iterator-helpers]
 flags: [async]
 ---*/
+let nextCalls = 0;
 let valueCount = 0;
 
 class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
   async next() {
-    this.nextCalls++;
+    nextCalls++;
 
     return {
       get value() {
@@ -42,8 +43,7 @@ class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
 
 (async () => {
   let count = 0;
-  let iterator = new Test262AsyncIteratorAbrupt([0, 1, 2, 3]);
-  assert.sameValue(iterator.nextCalls, 0, 'The value of iterator.nextCalls is 0');
+  let iterator = new Test262AsyncIteratorAbrupt([1, 2]);
   let indexedPairs = iterator.asIndexedPairs();
 
   try {
@@ -57,7 +57,6 @@ class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
     assert.sameValue(e instanceof Test262Error, true, 'The result of evaluating `(e instanceof Test262Error)` is true');
   }
 
+  assert.sameValue(nextCalls, 1, 'The value of `nextCalls` is 1');
   assert.sameValue(valueCount, 1, 'The value of `valueCount` is 1');
-  assert.sameValue(iterator.nextCalls, 1, 'The value of iterator.nextCalls is 1');
-  assert.sameValue(iterator.iterable.length, 4, 'The value of iterator.iterable.length is 4');
 })().then($DONE, $DONE);

test/built-ins/AsyncIterator/prototype/asIndexedPairs/abrupt-iterator-next.js

@@ -7,12 +7,8 @@ description: >
 info: |
   %AsyncIterator.prototype%.asIndexedPairs ( )
 
-  %AsyncIterator.prototype%.asIndexedPairs is a built-in async generator function which, when called, performs the following prelude steps:
-
-    Let iterated be ? GetIteratorDirect(this value).
-
-  The body of %AsyncIterator.prototype%.asIndexedPairs is composed of the following steps:
-
+  Let iterated be ? GetIteratorDirect(this value).
+  Let closure be a new Abstract Closure with no parameters that captures iterated and performs the following steps when called:
     Let index be 0.
     Let lastValue be undefined.
     Repeat,
@@ -23,29 +19,33 @@ includes: [iterators.js]
 features: [async-iteration, iterator-helpers]
 flags: [async]
 ---*/
+let nextCalls = 0;
+
 class Test262AsyncIteratorAbrupt extends Test262AsyncIterator {
   async next() {
+    nextCalls++;
     throw new Test262Error();
   }
 }
 
 (async () => {
-  let count = 0;
-  let iterator = new Test262AsyncIteratorAbrupt([0, 1, 2, 3]);
-  assert.sameValue(iterator.nextCalls, 0, 'The value of iterator.nextCalls is 0');
+  let tryCount = 0;
+  let catchCount = 0;
+  let iterator = new Test262AsyncIteratorAbrupt([1, 2]);
   let indexedPairs = iterator.asIndexedPairs();
 
   try {
-    count++;
+    tryCount++;
 
     for await (const [i, v] of indexedPairs) {
       $DONE('for await body must not be reachable');
     }
   } catch (e) {
-    count++;
+    catchCount++;
     assert.sameValue(e instanceof Test262Error, true, 'The result of evaluating `(e instanceof Test262Error)` is true');
   }
 
-  assert.sameValue(iterator.nextCalls, 0, 'The value of iterator.nextCalls is 0');
-  assert.sameValue(iterator.iterable.length, 4, 'The value of iterator.iterable.length is 4');
+  assert.sameValue(tryCount, 1, 'The value of `tryCount` is 1');
+  assert.sameValue(catchCount, 1, 'The value of `catchCount` is 1');
+  assert.sameValue(nextCalls, 1, 'The value of `nextCalls` is 1');
 })().then($DONE, $DONE);

test/built-ins/AsyncIterator/prototype/asIndexedPairs/abrupt-iterator-return.js

@@ -7,12 +7,8 @@ description: >
 info: |
   %AsyncIterator.prototype%.asIndexedPairs ( )
 
-  %AsyncIterator.prototype%.asIndexedPairs is a built-in async generator function which, when called, performs the following prelude steps:
-
-    Let iterated be ? GetIteratorDirect(this value).
-
-  The body of %AsyncIterator.prototype%.asIndexedPairs is composed of the following steps:
-
+  Let iterated be ? GetIteratorDirect(this value).
+  Let closure be a new Abstract Closure with no parameters that captures iterated and performs the following steps when called:
     Let index be 0.
     Let lastValue be undefined.
     Repeat,
@@ -23,17 +19,17 @@ info: |
       Set index to index + 1.
       Set lastValue to Yield(pair).
       IfAbruptCloseAsyncIterator(iterated, lastValue).
-
+  ...
 
 features: [async-iteration, iterator-helpers]
 flags: [async]
 ---*/
-let genCount = 0;
+let yieldCount = 0;
 
 async function* g() {
-  genCount++;
+  yieldCount++;
   yield 1;
-  genCount++;
+  yieldCount++;
   throw new Test262Error();
 }
 
@@ -54,7 +50,7 @@ async function* g() {
     assert.sameValue(e instanceof Test262Error, true, 'The result of evaluating `(e instanceof Test262Error)` is true');
   }
 
-  assert.sameValue(genCount, 2, 'The value of `genCount` is 2');
+  assert.sameValue(yieldCount, 2, 'The value of `yieldCount` is 2');
   assert.sameValue(tryCount, 1, 'The value of `tryCount` is 1');
   assert.sameValue(catchCount, 1, 'The value of `catchCount` is 1');
   assert.sameValue(forAwaitCount, 1, 'The value of `forAwaitCount` is 1');

test/built-ins/AsyncIterator/prototype/asIndexedPairs/callable.js

@@ -17,4 +17,4 @@ assert.throws(TypeError, () => {
 
 assert.throws(TypeError, () => {
   new (new (class S extends AsyncIterator {})()).asIndexedPairs();
-}, '`let instance = new (class S extends AsyncIterator {})(); new instance.asIndexedPairs()` throws a TypeError exception');
+}, '`new (new (class S extends AsyncIterator {})()).asIndexedPairs()` throws a TypeError exception');