Merge pull request microsoft#18927 from amcasey/ExtractBaselineShowRange

Include range markers in Extract Constant/Function baselines

Author: amcasey

src/harness/unittests/extractTestHelpers.ts

@@ -135,7 +135,7 @@ namespace ts {
             Harness.Baseline.runBaseline(`${baselineFolder}/${caption}${extension}`, () => {
                 const data: string[] = [];
                 data.push(`// ==ORIGINAL==`);
-                data.push(sourceFile.text);
+                data.push(text.replace("[#|", "/*[#|*/").replace("|]", "/*|]*/"));
                 for (const action of actions) {
                     const { renameLocation, edits } = refactor.extractSymbol.getEditsForAction(context, action.name);
                     assert.lengthOf(edits, 1);

tests/baselines/reference/extractConstant/extractConstant_ArrowFunction_Block.js

@@ -1,7 +1,7 @@
 // ==ORIGINAL==
 
 const f = () => {
-    return 2 + 1;
+    return /*[#|*/2 + 1/*|]*/;
 };
 // ==SCOPE::Extract to constant in enclosing scope==
 

tests/baselines/reference/extractConstant/extractConstant_ArrowFunction_Block.ts

@@ -1,7 +1,7 @@
 // ==ORIGINAL==
 
 const f = () => {
-    return 2 + 1;
+    return /*[#|*/2 + 1/*|]*/;
 };
 // ==SCOPE::Extract to constant in enclosing scope==
 

tests/baselines/reference/extractConstant/extractConstant_ArrowFunction_Expression.js

@@ -1,5 +1,5 @@
 // ==ORIGINAL==
-const f = () => 2 + 1;
+const f = () => /*[#|*/2 + 1/*|]*/;
 // ==SCOPE::Extract to constant in global scope==
 const newLocal = 2 + 1;
 

tests/baselines/reference/extractConstant/extractConstant_ArrowFunction_Expression.ts

@@ -1,5 +1,5 @@
 // ==ORIGINAL==
-const f = () => 2 + 1;
+const f = () => /*[#|*/2 + 1/*|]*/;
 // ==SCOPE::Extract to constant in global scope==
 const newLocal = 2 + 1;
 

tests/baselines/reference/extractConstant/extractConstant_BlockScopeMismatch.js

@@ -2,7 +2,7 @@
 
 for (let i = 0; i < 10; i++) {
     for (let j = 0; j < 10; j++) {
-        const x = i + 1;
+        const x = /*[#|*/i + 1/*|]*/;
     }
 }
 

tests/baselines/reference/extractConstant/extractConstant_BlockScopeMismatch.ts

@@ -2,7 +2,7 @@
 
 for (let i = 0; i < 10; i++) {
     for (let j = 0; j < 10; j++) {
-        const x = i + 1;
+        const x = /*[#|*/i + 1/*|]*/;
     }
 }
 

tests/baselines/reference/extractConstant/extractConstant_BlockScopes_NoDependencies.js

@@ -1,7 +1,7 @@
 // ==ORIGINAL==
 for (let i = 0; i < 10; i++) {
     for (let j = 0; j < 10; j++) {
-        let x = 1;
+        let x = /*[#|*/1/*|]*/;
     }
 }
 // ==SCOPE::Extract to constant in enclosing scope==

tests/baselines/reference/extractConstant/extractConstant_BlockScopes_NoDependencies.ts

@@ -1,7 +1,7 @@
 // ==ORIGINAL==
 for (let i = 0; i < 10; i++) {
     for (let j = 0; j < 10; j++) {
-        let x = 1;
+        let x = /*[#|*/1/*|]*/;
     }
 }
 // ==SCOPE::Extract to constant in enclosing scope==

tests/baselines/reference/extractConstant/extractConstant_Class.js

@@ -1,6 +1,6 @@
 // ==ORIGINAL==
 class C {
-    x = 1;
+    x = /*[#|*/1/*|]*/;
 }
 // ==SCOPE::Extract to constant in global scope==
 const newLocal = 1;