Merge pull request microsoft#18077 from fwouts/regex-bug

DanielRosenwasser · web-flow · commit a60b1037f849 · 2017-08-28T14:10:29.000-07:00
Fix microsoft#18071: Printer failure when encountering RegularExpressionLiteral
diff --git a/src/compiler/utilities.ts b/src/compiler/utilities.ts
@@ -359,6 +359,7 @@ namespace ts {
             case SyntaxKind.TemplateTail:
                 return "}" + escapeText(node.text, CharacterCodes.backtick) + "`";
             case SyntaxKind.NumericLiteral:
+            case SyntaxKind.RegularExpressionLiteral:
                 return node.text;
         }
 
diff --git a/src/harness/unittests/printer.ts b/src/harness/unittests/printer.ts
@@ -56,6 +56,9 @@ namespace ts {
 
             // github #14948
             printsCorrectly("templateLiteral", {}, printer => printer.printFile(createSourceFile("source.ts", "let greeting = `Hi ${name}, how are you?`;", ScriptTarget.ES2017)));
+
+            // github #18071
+            printsCorrectly("regularExpressionLiteral", {}, printer => printer.printFile(createSourceFile("source.ts", "let regex = /abc/;", ScriptTarget.ES2017)));
         });
 
         describe("printBundle", () => {
diff --git a/tests/baselines/reference/printerApi/printsFileCorrectly.regularExpressionLiteral.js b/tests/baselines/reference/printerApi/printsFileCorrectly.regularExpressionLiteral.js
@@ -0,0 +1 @@
+let regex = /abc/;

Original file line number	Diff line number	Diff line change
`@@ -359,6 +359,7 @@ namespace ts {`
`359`	`359`	`case SyntaxKind.TemplateTail:`
`360`	`360`	return "}" + escapeText(node.text, CharacterCodes.backtick) + "`";
`361`	`361`	`case SyntaxKind.NumericLiteral:`
	`362`	`+ case SyntaxKind.RegularExpressionLiteral:`
`362`	`363`	`return node.text;`
`363`	`364`	`}`
`364`	`365`