Skip to content

Commit

Permalink
Update iteration tests with respect to spec changes
Browse files Browse the repository at this point in the history
tc39/ecma262#988 changes the iteration protocol
such that the "next" method is only loaded from the iterator object once
during the prologue of iteration, rather than during each step.
  • Loading branch information
caitp committed Sep 27, 2017
1 parent d41eb5d commit 30358ea
Showing 1 changed file with 13 additions and 19 deletions.
32 changes: 13 additions & 19 deletions test/language/statements/for-of/iterator-next-reference.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,39 +3,33 @@
/*---
es6id: 13.6.4.13 S5.c
description: >
The iterator's `next` method should be accessed with each iteration as per
the `IteratorStep` abstract operation (7.4.5).
The iterator's `next` method should be accessed only once with each
iteration as per the `GetIterator` abstract operation (7.4.1).
features: [Symbol.iterator]
---*/

var iterable = {};
var iterator = {};
var firstIterResult = { done: false };
var iterationCount, invocationCount;
var iterationCount = 0;

iterable[Symbol.iterator] = function() {
return iterator;
};

iterator.next = function() { return { value: 45, done: false }; };
iterationCount = 0;
invocationCount = 0;
iterator.next = function() {
if (iterationCount) return { done: true };
return { value: 45, done: false };
};
for (var x of iterable) {
assert.sameValue(x, 45);

iterator.next = function() {
invocationCount++;

Object.defineProperty(iterator, 'next', {
get: function() {
$ERROR('Should not access the `next` method after iteration ' +
'is complete.');
}
});

return { value: null, done: true };
};
Object.defineProperty(iterator, 'next', {
get: function() {
$ERROR('Should not access the `next` method after the iteration ' +
'prologue.');
}
});
iterationCount++;
}
assert.sameValue(iterationCount, 1);
assert.sameValue(invocationCount, 1);

0 comments on commit 30358ea

Please sign in to comment.