fix(server): send diagnostic range to the Angular language service when fixing code errors (#1747)

This will make the fix error code simple, the range the fix error code get is always
the start and end of the diagnostic. For example, `<span>{{title}}</span>`, the `title` is
the missing member of component. When the range the user selects is from the start of `<span>`
to the end of `</span>`, the fix error code needs to check the start of the diagnostic.

Author: ivanwonder

integration/lsp/ivy_spec.ts

@@ -700,6 +700,33 @@ describe('code fixes', () => {
     expect(codeActions).toContain(jasmine.objectContaining(expectedCodeActionInTemplate));
   });
 
+  it('should fix error when the range the user selects is larger than the diagnostic', async () => {
+    const template = `<span>{{titl}}</span>`;
+    openTextDocument(client, FOO_TEMPLATE, template);
+    const languageServiceEnabled = await waitForNgcc(client);
+    expect(languageServiceEnabled).toBeTrue();
+    const diags = await getDiagnosticsForFile(client, FOO_TEMPLATE);
+    const codeActions = await client.sendRequest(lsp.CodeActionRequest.type, {
+      textDocument: {
+        uri: FOO_TEMPLATE_URI,
+      },
+      range:
+          lsp.Range.create(lsp.Position.create(0, 0), lsp.Position.create(0, template.length - 1)),
+      context: lsp.CodeActionContext.create(diags),
+    }) as lsp.CodeAction[];
+    const expectedCodeActionInTemplate = {
+      'edit': {
+        'changes': {
+          [FOO_TEMPLATE_URI]: [{
+            'newText': 'title',
+            'range': {'start': {'line': 0, 'character': 8}, 'end': {'line': 0, 'character': 12}}
+          }]
+        }
+      }
+    };
+    expect(codeActions).toContain(jasmine.objectContaining(expectedCodeActionInTemplate));
+  });
+
   describe('should work', () => {
     beforeEach(async () => {
       openTextDocument(client, FOO_COMPONENT, `

server/src/session.ts

@@ -210,13 +210,20 @@ export class Session {
     if (!lsInfo) {
       return null;
     }
-    const start = lspPositionToTsPosition(lsInfo.scriptInfo, params.range.start);
-    const end = lspPositionToTsPosition(lsInfo.scriptInfo, params.range.end);
-    const errorCodes = params.context.diagnostics.map(diag => diag.code)
-                           .filter((code): code is number => typeof code === 'number');
 
-    const codeActions = lsInfo.languageService.getCodeFixesAtPosition(
-        filePath, start, end, errorCodes, defaultFormatOptions, defaultPreferences);
+    const codeActions: ts.CodeFixAction[] = [];
+    for (const diagnostic of params.context.diagnostics) {
+      const errorCode = diagnostic.code;
+      if (typeof errorCode !== 'number') {
+        continue;
+      }
+      const start = lspPositionToTsPosition(lsInfo.scriptInfo, diagnostic.range.start);
+      const end = lspPositionToTsPosition(lsInfo.scriptInfo, diagnostic.range.end);
+      const codeActionsForDiagnostic = lsInfo.languageService.getCodeFixesAtPosition(
+          filePath, start, end, [errorCode], defaultFormatOptions, defaultPreferences);
+      codeActions.push(...codeActionsForDiagnostic);
+    }
+
     const individualCodeFixes = codeActions.map<lsp.CodeAction>(codeAction => {
       return {
         title: codeAction.description,