#110: Visual Studio test explorer cannot filter skipped tests by message

xunit · Jul 1, 2023 · c29a760 · c29a760
1 parent 5e96160
commit c29a760
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/xunit.runner.visualstudio/Sinks/VsExecutionSink.cs b/src/xunit.runner.visualstudio/Sinks/VsExecutionSink.cs
@@ -264,14 +264,15 @@ void HandleTestMethodCleanupFailure(MessageHandlerArgs<ITestMethodCleanupFailure
 	VsTestResult? MakeVsTestResult(
 		TestOutcome outcome,
 		ITestSkipped skippedResult) =>
-			MakeVsTestResult(outcome, skippedResult.TestCase, skippedResult.Test.DisplayName, (double)skippedResult.ExecutionTime, skippedResult.Reason);
+			MakeVsTestResult(outcome, skippedResult.TestCase, skippedResult.Test.DisplayName, (double)skippedResult.ExecutionTime, errorMessage: skippedResult.Reason);
 
 	VsTestResult? MakeVsTestResult(
 		TestOutcome outcome,
 		ITestCase testCase,
 		string displayName,
 		double executionTime = 0.0,
-		string? output = null)
+		string? output = null,
+		string? errorMessage = null)
 	{
 		var vsTestCase = FindTestCase(testCase);
 		if (vsTestCase == null)
@@ -292,6 +293,9 @@ void HandleTestMethodCleanupFailure(MessageHandlerArgs<ITestMethodCleanupFailure
 		if (!string.IsNullOrEmpty(output))
 			result.Messages.Add(new VsTestResultMessage(VsTestResultMessage.StandardOutCategory, output));
 
+		if (!string.IsNullOrEmpty(errorMessage))
+			result.ErrorMessage = errorMessage;
+
 		return result;
 	}