mirrors/bun.sh
1
0
Fork 0
mirror of https://github.com/oven-sh/bun synced 2026-02-02 15:08:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

bindings: fix createTypeError and createRangeError inheritance (#11341)

Browse Source
This commit is contained in:
Meghan Denny
2024-05-28 16:54:08 -07:00
committed by GitHub
parent 96f29e8555
commit 367b69dff5
3 changed files with 8 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
.gitignore vendored
View File

@@ -54,6 +54,7 @@
/test.js
/test.ts
/testdir
/test.zig
build
build.ninja
bun-binary

18
src/bun.js/bindings/bindings.cpp
View File

@@ -2465,14 +2465,7 @@ JSC__JSValue JSC__JSValue__createRangeError(const ZigString* message, const ZigS
{
JSC::VM& vm = globalObject->vm();
ZigString code = *arg1;
JSC::JSObject* rangeError = Zig::getErrorInstance(message, globalObject).asCell()->getObject();
static const char* range_error_name = "RangeError";
rangeError->putDirect(
vm, vm.propertyNames->name,
JSC::JSValue(JSC::jsOwnedString(
vm, WTF::String(WTF::StringImpl::createWithoutCopying({ range_error_name, 10 })))),
0);
JSC::JSObject* rangeError = Zig::getRangeErrorInstance(message, globalObject).asCell()->getObject();
if (code.len > 0) {
auto clientData = WebCore::clientData(vm);
@@ -2488,14 +2481,7 @@ JSC__JSValue JSC__JSValue__createTypeError(const ZigString* message, const ZigSt
{
JSC::VM& vm = globalObject->vm();
ZigString code = *arg1;
JSC::JSObject* typeError = Zig::getErrorInstance(message, globalObject).asCell()->getObject();
static const char* range_error_name = "TypeError";
typeError->putDirect(
vm, vm.propertyNames->name,
JSC::JSValue(JSC::jsOwnedString(
vm, WTF::String(WTF::StringImpl::createWithoutCopying({ range_error_name, 9 })))),
0);
JSC::JSObject* typeError = Zig::getTypeErrorInstance(message, globalObject).asCell()->getObject();
if (code.len > 0) {
auto clientData = WebCore::clientData(vm);

5
test/js/bun/globals.test.js
View File

@@ -153,3 +153,8 @@ it("self is a getter", () => {
expect(descriptor.configurable).toBe(true);
expect(globalThis.self).toBe(globalThis);
});
it("errors thrown by native code should be TypeError", async () => {
expect(() => Bun.dns.prefetch()).toThrowError(TypeError);
expect(async () => await fetch("http://localhost", { body: "123" })).toThrowError(TypeError);
});