Skip to content

Commit bc91613

Browse files
authored
Downgrade xunit skip warning to info (#10379)
1 parent c8f1628 commit bc91613

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

src/vstest.console/Internal/MSBuildLogger.cs

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,15 @@ private void TestMessageHandler(object? sender, TestRunMessageEventArgs e)
7878
SendMessage($"output-info", e.Message);
7979
break;
8080
case TestMessageLevel.Warning:
81-
SendMessage($"output-warning", e.Message);
81+
// Downgrade xUnit skip warning to info, otherwise any skipped test will report warning, which is often upgraded to error.
82+
if (e.Message.EndsWith("[SKIP]"))
83+
{
84+
SendMessage($"output-info", e.Message);
85+
}
86+
else
87+
{
88+
SendMessage($"output-warning", e.Message);
89+
}
8290
break;
8391
case TestMessageLevel.Error:
8492
SendMessage($"output-error", e.Message);

0 commit comments

Comments
 (0)