Fix bug: In newFileChanges, setParentNodes in new source file (microsoft#24765)

Andy · web-flow · commit fde6f2240888 · 2018-06-07T14:28:06.000-07:00
diff --git a/src/services/textChanges.ts b/src/services/textChanges.ts
@@ -735,7 +735,7 @@ namespace ts.textChanges {
         export function newFileChanges(oldFile: SourceFile, fileName: string, statements: ReadonlyArray<Statement>, newLineCharacter: string, formatContext: formatting.FormatContext): FileTextChanges {
             // TODO: this emits the file, parses it back, then formats it that -- may be a less roundabout way to do this
             const nonFormattedText = statements.map(s => getNonformattedText(s, oldFile, newLineCharacter).text).join(newLineCharacter);
-            const sourceFile = createSourceFile(fileName, nonFormattedText, ScriptTarget.ESNext);
+            const sourceFile = createSourceFile(fileName, nonFormattedText, ScriptTarget.ESNext, /*setParentNodes*/ true);
             const changes = formatting.formatDocument(sourceFile, formatContext);
             const text = applyChanges(nonFormattedText, changes);
             return { fileName, textChanges: [createTextChange(createTextSpan(0, 0), text)], isNewFile: true };
diff --git a/tests/cases/fourslash/moveToNewFile_jsx.ts b/tests/cases/fourslash/moveToNewFile_jsx.ts
@@ -0,0 +1,14 @@
+/// <reference path='fourslash.ts' />
+
+// @Filename: /a.tsx
+////[|<div>a</div>;|]
+
+verify.moveToNewFile({
+    newFileContents: {
+        "/a.tsx":
+``,
+
+        "/newFile.tsx":
+`<div>a</div>;`,
+    }
+});
