Skip to content

Commit 14bb2be

Browse files
committed
test(codegen): add more tests for strings (#10552)
We lacked a bit of test coverage for printing strings. Add more tests.
1 parent b9d84b0 commit 14bb2be

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

crates/oxc_codegen/tests/integration/esbuild.rs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -339,9 +339,14 @@ fn test_string() {
339339
test("let x = '\t'", "let x = \"\t\";\n");
340340
test("let x = '\\v'", "let x = \"\\v\";\n");
341341
test("let x = '\\n'", "let x = \"\\n\";\n");
342+
test("let x = '\\r'", "let x = \"\\r\";\n");
343+
test("let x = '\\r\\n'", "let x = \"\\r\\n\";\n");
342344
test("let x = '\\''", "let x = \"'\";\n");
343-
test("let x = '\\\"'", "let x = \"\\\"\";\n");
345+
test("let x = '\"'", "let x = \"\\\"\";\n");
346+
test("let x = '`'", "let x = \"`\";\n");
344347
test("let x = '\\'\"'", "let x = \"'\\\"\";\n");
348+
test("let x = '\\'`'", "let x = \"'`\";\n");
349+
test("let x = '\"`'", "let x = \"\\\"`\";\n");
345350
test("let x = '\\\\'", "let x = \"\\\\\";\n");
346351
test("let x = '\x00'", "let x = \"\\0\";\n");
347352
test("let x = '\x00!'", "let x = \"\\0!\";\n");
@@ -358,10 +363,12 @@ fn test_string() {
358363
test("let x = '\\x10'", "let x = \"\x10\";\n");
359364
test("let x = '\x1B'", "let x = \"\\x1B\";\n");
360365
test("let x = '\\x1B'", "let x = \"\\x1B\";\n");
366+
test("let x = '\\x41'", "let x = \"A\";\n");
361367
test("let x = '\u{ABCD}'", "let x = \"\u{ABCD}\";\n");
362368
test("let x = '\\uABCD'", "let x = \"\u{ABCD}\";\n");
363369
test("let x = '\\U000123AB'", "let x = \"U000123AB\";\n");
364370
test("let x = '\\u{123AB}'", "let x = \"\u{123ab}\";\n");
371+
test("let x = '\\u{41}'", "let x = \"A\";\n");
365372
test("let x = '\\uD808\\uDFAB'", "let x = \"\u{123ab}\";\n");
366373
test("let x = '\\uD808'", "let x = \"\\ud808\";\n"); // lone surrogate
367374
test("let x = '\\uD808X'", "let x = \"\\ud808X\";\n");

0 commit comments

Comments
 (0)