Fix case issues

Mods may not be lowercase to begin with, need to handle this
sarbian · Sep 14, 2017 · 50346dc · 50346dc
1 parent 3f8adee
commit 50346dc
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 5 deletions.
diff --git a/ModuleManager/PatchList.cs b/ModuleManager/PatchList.cs
@@ -38,14 +38,14 @@ public ModPassCollection(IEnumerable<string> modList)
                 {
                     ModPass pass = new ModPass(mod);
                     passesArray[i] = pass;
-                    passesDict.Add(mod, pass);
+                    passesDict.Add(mod.ToLowerInvariant(), pass);
                     i++;
                 }
             }
 
-            public ModPass this[string name] => passesDict[name];
+            public ModPass this[string name] => passesDict[name.ToLowerInvariant()];
 
-            public bool HasMod(string name) => passesDict.ContainsKey(name);
+            public bool HasMod(string name) => passesDict.ContainsKey(name.ToLowerInvariant());
 
             public ArrayEnumerator<ModPass> GetEnumerator() => new ArrayEnumerator<ModPass>(passesArray);
             IEnumerator<ModPass> IEnumerable<ModPass>.GetEnumerator() => GetEnumerator();

diff --git a/ModuleManagerTests/PatchListTest.cs b/ModuleManagerTests/PatchListTest.cs
@@ -21,13 +21,25 @@ public void TestConstructor()
         [Fact]
         public void TestModPasses__HasMod()
         {
-            PatchList list = new PatchList(new[] { "mod1", "mod2" });
+            PatchList list = new PatchList(new[] { "mod1", "Mod2", "MOD3" });
 
             PatchList.ModPassCollection collection = list.modPasses;
 
             Assert.True(collection.HasMod("mod1"));
+            Assert.True(collection.HasMod("Mod1"));
+            Assert.True(collection.HasMod("MOD1"));
+
             Assert.True(collection.HasMod("mod2"));
-            Assert.False(collection.HasMod("mod3"));
+            Assert.True(collection.HasMod("Mod2"));
+            Assert.True(collection.HasMod("MOD2"));
+
+            Assert.True(collection.HasMod("mod3"));
+            Assert.True(collection.HasMod("Mod3"));
+            Assert.True(collection.HasMod("MOD3"));
+
+            Assert.False(collection.HasMod("mod4"));
+            Assert.False(collection.HasMod("Mod4"));
+            Assert.False(collection.HasMod("MOD4"));
         }
 
         [Fact]