diff options
author | Steven Guerrero <stephenguerrero43@gmail.com> | 2021-01-14 14:18:51 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-01-14 14:18:51 -0500 |
commit | 2d1208556ad09844407c484b1e23887e3ade97c7 (patch) | |
tree | e688ac2af56aa0dd12044c51cc9312c35de741e2 /cli/tests | |
parent | 979d71c883ee76c5e1246c5ad02f2791b745bef6 (diff) |
fix: don't swallow customInspect exceptions (#9095)
Diffstat (limited to 'cli/tests')
-rw-r--r-- | cli/tests/unit/console_test.ts | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/cli/tests/unit/console_test.ts b/cli/tests/unit/console_test.ts index 6f37c8050..64345be84 100644 --- a/cli/tests/unit/console_test.ts +++ b/cli/tests/unit/console_test.ts @@ -12,6 +12,7 @@ import { assert, assertEquals, assertStringIncludes, + assertThrows, unitTest, } from "./test_util.ts"; import { stripColor } from "../../../std/fmt/colors.ts"; @@ -834,19 +835,11 @@ unitTest(function consoleTestWithCustomInspectorError(): void { } } - assertEquals(stringify(new A()), "A {}"); - - class B { - constructor(public field: { a: string }) {} - [customInspect](): string { - return this.field.a; - } - } - - assertEquals(stringify(new B({ a: "a" })), "a"); - assertEquals( - stringify(B.prototype), - "B { [Symbol(Deno.customInspect)]: [Function: [Deno.customInspect]] }", + assertThrows( + () => stringify(new A()), + Error, + "BOOM", + "Custom inspect won't attempt to parse if user defined function throws", ); }); |