Removed the case checking on upsert. Everything is forced to lowercas…

…e anyway.
TwitchPlaysPokemon · Dec 14, 2023 · 88313ae · 88313ae
1 parent 69c5917
commit 88313ae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/TPP.Persistence.MongoDB/Repos/ResponseCommandRepo.cs b/TPP.Persistence.MongoDB/Repos/ResponseCommandRepo.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Immutable;
 using System.Threading.Tasks;
+using System.Xml.Linq;
 using MongoDB.Bson.Serialization;
 using MongoDB.Driver;
 using TPP.Model;
@@ -39,7 +40,7 @@ public async Task<ResponseCommand> UpsertCommand(string command, string response
             var commandLower = command.ToLower();
             ResponseCommand newCommand = new(commandLower, response);
             ResponseCommand? oldCommand = await Collection.FindOneAndReplaceAsync(
-                Builders<ResponseCommand>.Filter.Eq(c => c.Command.ToLower(), commandLower),
+                Builders<ResponseCommand>.Filter.Eq(c => c.Command, commandLower),
                 newCommand,
                 new FindOneAndReplaceOptions<ResponseCommand>
                 {
@@ -54,8 +55,7 @@ public async Task<ResponseCommand> UpsertCommand(string command, string response
 
         public async Task<bool> RemoveCommand(string command)
         {
-            var commandLower = command.ToLower();
-            DeleteResult deleteOneAsync = await Collection.DeleteOneAsync(c => c.Command.ToLower() == commandLower);
+            DeleteResult deleteOneAsync = await Collection.DeleteOneAsync(c => c.Command.Equals(command, StringComparison.InvariantCultureIgnoreCase));
             CommandRemoved?.Invoke(this, command);
             return deleteOneAsync.DeletedCount > 0;
         }

diff --git a/tests/TPP.Persistence.MongoDB.Tests/Repos/ResponseCommandRepoTest.cs b/tests/TPP.Persistence.MongoDB.Tests/Repos/ResponseCommandRepoTest.cs
@@ -13,11 +13,11 @@ public async Task persists_and_deletes_successfully()
             ResponseCommandRepo repo = new(CreateTemporaryDatabase());
             Assert.That(await repo.GetCommands(), Is.Empty);
 
-            ResponseCommand command1 = await repo.UpsertCommand("command1", "response 1");
+            ResponseCommand command1 = await repo.UpsertCommand("Command1", "response 1");
             ResponseCommand command2 = await repo.UpsertCommand("command2", "response 2");
             Assert.That(await repo.GetCommands(), Is.EquivalentTo(new[] { command1, command2 }));
 
-            Assert.That(await repo.RemoveCommand("command1"), Is.True);
+            Assert.That(await repo.RemoveCommand("COMMAND1"), Is.True);
             Assert.That(await repo.RemoveCommand("command1"), Is.False); // already deleted
             Assert.That(await repo.GetCommands(), Is.EquivalentTo(new[] { command2 }));
         }
@@ -30,7 +30,7 @@ public async Task updates_existing()
 
             ResponseCommand command1 = await repo.UpsertCommand("command", "response 1");
             Assert.That(await repo.GetCommands(), Is.EquivalentTo(new[] { command1 }));
-            ResponseCommand command2 = await repo.UpsertCommand("command", "response 2");
+            ResponseCommand command2 = await repo.UpsertCommand("COMMAND", "response 2");
             Assert.That(await repo.GetCommands(), Is.EquivalentTo(new[] { command2 }));
         }
     }