foster/deno
0
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2025-03-03 17:34:47 -05:00
Code Issues Packages 1 Wiki Activity

feat(core): cleaner opcall stack traces (#12358)

Browse source
This commit is contained in:
Aaron O'Mullan 2021-10-07 18:39:27 +02:00 • committed by GitHub
parent ab2e0a465e
commit 370c27e09a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 17 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
cli/tests/unit/opcall_test.ts
View file

@ -1,4 +1,9 @@
import { assertStringIncludes, unitTest, unreachable } from "./test_util.ts";
import {
assert,
assertStringIncludes,
unitTest,
unreachable,
} from "./test_util.ts";
unitTest(async function sendAsyncStackTrace() {
const buf = new Uint8Array(10);
@ -11,6 +16,10 @@ unitTest(async function sendAsyncStackTrace() {
console.log(s);
assertStringIncludes(s, "opcall_test.ts");
assertStringIncludes(s, "read");
assert(
!s.includes("deno:core"),
"opcall stack traces should NOT include deno:core internals such as unwrapOpResult",
);
}
});

9
core/01_core.js
View file

@ -12,6 +12,7 @@
Map,
Array,
ArrayPrototypeFill,
ErrorCaptureStackTrace,
Promise,
ObjectFreeze,
ObjectFromEntries,
@ -113,12 +114,12 @@
if (res?.$err_class_name) {
const className = res.$err_class_name;
const errorBuilder = errorMap[className];
if (!errorBuilder) {
throw new Error(
const err = errorBuilder ? errorBuilder(res.message) : new Error(
`Unregistered error class: "${className}"\n ${res.message}\n Classes of errors returned from ops should be registered via Deno.core.registerErrorClass().`,
);
}
throw errorBuilder(res.message);
// Strip unwrapOpResult() and errorBuilder() calls from stack trace
ErrorCaptureStackTrace(err, unwrapOpResult);
throw err;
}
return res;
}