Latest dependencies (and use ExecutionSink instead of delegating sinks)

xunit · Nov 22, 2023 · 56db5de · 56db5de
1 parent 1f419e6
commit 56db5de
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/Versions.props b/Versions.props
@@ -9,8 +9,8 @@
     <NerdbankGitVersioningVersion>3.6.133</NerdbankGitVersioningVersion>
     <NSubstituteVersion>5.1.0</NSubstituteVersion>
     <TunnelVisionLabsReferenceAssemblyAnnotatorVersion>1.0.0-alpha.160</TunnelVisionLabsReferenceAssemblyAnnotatorVersion>
-    <XunitAnalyzersVersion>1.6.0</XunitAnalyzersVersion>
-    <XunitVersion>2.6.2</XunitVersion>
+    <XunitAnalyzersVersion>1.7.0-pre.10</XunitAnalyzersVersion>
+    <XunitVersion>2.6.3-pre.3</XunitVersion>
   </PropertyGroup>
 
 </Project>
diff --git a/src/xunit.runner.visualstudio/VsTestRunner.cs b/src/xunit.runner.visualstudio/VsTestRunner.cs
@@ -541,11 +541,13 @@ static bool IsXunitPackageReferenced(string assemblyFileName)
 				reporterMessageHandler.OnMessage(new TestAssemblyExecutionStarting(runInfo.Assembly, executionOptions));
 
 				using var vsExecutionSink = new VsExecutionSink(reporterMessageHandler, frameworkHandle, logger, testCasesMap, () => cancelled);
-				IExecutionSink resultsSink = vsExecutionSink;
-				if (longRunningSeconds > 0)
-					resultsSink = new DelegatingLongRunningTestDetectionSink(resultsSink, TimeSpan.FromSeconds(longRunningSeconds), diagnosticsSinkLocal);
-				if (configuration.FailSkipsOrDefault)
-					resultsSink = new DelegatingFailSkipSink(resultsSink);
+				var executionSinkOptions = new ExecutionSinkOptions
+				{
+					DiagnosticMessageSink = diagnosticsSinkRemote,
+					FailSkips = configuration.FailSkipsOrDefault,
+					LongRunningTestTime = TimeSpan.FromSeconds(longRunningSeconds),
+				};
+				var resultsSink = new ExecutionSink(vsExecutionSink, executionSinkOptions);
 
 				controller.RunTests(testCases, resultsSink, executionOptions);
 				resultsSink.Finished.WaitOne();