Update NuGet packages

Tyrrrz · Dec 31, 2023 · eda84c5 · eda84c5
1 parent 4277e34
commit eda84c5
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 21 deletions.
diff --git a/GitHubActionsTestLogger.Demo/GitHubActionsTestLogger.Demo.csproj b/GitHubActionsTestLogger.Demo/GitHubActionsTestLogger.Demo.csproj
@@ -8,10 +8,10 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="CSharpier.MsBuild" Version="0.26.5" PrivateAssets="all" />
+    <PackageReference Include="CSharpier.MsBuild" Version="0.26.7" PrivateAssets="all" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-    <PackageReference Include="xunit" Version="2.6.3" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5" PrivateAssets="all" />
+    <PackageReference Include="xunit" Version="2.6.4" />
+    <PackageReference Include="xunit.runner.visualstudio" Version="2.5.6" PrivateAssets="all" />
   </ItemGroup>
 
   <ItemGroup>

diff --git a/GitHubActionsTestLogger.Tests/GitHubActionsTestLogger.Tests.csproj b/GitHubActionsTestLogger.Tests/GitHubActionsTestLogger.Tests.csproj
@@ -10,11 +10,11 @@
 
   <ItemGroup>
     <PackageReference Include="coverlet.collector" Version="6.0.0" PrivateAssets="all" />
-    <PackageReference Include="CSharpier.MsBuild" Version="0.26.5" PrivateAssets="all" />
+    <PackageReference Include="CSharpier.MsBuild" Version="0.26.7" PrivateAssets="all" />
     <PackageReference Include="FluentAssertions" Version="6.12.0" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-    <PackageReference Include="xunit" Version="2.6.3" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5" PrivateAssets="all" />
+    <PackageReference Include="xunit" Version="2.6.4" />
+    <PackageReference Include="xunit.runner.visualstudio" Version="2.5.6" PrivateAssets="all" />
   </ItemGroup>
 
   <ItemGroup>

diff --git a/GitHubActionsTestLogger/GitHubActionsTestLogger.csproj b/GitHubActionsTestLogger/GitHubActionsTestLogger.csproj
@@ -21,10 +21,10 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="CSharpier.MsBuild" Version="0.26.5" PrivateAssets="all" />
+    <PackageReference Include="CSharpier.MsBuild" Version="0.26.7" PrivateAssets="all" />
     <PackageReference Include="Microsoft.TestPlatform.ObjectModel" Version="17.8.0" />
     <PackageReference Include="Microsoft.SourceLink.GitHub" Version="8.0.0" PrivateAssets="all" />
-    <PackageReference Include="PolyShim" Version="1.8.0" PrivateAssets="all" />
+    <PackageReference Include="PolyShim" Version="1.9.0" PrivateAssets="all" />
     <PackageReference Include="RazorBlade" Version="0.5.0" ExcludeAssets="compile;runtime" PrivateAssets="all" />
   </ItemGroup>
 

diff --git a/GitHubActionsTestLogger/GitHubWorkflow.cs b/GitHubActionsTestLogger/GitHubWorkflow.cs
@@ -19,7 +19,10 @@ private void InvokeCommand(
         // URL-encode certain characters to ensure they don't get parsed as command tokens
         // https://pakstech.com/blog/github-actions-workflow-commands
         static string Escape(string value) =>
-            value.Replace("%", "%25").Replace("\n", "%0A").Replace("\r", "%0D");
+            value
+                .Replace("%", "%25", StringComparison.Ordinal)
+                .Replace("\n", "%0A", StringComparison.Ordinal)
+                .Replace("\r", "%0D", StringComparison.Ordinal);
 
         var formattedOptions = options
             ?.Select(kvp => Escape(kvp.Key) + '=' + Escape(kvp.Value))

diff --git a/GitHubActionsTestLogger/TestSummaryTemplate.cshtml b/GitHubActionsTestLogger/TestSummaryTemplate.cshtml
@@ -166,9 +166,10 @@
             base.WriteLiteral(
                 literal
                     // Remove indentation
-                    .Replace("    ", "")
+                    .Replace("    ", "", StringComparison.Ordinal)
                     // Remove linebreaks
-                    .Replace("\r", "").Replace("\n", "")
+                    .Replace("\r", "", StringComparison.Ordinal)
+                    .Replace("\n", "", StringComparison.Ordinal)
             );
         }
         else

diff --git a/GitHubActionsTestLogger/Utils/Extensions/StringExtensions.cs b/GitHubActionsTestLogger/Utils/Extensions/StringExtensions.cs
@@ -46,12 +46,6 @@ public static string SubstringAfterLast(
         return index >= 0 ? str.Substring(index + sub.Length, str.Length - index - sub.Length) : "";
     }
 
-    public static string Indent(this string str, int spaces)
-    {
-        var indentUnit = new string(' ', spaces);
-        return indentUnit + str.Replace("\n", "\n" + new string(' ', spaces));
-    }
-
     public static int? TryParseInt(this string? str) =>
         int.TryParse(str, NumberStyles.Integer, CultureInfo.InvariantCulture, out var result)
             ? result

diff --git a/GitHubActionsTestLogger/Utils/Extensions/TestResultExtensions.cs b/GitHubActionsTestLogger/Utils/Extensions/TestResultExtensions.cs
@@ -16,10 +16,10 @@ internal static class TestResultExtensions
         if (string.IsNullOrWhiteSpace(testResult.TestCase.FullyQualifiedName))
             return null;
 
-        var testMethodFullyQualifiedName = testResult
-            .TestCase
-            .FullyQualifiedName
-            .SubstringUntil("(", StringComparison.OrdinalIgnoreCase);
+        var testMethodFullyQualifiedName = testResult.TestCase.FullyQualifiedName.SubstringUntil(
+            "(",
+            StringComparison.OrdinalIgnoreCase
+        );
 
         var testMethodName = testMethodFullyQualifiedName.SubstringAfterLast(
             ".",