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

fix(cli/console): enclose symbol keys in brackets (#7642)

Browse source 
This encloses symbol keys when used in objects with brackets (e.g
[Symbol("Symbol.iterator")]).
This commit is contained in:
Casper Beyer 2020-09-23 19:58:28 +08:00 • committed by GitHub
parent 29dd62b08c
commit d68fb81342
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 16 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
cli/rt/02_console.js
View file

@ -747,7 +747,7 @@
}
for (const key of symbolKeys) {
entries.push(
`${maybeQuoteSymbol(key)}: ${
`[${maybeQuoteSymbol(key)}]: ${
inspectValueWithQuotes(
value[key],
ctx,

2
cli/tests/041_dyn_import_eval.out
View file

@ -1 +1 @@
Module { isMod4: true, Symbol(Symbol.toStringTag): "Module" }
Module { isMod4: true, [Symbol(Symbol.toStringTag)]: "Module" }

2
cli/tests/042_dyn_import_evalcontext.ts.out
View file

@ -1 +1 @@
Module { isMod4: true, Symbol(Symbol.toStringTag): "Module" }
Module { isMod4: true, [Symbol(Symbol.toStringTag)]: "Module" }

2
cli/tests/fix_js_imports.ts.out
View file

@ -1 +1 @@
Module { Symbol(Symbol.toStringTag): "Module" }
Module { [Symbol(Symbol.toStringTag)]: "Module" }

21
cli/tests/unit/console_test.ts
View file

@ -121,11 +121,11 @@ unitTest(
},
),
`{
Symbol("foo\\b"): 'Symbol("foo\\n\")',
Symbol("bar\\n"): 'Symbol("bar\\n\")',
Symbol("bar\\r"): 'Symbol("bar\\r\")',
Symbol("baz\\t"): 'Symbol("baz\\t\")',
Symbol("qux\\x00"): 'Symbol(\"qux\\x00")'
[Symbol("foo\\b")]: 'Symbol("foo\\n\")',
[Symbol("bar\\n")]: 'Symbol("bar\\n\")',
[Symbol("bar\\r")]: 'Symbol("bar\\r\")',
[Symbol("baz\\t")]: 'Symbol("baz\\t\")',
[Symbol("qux\\x00")]: 'Symbol(\"qux\\x00")'
}`,
);
assertEquals(
@ -277,7 +277,10 @@ unitTest(function consoleTestStringifyCircular(): void {
"{ a: { b: { c: { d: [Set] } } } }",
);
assertEquals(stringify(nestedObj), nestedObjExpected);
assertEquals(stringify(JSON), 'JSON { Symbol(Symbol.toStringTag): "JSON" }');
assertEquals(
stringify(JSON),
'JSON { [Symbol(Symbol.toStringTag)]: "JSON" }',
);
assertEquals(
stringify(console),
`{
@ -301,12 +304,12 @@ unitTest(function consoleTestStringifyCircular(): void {
clear: [Function: clear],
trace: [Function: trace],
indentLevel: 0,
Symbol(isConsoleInstance): true
[Symbol(isConsoleInstance)]: true
}`,
);
assertEquals(
stringify({ str: 1, [Symbol.for("sym")]: 2, [Symbol.toStringTag]: "TAG" }),
'TAG { str: 1, Symbol(sym): 2, Symbol(Symbol.toStringTag): "TAG" }',
'TAG { str: 1, [Symbol(sym)]: 2, [Symbol(Symbol.toStringTag)]: "TAG" }',
);
// test inspect is working the same
assertEquals(stripColor(Deno.inspect(nestedObj)), nestedObjExpected);
@ -798,7 +801,7 @@ unitTest(function consoleTestWithCustomInspectorError(): void {
assertEquals(stringify(new B({ a: "a" })), "a");
assertEquals(
stringify(B.prototype),
"{ Symbol(Deno.customInspect): [Function: [Deno.customInspect]] }",
"{ [Symbol(Deno.customInspect)]: [Function: [Deno.customInspect]] }",
);
});