Add more error messages

whatwg · Apr 15, 2021 · ca24228 · ca24228
1 parent fa13e7f
commit ca24228
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/reference-implementation/lib/abstract-ops/ecmascript.js b/reference-implementation/lib/abstract-ops/ecmascript.js
@@ -51,7 +51,7 @@ exports.GetMethod = (V, P) => {
  return undefined;
  }
  if (typeof func !== 'function') {
- throw new TypeError();
+ throw new TypeError(`${P} is not a function`);
  }
  return func;
 };
@@ -89,7 +89,7 @@ exports.GetIterator = (obj, hint = 'sync', method) => {
  }
  const iterator = exports.Call(method, obj);
  if (!exports.typeIsObject(iterator)) {
- throw new TypeError();
+ throw new TypeError(`The iterator method must return an object`);
  }
  const nextMethod = iterator.next;
  return { iterator, nextMethod, done: false };
@@ -103,7 +103,7 @@ exports.IteratorNext = (iteratorRecord, value) => {
  result = exports.Call(iteratorRecord.nextMethod, iteratorRecord.iterator, [value]);
  }
  if (!exports.typeIsObject(result)) {
- throw new TypeError();
+ throw new TypeError(`The iterator.next() method must return an object`);
  }
  return result;
 };

diff --git a/reference-implementation/lib/abstract-ops/readable-streams.js b/reference-implementation/lib/abstract-ops/readable-streams.js
@@ -1432,7 +1432,7 @@ function ReadableStreamFromIterable(asyncIterable) {
  const nextPromise = promiseResolvedWith(nextResult);
  return transformPromiseWith(nextPromise, iterResult => {
  if (!typeIsObject(iterResult)) {
- throw new TypeError('The iterator.next() method must return an object');
+ throw new TypeError('The promise returned by the iterator.next() method must fulfill with an object');
  }
  const done = IteratorComplete(iterResult);
  if (done === true) {