Fix bug in reduceEachChild (fixes microsoft#27854)

Igorbek · Igorbek · commit b2cbbbd1d37c · 2018-10-18T01:34:04.000-07:00
- add reducing of type arguments in tagged template expression
diff --git a/src/compiler/visitor.ts b/src/compiler/visitor.ts
@@ -1111,6 +1111,7 @@ namespace ts {
 
             case SyntaxKind.TaggedTemplateExpression:
                 result = reduceNode((<TaggedTemplateExpression>node).tag, cbNode, result);
+                result = reduceNodes((<NewExpression>node).typeArguments, cbNodes, result);
                 result = reduceNode((<TaggedTemplateExpression>node).template, cbNode, result);
                 break;
 
diff --git a/src/testRunner/unittests/transform.ts b/src/testRunner/unittests/transform.ts
@@ -45,9 +45,8 @@ namespace ts {
                 if (isIdentifier(node) && node.text === "oldName") {
                     return createIdentifier("newName");
                 }
-        
                 return visitEachChild(node, visitor, context);
-            }
+            };
             return (node: SourceFile) => visitNode(node, visitor);
         }
 
@@ -99,7 +98,7 @@ namespace ts {
                 },
                 compilerOptions: {
                     newLine: NewLineKind.CarriageReturnLineFeed,
-                    target: ts.ScriptTarget.Latest
+                    target: ScriptTarget.Latest
                 }
             }).outputText;
         });
diff --git a/tests/baselines/reference/transformApi/transformsCorrectly.issue27854.js b/tests/baselines/reference/transformApi/transformsCorrectly.issue27854.js
@@ -1,3 +1 @@
-newName<{
-    a: string;
-}> ` ... `;
+newName ` ... `;

Original file line number	Diff line number	Diff line change
`@@ -45,9 +45,8 @@ namespace ts {`
`45`	`45`	`if (isIdentifier(node) && node.text === "oldName") {`
`46`	`46`	`return createIdentifier("newName");`
`47`	`47`	`}`
`48`		`-`
`49`	`48`	`return visitEachChild(node, visitor, context);`
`50`		`- }`
	`49`	`+ };`
`51`	`50`	`return (node: SourceFile) => visitNode(node, visitor);`
`52`	`51`	`}`
`53`	`52`
`@@ -99,7 +98,7 @@ namespace ts {`
`99`	`98`	`},`
`100`	`99`	`compilerOptions: {`
`101`	`100`	`newLine: NewLineKind.CarriageReturnLineFeed,`
`102`		`- target: ts.ScriptTarget.Latest`
	`101`	`+ target: ScriptTarget.Latest`
`103`	`102`	`}`
`104`	`103`	`}).outputText;`
`105`	`104`	`});`