Skip to content

Commit

Permalink
Bug 1794566 [wpt PR 36124] - Expand iterable<> and async iterable<> c…
Browse files Browse the repository at this point in the history 
…hecks, a=testonly

Automatic update from web-platform-tests
Expand iterable<> and async iterable<> checks (#36124)

In particular, check the enumerable/configurable/writable-ness of all the methods.

Related to whatwg/webidl#738.
--

wpt-commits: 4e5209fdff87f351bfcbfe765ed669a6e9eeb996
wpt-pr: 36124
  • Loading branch information
@domenic @moz-wptsync-bot
domenic authored and moz-wptsync-bot committed Oct 21, 2022
1 parent 529f806 commit 6cdb086
Showing 1 changed file with 74 additions and 62 deletions.
136 changes: 74 additions & 62 deletions testing/web-platform/tests/resources/idlharness.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2433,44 +2433,6 @@ IdlInterface.prototype.test_to_json_operation = function(desc, memberHolderObjec
}
};

IdlInterface.prototype.test_member_iterable = function(member)
{
subsetTestByKey(this.name, test, function()
{
var isPairIterator = member.idlType.length === 2;
var proto = this.get_interface_object().prototype;
var iteratorDesc = Object.getOwnPropertyDescriptor(proto, Symbol.iterator);

assert_true(iteratorDesc.writable, "@@iterator property should be writable");
assert_true(iteratorDesc.configurable, "@@iterator property should be configurable");
assert_false(iteratorDesc.enumerable, "@@iterator property should not be enumerable");
assert_equals(typeof iteratorDesc.value, "function", "@@iterator property should be a function");
assert_equals(iteratorDesc.value.length, 0, "@@iterator function object length should be 0");
assert_equals(iteratorDesc.value.name, isPairIterator ? "entries" : "values", "@@iterator function object should have the right name");

if (isPairIterator) {
assert_equals(proto["entries"], proto[Symbol.iterator], "entries method should be the same as @@iterator method");
[
["entries", 0],
["keys", 0],
["values", 0],
["forEach", 1]
].forEach(([property, length]) => {
var desc = Object.getOwnPropertyDescriptor(proto, property);
assert_equals(typeof desc.value, "function", property + " property should be a function");
assert_equals(desc.value.length, length, property + " function object should have the right length");
assert_equals(desc.value.name, property, property + " function object should have the right name");
});
} else {
assert_equals(proto[Symbol.iterator], Array.prototype[Symbol.iterator], "@@iterator method should be the same as Array prototype's");
["entries", "keys", "values", "forEach", Symbol.iterator].forEach(property => {
var propertyName = property === Symbol.iterator ? "@@iterator" : property;
assert_equals(proto[property], Array.prototype[property], propertyName + " method should be the same as Array prototype's");
});
}
}.bind(this), this.name + " interface: iterable<" + member.idlType.map(function(t) { return t.idlType; }).join(", ") + ">");
};

IdlInterface.prototype.test_member_maplike = function(member) {
subsetTestByKey(this.name, test, () => {
const proto = this.get_interface_object().prototype;
Expand Down Expand Up @@ -2562,35 +2524,85 @@ IdlInterface.prototype.test_member_setlike = function(member) {
}, `${this.name} interface: setlike<${member.idlType.map(t => t.idlType).join(", ")}>`);
};

IdlInterface.prototype.test_member_async_iterable = function(member)
{
subsetTestByKey(this.name, test, function()
{
var isPairIterator = member.idlType.length === 2;
var proto = this.get_interface_object().prototype;
var iteratorDesc = Object.getOwnPropertyDescriptor(proto, Symbol.asyncIterator);
IdlInterface.prototype.test_member_iterable = function(member) {
subsetTestByKey(this.name, test, () => {
const isPairIterator = member.idlType.length === 2;
const proto = this.get_interface_object().prototype;

assert_true(iteratorDesc.writable, "@@asyncIterator property should be writable");
assert_true(iteratorDesc.configurable, "@@asyncIterator property should be configurable");
assert_false(iteratorDesc.enumerable, "@@asyncIterator property should not be enumerable");
assert_equals(typeof iteratorDesc.value, "function", "@@asyncIterator property should be a function");
assert_equals(iteratorDesc.value.length, 0, "@@asyncIterator function object length should be 0");
assert_equals(iteratorDesc.value.name, isPairIterator ? "entries" : "values", "@@asyncIterator function object should have the right name");
const methods = [
["entries", 0],
["keys", 0],
["values", 0],
["forEach", 1]
];

for (const [name, length] of methods) {
const desc = Object.getOwnPropertyDescriptor(proto, name);
assert_equals(typeof desc.value, "function", `${name} should be a function`);
assert_equals(desc.enumerable, true, `${name} enumerable`);
assert_equals(desc.configurable, true, `${name} configurable`);
assert_equals(desc.writable, true, `${name} writable`);
assert_equals(desc.value.length, length, `${name} function object length should be ${length}`);
assert_equals(desc.value.name, name, `${name} function object should have the right name`);

if (!isPairIterator) {
assert_equals(desc.value, Array.prototype[name], `${name} equality with Array.prototype version`);
}
}

const iteratorDesc = Object.getOwnPropertyDescriptor(proto, Symbol.iterator);
assert_equals(iteratorDesc.enumerable, false, `@@iterator enumerable`);
assert_equals(iteratorDesc.configurable, true, `@@iterator configurable`);
assert_equals(iteratorDesc.writable, true, `@@iterator writable`);

if (isPairIterator) {
assert_equals(proto["entries"], proto[Symbol.asyncIterator], "entries method should be the same as @@asyncIterator method");
["entries", "keys", "values"].forEach(property => {
var desc = Object.getOwnPropertyDescriptor(proto, property);
assert_equals(typeof desc.value, "function", property + " property should be a function");
assert_equals(desc.value.length, 0, property + " function object length should be 0");
assert_equals(desc.value.name, property, property + " function object should have the right name");
});
assert_equals(iteratorDesc.value, proto.entries, `@@iterator equality with entries`);
} else {
assert_equals(iteratorDesc.value, Array.prototype[Symbol.iterator], `@@iterator equality with Array.prototype version`);
}
}, `${this.name} interface: iterable<${member.idlType.map(t => t.idlType).join(", ")}>`);
};

IdlInterface.prototype.test_member_async_iterable = function(member) {
subsetTestByKey(this.name, test, () => {
const isPairIterator = member.idlType.length === 2;
const proto = this.get_interface_object().prototype;

// Note that although the spec allows arguments, which will be passed to the @@asyncIterator
// method (which is either values or entries), those arguments must always be optional. So
// length of 0 is still correct for values and entries.
const methods = [
["values", 0],
];

if (isPairIterator) {
methods.push(
["entries", 0],
["keys", 0]
);
}

for (const [name, length] of methods) {
const desc = Object.getOwnPropertyDescriptor(proto, name);
assert_equals(typeof desc.value, "function", `${name} should be a function`);
assert_equals(desc.enumerable, true, `${name} enumerable`);
assert_equals(desc.configurable, true, `${name} configurable`);
assert_equals(desc.writable, true, `${name} writable`);
assert_equals(desc.value.length, length, `${name} function object length should be ${length}`);
assert_equals(desc.value.name, name, `${name} function object should have the right name`);
}

const iteratorDesc = Object.getOwnPropertyDescriptor(proto, Symbol.asyncIterator);
assert_equals(iteratorDesc.enumerable, false, `@@iterator enumerable`);
assert_equals(iteratorDesc.configurable, true, `@@iterator configurable`);
assert_equals(iteratorDesc.writable, true, `@@iterator writable`);

if (isPairIterator) {
assert_equals(iteratorDesc.value, proto.entries, `@@iterator equality with entries`);
} else {
assert_equals(proto["values"], proto[Symbol.asyncIterator], "values method should be the same as @@asyncIterator method");
assert_false("entries" in proto, "should not have an entries method");
assert_false("keys" in proto, "should not have a keys method");
assert_equals(iteratorDesc.value, proto.values, `@@iterator equality with values`);
}
}.bind(this), this.name + " interface: async iterable<" + member.idlType.map(function(t) { return t.idlType; }).join(", ") + ">");
}, `${this.name} interface: async iterable<${member.idlType.map(t => t.idlType).join(", ")}>`);
};

IdlInterface.prototype.test_member_stringifier = function(member)
Expand Down

0 comments on commit 6cdb086

Please sign in to comment.