Fix up warnings

gabrielweyer · Oct 25, 2024 · 6db77c4 · 6db77c4
1 parent 81efec2
commit 6db77c4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/coster/src/AzureVmCoster/Program.cs b/coster/src/AzureVmCoster/Program.cs
@@ -30,7 +30,9 @@ public static async Task<int> Main(string[] args)
 
             return 0;
         }
+#pragma warning disable CA1031 // This is a catch-all so that we can log the exception
         catch (Exception e)
+#pragma warning restore CA1031
         {
             Console.WriteLine(e);
             return -1;

diff --git a/coster/tests/AzureVmCosterTests/Services/ArgumentReaderTests.cs b/coster/tests/AzureVmCosterTests/Services/ArgumentReaderTests.cs
@@ -27,7 +27,7 @@ public void GivenDoubleQuotePaths_ThenStripDoubleQuotes()
         var args = new[] { "-i", @"""C:\tmp\input.json""", "-c", @"""C:\tmp\input\config.json""" };
 
         // Act
-        var (inputFilePath, configurationFilePath, culture) = ArgumentReader.Read(args);
+        var (inputFilePath, configurationFilePath, _) = ArgumentReader.Read(args);
 
         // Assert
         inputFilePath.Should().Be(@"C:\tmp\input.json");