getEditsForFileRename: Don't update unrelated import

Author: Andy Hanson

Diff for: src/services/getEditsForFileRename.ts

@@ -104,6 +104,7 @@ namespace ts {
     ): void {
         for (const sourceFile of program.getSourceFiles()) {
             const newImportFromPath = oldToNew(sourceFile.fileName) || sourceFile.fileName;
+            const importingSourceFileMoved = newImportFromPath !== sourceFile.fileName;
             const newImportFromDirectory = getDirectoryPath(newImportFromPath);
 
             const oldFromNew: string | undefined = newToOld(sourceFile.fileName);
@@ -123,7 +124,10 @@ namespace ts {
                         // TODO:GH#18217
                         ? getSourceFileToImportFromResolved(resolveModuleName(importLiteral.text, oldImportFromPath, program.getCompilerOptions(), host as ModuleResolutionHost), oldToNew, program)
                         : getSourceFileToImport(importLiteral, sourceFile, program, host, oldToNew);
-                    return toImport === undefined ? undefined : moduleSpecifiers.getModuleSpecifier(program, sourceFile, newImportFromPath, toImport, host, preferences);
+                    // If neither the importing source file nor the imported file moved, do nothing.
+                    return toImport === undefined || !toImport.updated && !importingSourceFileMoved
+                        ? undefined
+                        : moduleSpecifiers.getModuleSpecifier(program, sourceFile, newImportFromPath, toImport.newFileName, host, preferences);
                 });
         }
     }
@@ -135,12 +139,18 @@ namespace ts {
         return ensurePathIsNonModuleName(combineNormal(pathA, pathB));
     }
 
-    function getSourceFileToImport(importLiteral: StringLiteralLike, importingSourceFile: SourceFile, program: Program, host: LanguageServiceHost, oldToNew: PathUpdater): string | undefined {
+    interface ToImport {
+        readonly newFileName: string;
+        /** True if the imported file was renamed. */
+        readonly updated: boolean;
+    }
+    function getSourceFileToImport(importLiteral: StringLiteralLike, importingSourceFile: SourceFile, program: Program, host: LanguageServiceHost, oldToNew: PathUpdater): ToImport | undefined {
         const symbol = program.getTypeChecker().getSymbolAtLocation(importLiteral);
         if (symbol) {
             if (symbol.declarations.some(d => isAmbientModule(d))) return undefined; // No need to update if it's an ambient module
             const oldFileName = find(symbol.declarations, isSourceFile)!.fileName;
-            return oldToNew(oldFileName) || oldFileName;
+            const newFileName = oldToNew(oldFileName);
+            return newFileName === undefined ? { newFileName: oldFileName, updated: false } : { newFileName, updated: true };
         }
         else {
             const resolved = host.resolveModuleNames
@@ -150,14 +160,15 @@ namespace ts {
         }
     }
 
-    function getSourceFileToImportFromResolved(resolved: ResolvedModuleWithFailedLookupLocations | undefined, oldToNew: PathUpdater, program: Program): string | undefined {
+    function getSourceFileToImportFromResolved(resolved: ResolvedModuleWithFailedLookupLocations | undefined, oldToNew: PathUpdater, program: Program): ToImport | undefined {
         return resolved && (
             (resolved.resolvedModule && getIfInProgram(resolved.resolvedModule.resolvedFileName)) || firstDefined(resolved.failedLookupLocations, getIfInProgram));
 
-        function getIfInProgram(oldLocation: string): string | undefined {
+        function getIfInProgram(oldLocation: string): ToImport | undefined {
             const newLocation = oldToNew(oldLocation);
+
             return program.getSourceFile(oldLocation) || newLocation !== undefined && program.getSourceFile(newLocation)
-                ? newLocation || oldLocation
+                ? newLocation !== undefined ? { newFileName: newLocation, updated: true } : { newFileName: oldLocation, updated: false }
                 : undefined;
         }
     }

Diff for: tests/cases/fourslash/getEditsForFileRename.ts

@@ -14,6 +14,10 @@
 /////// <reference path="../old.ts" />
 ////import old from "../old";
 
+// @Filename: /unrelated.ts
+// Don't update an unrelated import
+////import { x } from "././src/./foo/./a";
+
 // @Filename: /src/new.ts
 ////