From 56db5de6cece6ca1341664ceece0a1e186dceca8 Mon Sep 17 00:00:00 2001 From: Brad Wilson Date: Wed, 22 Nov 2023 01:34:48 -0800 Subject: [PATCH] Latest dependencies (and use ExecutionSink instead of delegating sinks) --- Versions.props | 4 ++-- src/xunit.runner.visualstudio/VsTestRunner.cs | 12 +++++++----- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/Versions.props b/Versions.props index 7cb6acc..7ae8dc5 100644 --- a/Versions.props +++ b/Versions.props @@ -9,8 +9,8 @@ 3.6.133 5.1.0 1.0.0-alpha.160 - 1.6.0 - 2.6.2 + 1.7.0-pre.10 + 2.6.3-pre.3 diff --git a/src/xunit.runner.visualstudio/VsTestRunner.cs b/src/xunit.runner.visualstudio/VsTestRunner.cs index 8f80168..b0fafa5 100644 --- 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();