beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 23:50:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibJS: Change error message for non-objects in GetIteratorFlattenable

Browse source 
Matches other usages in e.g. Temporal now.
This commit is contained in:
Timothy Flynn 2023-07-12 17:52:36 -04:00 committed by Linus Groh
parent 92fa1efc76
commit 705e96568c
Notes: sideshowbarker 2024-07-17 16:23:55 +09:00 
Author: https://github.com/trflynn89
Commit: https://github.com/SerenityOS/serenity/commit/705e96568c
Pull-request: https://github.com/SerenityOS/serenity/pull/19983
Reviewed-by: https://github.com/linusg ✅
3 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibJS/Runtime/Iterator.cpp
View file

@ -46,7 +46,7 @@ ThrowCompletionOr<IteratorRecord> get_iterator_flattenable(VM& vm, Value object,
if (!object.is_object()) {
// a. If stringHandling is reject-strings or obj is not a String, throw a TypeError exception.
if (string_handling == StringHandling::RejectStrings || !object.is_string())
return vm.throw_completion<TypeError>(ErrorType::NotAnObject, "obj"sv);
return vm.throw_completion<TypeError>(ErrorType::NotAnObject, TRY_OR_THROW_OOM(vm, object.to_string_without_side_effects()));
}
// 2. Let method be ? GetMethod(obj, @@iterator).
@ -67,7 +67,7 @@ ThrowCompletionOr<IteratorRecord> get_iterator_flattenable(VM& vm, Value object,
// 5. If iterator is not an Object, throw a TypeError exception.
if (!iterator.is_object())
return vm.throw_completion<TypeError>(ErrorType::NotAnObject, "iterator"sv);
return vm.throw_completion<TypeError>(ErrorType::NotAnObject, TRY_OR_THROW_OOM(vm, iterator.to_string_without_side_effects()));
// 6. Return ? GetIteratorDirect(iterator).
return TRY(get_iterator_direct(vm, iterator.as_object()));

4
Userland/Libraries/LibJS/Tests/builtins/Iterator/Iterator.from.js
View file

@ -2,7 +2,7 @@ describe("errors", () => {
test("called with non-Object", () => {
expect(() => {
Iterator.from(Symbol.hasInstance);
}).toThrowWithMessage(TypeError, "obj is not an object");
}).toThrowWithMessage(TypeError, "Symbol(Symbol.hasInstance) is not an object");
});
test("@@iterator is not callable", () => {
@ -35,7 +35,7 @@ describe("errors", () => {
expect(() => {
Iterator.from(iterable);
}).toThrowWithMessage(TypeError, "iterator is not an object");
}).toThrowWithMessage(TypeError, "Symbol(Symbol.hasInstance) is not an object");
});
});

2
Userland/Libraries/LibJS/Tests/builtins/Iterator/Iterator.prototype.flatMap.js
View file

@ -69,7 +69,7 @@ describe("errors", () => {
expect(() => {
const iterator = generator().flatMap(() => Symbol.hasInstance);
iterator.next();
}).toThrowWithMessage(TypeError, "obj is not an object");
}).toThrowWithMessage(TypeError, "Symbol(Symbol.hasInstance) is not an object");
});
});