Fix call to getCodeFixesAtPosition (microsoft#17063)

Andy · web-flow · commit dab682767cc6 · 2017-07-10T13:25:48.000-07:00
diff --git a/src/harness/fourslash.ts b/src/harness/fourslash.ts
@@ -2337,7 +2337,7 @@ namespace FourSlash {
                     continue;
                 }
 
-                const newActions = this.languageService.getCodeFixesAtPosition(fileName, diagnostic.start, diagnostic.length, [diagnostic.code], this.formatCodeSettings);
+                const newActions = this.languageService.getCodeFixesAtPosition(fileName, diagnostic.start, diagnostic.start + diagnostic.length, [diagnostic.code], this.formatCodeSettings);
                 if (newActions && newActions.length) {
                     actions = actions ? actions.concat(newActions) : newActions;
                 }
diff --git a/src/services/services.ts b/src/services/services.ts
@@ -1768,7 +1768,7 @@ namespace ts {
         function getCodeFixesAtPosition(fileName: string, start: number, end: number, errorCodes: number[], formatOptions: FormatCodeSettings): CodeAction[] {
             synchronizeHostData();
             const sourceFile = getValidSourceFile(fileName);
-            const span = { start, length: end - start };
+            const span = createTextSpanFromBounds(start, end);
             const newLineCharacter = getNewLineOrDefaultFromHost(host);
 
             let allFixes: CodeAction[] = [];

Original file line number	Diff line number	Diff line change
`@@ -2337,7 +2337,7 @@ namespace FourSlash {`
`2337`	`2337`	`continue;`
`2338`	`2338`	`}`
`2339`	`2339`
`2340`		`- const newActions = this.languageService.getCodeFixesAtPosition(fileName, diagnostic.start, diagnostic.length, [diagnostic.code], this.formatCodeSettings);`
	`2340`	`+ const newActions = this.languageService.getCodeFixesAtPosition(fileName, diagnostic.start, diagnostic.start + diagnostic.length, [diagnostic.code], this.formatCodeSettings);`
`2341`	`2341`	`if (newActions && newActions.length) {`
`2342`	`2342`	`actions = actions ? actions.concat(newActions) : newActions;`
`2343`	`2343`	`}`