Ignore case when sorting imports

amcasey · amcasey · commit 543c5daf9c1e · 2018-05-10T18:10:05.000-07:00
diff --git a/src/harness/unittests/organizeImports.ts b/src/harness/unittests/organizeImports.ts
@@ -23,6 +23,15 @@ namespace ts {
                     `import x from "./lib";`);
             });
 
+            it("Sort - case-insensitive", () => {
+                assertSortsBefore(
+                    `import y from "a";`,
+                    `import x from "Z";`);
+                assertSortsBefore(
+                    `import y from "A";`,
+                    `import x from "z";`);
+            });
+
             function assertSortsBefore(importString1: string, importString2: string) {
                 const [{moduleSpecifier: moduleSpecifier1}, {moduleSpecifier: moduleSpecifier2}] = parseImports(importString1, importString2);
                 assert.equal(OrganizeImports.compareModuleSpecifiers(moduleSpecifier1, moduleSpecifier2), Comparison.LessThan);
@@ -42,6 +51,13 @@ namespace ts {
                 assertListEqual(actualCoalescedImports, expectedCoalescedImports);
             });
 
+            it("Sort specifiers - case-insensitive", () => {
+                const sortedImports = parseImports(`import { default as M, a as n, B, y, Z as O } from "lib";`);
+                const actualCoalescedImports = OrganizeImports.coalesceImports(sortedImports);
+                const expectedCoalescedImports = parseImports(`import { a as n, B, default as M, y, Z as O } from "lib";`);
+                assertListEqual(actualCoalescedImports, expectedCoalescedImports);
+            });
+
             it("Combine side-effect-only imports", () => {
                 const sortedImports = parseImports(
                     `import "lib";`,
diff --git a/src/services/organizeImports.ts b/src/services/organizeImports.ts
@@ -254,7 +254,7 @@ namespace ts.OrganizeImports {
         }
 
         function compareIdentifiers(s1: Identifier, s2: Identifier) {
-            return compareStringsCaseSensitive(s1.text, s2.text);
+            return compareStringsCaseInsensitive(s1.text, s2.text);
         }
     }
 
@@ -277,6 +277,6 @@ namespace ts.OrganizeImports {
         const name2 = getExternalModuleName(m2);
         return compareBooleans(name1 === undefined, name2 === undefined) ||
             compareBooleans(isExternalModuleNameRelative(name1), isExternalModuleNameRelative(name2)) ||
-            compareStringsCaseSensitive(name1, name2);
+            compareStringsCaseInsensitive(name1, name2);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -254,7 +254,7 @@ namespace ts.OrganizeImports {`
`254`	`254`	`}`
`255`	`255`
`256`	`256`	`function compareIdentifiers(s1: Identifier, s2: Identifier) {`
`257`		`- return compareStringsCaseSensitive(s1.text, s2.text);`
	`257`	`+ return compareStringsCaseInsensitive(s1.text, s2.text);`
`258`	`258`	`}`
`259`	`259`	`}`
`260`	`260`
`@@ -277,6 +277,6 @@ namespace ts.OrganizeImports {`
`277`	`277`	`const name2 = getExternalModuleName(m2);`
`278`	`278`	`return compareBooleans(name1 === undefined, name2 === undefined) \|\|`
`279`	`279`	`compareBooleans(isExternalModuleNameRelative(name1), isExternalModuleNameRelative(name2)) \|\|`
`280`		`- compareStringsCaseSensitive(name1, name2);`
	`280`	`+ compareStringsCaseInsensitive(name1, name2);`
`281`	`281`	`}`
`282`	`282`	`}`