Filter empty lines when comparing stack traces in tests

AzureAD · Nov 14, 2024 · 3b5a830 · 3b5a830
1 parent 2a91435
commit 3b5a830
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/test/Microsoft.IdentityModel.TestUtils/IdentityComparer.cs b/test/Microsoft.IdentityModel.TestUtils/IdentityComparer.cs
@@ -1476,11 +1476,11 @@ internal static bool AreValidationErrorsEqual(ValidationError validationError1,
             else
             {
                 // If the stack trace is present, ensure it contains the expected amount of frames.
-                var stackTraceFrames = exception1.StackTrace.Trim('\n').Split('\n');
-                if (stackTraceFrames.Length != validationError2.StackFrames.Count)
+                var stackTraceFrames = exception1.StackTrace.Trim('\n').Split('\n').Where(s => !s.IsNullOrEmpty());
+                if (stackTraceFrames.Count() != validationError2.StackFrames.Count)
                 {
-                    localContext.Diffs.Add($"(exception1.StackTrace.Trim('\\n\').Split('\\n').Length != validationError2.StackFrames.Count: " +
-                        $"{stackTraceFrames.Length}, {validationError2.StackFrames.Count})");
+                    localContext.Diffs.Add($"(exception1.StackTrace.Trim('\\n\').Split('\\n').Count() != validationError2.StackFrames.Count: " +
+                        $"{stackTraceFrames.Count()}, {validationError2.StackFrames.Count})");
                     localContext.Diffs.Add($"exception1.StackTrace: {exception1.StackTrace}");
                 }
             }