From 4cf289e1a57b7ff32d73efc7d9b36b932df926ec Mon Sep 17 00:00:00 2001
From: Andrew Casey <andrew.casey@microsoft.com>
Date: Wed, 4 Oct 2017 11:26:41 -0700
Subject: [PATCH] Fix whitespace around inserted property initializer

Fixes #18741
---
 src/services/codefixes/fixAddMissingMember.ts     | 6 +++---
 tests/cases/fourslash/codeFixAddMissingMember4.ts | 7 +++----
 tests/cases/fourslash/codeFixAddMissingMember6.ts | 7 +++----
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/src/services/codefixes/fixAddMissingMember.ts b/src/services/codefixes/fixAddMissingMember.ts
index 9aba4c37f68..19bd592b7a7 100644
--- a/src/services/codefixes/fixAddMissingMember.ts
+++ b/src/services/codefixes/fixAddMissingMember.ts
@@ -112,11 +112,11 @@ namespace ts.codefix {
                     createIdentifier("undefined")));
 
                 const propertyInitializationChangeTracker = textChanges.ChangeTracker.fromContext(context);
-                propertyInitializationChangeTracker.insertNodeAt(
+                propertyInitializationChangeTracker.insertNodeBefore(
                     classDeclarationSourceFile,
-                    classConstructor.body.getEnd() - 1,
+                    classConstructor.body.getLastToken(),
                     propertyInitialization,
-                    { prefix: context.newLineCharacter, suffix: context.newLineCharacter });
+                    { suffix: context.newLineCharacter });
 
                 const initializeAction = {
                     description: formatStringFromArgs(getLocaleSpecificMessage(Diagnostics.Initialize_property_0_in_the_constructor), [tokenName]),
diff --git a/tests/cases/fourslash/codeFixAddMissingMember4.ts b/tests/cases/fourslash/codeFixAddMissingMember4.ts
index cfbec8977f6..a17b777ae5f 100644
--- a/tests/cases/fourslash/codeFixAddMissingMember4.ts
+++ b/tests/cases/fourslash/codeFixAddMissingMember4.ts
@@ -15,12 +15,11 @@
 verify.codeFix({
     description: "Initialize property 'foo' in the constructor.",
     index: 0,
-    // TODO: GH#18741 and GH#18445
+    // TODO: GH#18445
     newFileContent: `class C {
     constructor() {
-    \r
-this.foo = undefined;\r
-}
+        this.foo = undefined;\r
+    }
     method() {
         this.foo === 10;
     }
diff --git a/tests/cases/fourslash/codeFixAddMissingMember6.ts b/tests/cases/fourslash/codeFixAddMissingMember6.ts
index 2598014dde5..32525066657 100644
--- a/tests/cases/fourslash/codeFixAddMissingMember6.ts
+++ b/tests/cases/fourslash/codeFixAddMissingMember6.ts
@@ -13,12 +13,11 @@
 verify.codeFix({
     description: "Initialize property 'foo' in the constructor.",
     index: 0,
-    // TODO: GH#18741 and GH#18445
+    // TODO: GH#18445
     newFileContent: `class C {
     constructor() {
-    \r
-this.foo = undefined;\r
-}
+        this.foo = undefined;\r
+    }
     prop = ()=>{ this.foo === 10 };
 }`
 });