mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
c06af9a1b3
* Add dotnet integration test report to CI
- Add --report-junit flag to dotnet integration test step to generate
JUnit XML alongside TRX, with explicit --results-directory to
centralize output in IntegrationTestResults/
- Upload JUnit XML artifacts from each matrix leg (net10.0/ubuntu,
net472/windows) as dotnet-test-results-{framework}-{os}
- Add dotnet-integration-test-report job that downloads artifacts,
runs the existing aggregate.py script, posts markdown to Job Summary,
and saves trend history via actions/cache
- Refactor aggregate.py to discover JUnit XML files recursively,
supporting both pytest (pytest.xml) and xunit (*.junit.xml) layouts
- Handle provider name derivation for dotnet artifact naming convention
- Fix nodeid collision when same test runs under multiple frameworks
by qualifying keys with provider when collisions are detected
- Improve module extraction for dotnet C# classnames (recognizes
IntegrationTests/UnitTests namespace segments)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* chore: trigger dotnet CI for report validation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* fix: use .junit extension (not .junit.xml) for xunit v3 output
xUnit v3 generates files with .junit extension, not .junit.xml.
Update upload glob and aggregate.py discovery to match.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* fix: use deterministic provider-qualified keys for dotnet tests
Always prefix dotnet test keys with provider (e.g. net10.0 (ubuntu)::TestName)
to ensure stable, comparable counts across runs regardless of file parse order.
Also show Executed (passed+failed) instead of Total in summary table.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* fix: match Python report summary format (Total, passed/total, etc.)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* feat: split dotnet report into per-framework tables
Dotnet tests run on multiple frameworks (net10.0, net472). Instead of
one combined table with unstable totals, show separate sections per
framework — each with its own summary row and per-test table. Python
reports retain the original single-table format.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Re-enable 7 flaky dotnet integration tests with increased timeouts
Increase timeouts to reduce timing-related flakiness in LLM-backed
integration tests (issue #4971):
- ExternalClientTests: 60s -> 120s default timeout
- SamplesValidationBase: 60s -> 120s default timeout
- ConsoleAppSamplesValidation: 90s -> 150s for long-running tests
- AzureFunctions SamplesValidation: 2min -> 3min orchestration timeout,
60s -> 90s per-step WaitForConditionAsync timeouts
Remove all Skip=Flaky annotations and unused SkipFlakyTimingTest constants.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Re-skip LLM non-determinism flaky tests, keep timeout fixes
Re-skip SingleAgentOrchestrationHITLSampleValidationAsync and
LongRunningToolsSampleValidationAsync - these fail due to LLM producing
extra review notifications, not timeouts. Updated skip reasons to
accurately describe the root cause. Reverted unnecessary timeout change
on the skipped LongRunningTools test.
The remaining 5 re-enabled tests with timeout increases are stable.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Enable Anthropic integration tests in CI
Replace hardcoded skip with conditional skip pattern (matching
CopilotStudio approach): tests gracefully skip when ANTHROPIC_API_KEY
is missing, and run when present.
Changes:
- AnthropicChatCompletionFixture: try/catch in InitializeAsync with
Assert.Skip on missing config (replaces hardcoded SkipReason)
- AnthropicSkillsIntegrationTests: same pattern per test method
- dotnet-build-and-test.yml: wire up ANTHROPIC_API_KEY,
ANTHROPIC_CHAT_MODEL_NAME, and ANTHROPIC_REASONING_MODEL_NAME
env vars to the integration test step
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Fix missing System using in AnthropicSkillsIntegrationTests
Add 'using System;' for InvalidOperationException in try/catch blocks.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Skip flaky SingleAgentOrchestrationChainingSampleValidationAsync
LLM non-determinism causes Assert.NotNull failures on orchestration
results. Skip until test logic is hardened against non-deterministic
LLM responses.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Re-enable HITL and LongRunningTools tests with timeout and flexibility fixes
- Remove Skip attribute from SingleAgentOrchestrationHITLSampleValidationAsync
- Remove Skip attribute from LongRunningToolsSampleValidationAsync
- Increase timeout from 120s/90s to 180s to accommodate 2+ LLM round-trips
- Replace rigid 2-cycle assertion with flexible approval logic that handles
extra review cycles from LLM non-determinism
Fixes the two failure modes identified in #4971:
1. Timeout: 120s/90s was insufficient for multiple LLM calls under CI load
2. Extra notifications: Assert.Fail on 3rd+ review cycle was too rigid
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Increase AzureFunctions LongRunningTools test timeouts from 90s to 180s
The LongRunningToolsSampleValidationAsync test in the AzureFunctions integration
tests was failing in CI with TimeoutException at the 'Content published
notification is logged' step. The 90-second timeouts are too tight for CI
environments where LLM calls and orchestration overhead can be slow.
Increased all three WaitForConditionAsync timeouts from 90s to 180s:
- Waiting for human feedback notification
- Waiting for publish notification (the step that was failing)
- Waiting for orchestration completion
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Merge main and fix dotnet report path after flaky_report rename
Merge upstream/main which renamed scripts/flaky_report/ to
scripts/integration_test_report/ (from Python PR #5454). Update the
dotnet-build-and-test workflow to reference the new path.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Add RetryFact to DurableTask and AzureFunctions integration tests
These tests interact with LLMs via stdin/stdout (DurableTask) or HTTP
(AzureFunctions) and are inherently non-deterministic. Unlike the Python
side which uses pytest-retry, the dotnet tests had no retry mechanism
and a single transient failure would fail the entire CI run.
Changes:
- Switch [Fact] to [RetryFact(2, 5000)] on all LLM-dependent tests
across ConsoleAppSamplesValidation, ExternalClientTests,
WorkflowConsoleAppSamplesValidation, and AzureFunctions SamplesValidation
- Add re-prompt mechanism to LongRunningToolsSampleValidationAsync:
if the LLM doesn't invoke the tool within 60s, re-send the prompt
(up to 2 retries) instead of burning the full timeout
- Reduce LongRunningTools timeout from 240s to 180s (re-prompt makes
the extra buffer unnecessary)
- Leave simple/deterministic tests as [Fact] (SingleAgent, unit tests)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Add persist-credentials: false to Integration Test Report checkout step
Matches the convention used by other checkout steps in this workflow
to avoid leaving GITHUB_TOKEN credentials in the local git config.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* small fixes
* disable anthropic failing tests
---------
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
514 lines
18 KiB
C#
514 lines
18 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System.Collections.Concurrent;
|
|
using System.Diagnostics;
|
|
using System.Reflection;
|
|
using Microsoft.Extensions.Configuration;
|
|
using Microsoft.Extensions.Logging;
|
|
namespace Microsoft.Agents.AI.DurableTask.IntegrationTests;
|
|
|
|
/// <summary>
|
|
/// Base class for sample validation integration tests providing shared infrastructure
|
|
/// setup and utility methods for running console app samples.
|
|
/// </summary>
|
|
public abstract class SamplesValidationBase : IAsyncLifetime
|
|
{
|
|
protected const string DtsPort = "8080";
|
|
protected const string RedisPort = "6379";
|
|
|
|
protected static readonly string DotnetTargetFramework = GetTargetFramework();
|
|
protected static readonly IConfiguration Configuration =
|
|
new ConfigurationBuilder()
|
|
.AddUserSecrets(Assembly.GetExecutingAssembly())
|
|
.AddEnvironmentVariables()
|
|
.Build();
|
|
|
|
// Semaphores for thread-safe initialization of shared infrastructure.
|
|
// xUnit may run tests in parallel, so we need to ensure that DTS emulator and Redis
|
|
// are started only once across all test instances. Using SemaphoreSlim allows async-safe
|
|
// locking, and the double-check pattern (check flag, acquire lock, check flag again)
|
|
// minimizes lock contention after initialization is complete.
|
|
private static readonly SemaphoreSlim s_dtsInitLock = new(1, 1);
|
|
private static readonly SemaphoreSlim s_redisInitLock = new(1, 1);
|
|
private static bool s_dtsInfrastructureStarted;
|
|
private static bool s_redisInfrastructureStarted;
|
|
|
|
protected SamplesValidationBase(ITestOutputHelper outputHelper)
|
|
{
|
|
this.OutputHelper = outputHelper;
|
|
}
|
|
|
|
/// <summary>
|
|
/// Gets the test output helper for logging.
|
|
/// </summary>
|
|
protected ITestOutputHelper OutputHelper { get; }
|
|
|
|
/// <summary>
|
|
/// Gets the base path to the samples directory for this test class.
|
|
/// </summary>
|
|
protected abstract string SamplesPath { get; }
|
|
|
|
/// <summary>
|
|
/// Gets whether this test class requires Redis infrastructure.
|
|
/// </summary>
|
|
protected virtual bool RequiresRedis => false;
|
|
|
|
/// <summary>
|
|
/// Gets the task hub name prefix for this test class.
|
|
/// </summary>
|
|
protected virtual string TaskHubPrefix => "sample";
|
|
|
|
/// <inheritdoc />
|
|
public async ValueTask InitializeAsync()
|
|
{
|
|
await EnsureDtsInfrastructureStartedAsync(this.OutputHelper, this.StartDtsEmulatorAsync);
|
|
|
|
if (this.RequiresRedis)
|
|
{
|
|
await EnsureRedisInfrastructureStartedAsync(this.OutputHelper, this.StartRedisAsync);
|
|
}
|
|
|
|
await Task.Delay(TimeSpan.FromSeconds(5));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Ensures DTS infrastructure is started exactly once across all test instances.
|
|
/// Static method writes to static field to avoid the code smell of instance methods modifying shared state.
|
|
/// </summary>
|
|
private static async Task EnsureDtsInfrastructureStartedAsync(ITestOutputHelper outputHelper, Func<Task> startAction)
|
|
{
|
|
if (s_dtsInfrastructureStarted)
|
|
{
|
|
return;
|
|
}
|
|
|
|
await s_dtsInitLock.WaitAsync();
|
|
try
|
|
{
|
|
if (!s_dtsInfrastructureStarted)
|
|
{
|
|
outputHelper.WriteLine("Starting shared DTS infrastructure...");
|
|
await startAction();
|
|
s_dtsInfrastructureStarted = true;
|
|
}
|
|
}
|
|
finally
|
|
{
|
|
s_dtsInitLock.Release();
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Ensures Redis infrastructure is started exactly once across all test instances.
|
|
/// Static method writes to static field to avoid the code smell of instance methods modifying shared state.
|
|
/// </summary>
|
|
private static async Task EnsureRedisInfrastructureStartedAsync(ITestOutputHelper outputHelper, Func<Task> startAction)
|
|
{
|
|
if (s_redisInfrastructureStarted)
|
|
{
|
|
return;
|
|
}
|
|
|
|
await s_redisInitLock.WaitAsync();
|
|
try
|
|
{
|
|
if (!s_redisInfrastructureStarted)
|
|
{
|
|
outputHelper.WriteLine("Starting shared Redis infrastructure...");
|
|
await startAction();
|
|
s_redisInfrastructureStarted = true;
|
|
}
|
|
}
|
|
finally
|
|
{
|
|
s_redisInitLock.Release();
|
|
}
|
|
}
|
|
|
|
/// <inheritdoc />
|
|
public ValueTask DisposeAsync()
|
|
{
|
|
GC.SuppressFinalize(this);
|
|
return default;
|
|
}
|
|
|
|
protected sealed record OutputLog(DateTime Timestamp, LogLevel Level, string Message);
|
|
|
|
/// <summary>
|
|
/// Runs a sample test by starting the console app and executing the provided test action.
|
|
/// </summary>
|
|
protected async Task RunSampleTestAsync(string samplePath, Func<Process, BlockingCollection<OutputLog>, Task> testAction)
|
|
{
|
|
string uniqueTaskHubName = $"{this.TaskHubPrefix}-{Guid.NewGuid():N}"[..^26];
|
|
|
|
// Build the sample project first so that build failures are caught immediately
|
|
// instead of silently failing inside 'dotnet run' and causing a timeout.
|
|
await this.BuildSampleAsync(samplePath);
|
|
|
|
using BlockingCollection<OutputLog> logsContainer = [];
|
|
using Process appProcess = this.StartConsoleApp(samplePath, logsContainer, uniqueTaskHubName);
|
|
|
|
try
|
|
{
|
|
await testAction(appProcess, logsContainer);
|
|
}
|
|
catch (OperationCanceledException e)
|
|
{
|
|
throw new TimeoutException("Core test logic timed out!", e);
|
|
}
|
|
finally
|
|
{
|
|
if (!logsContainer.IsAddingCompleted)
|
|
{
|
|
logsContainer.CompleteAdding();
|
|
}
|
|
|
|
await this.StopProcessAsync(appProcess);
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Writes a line to the process's stdin and flushes it.
|
|
/// </summary>
|
|
protected async Task WriteInputAsync(Process process, string input, CancellationToken cancellationToken)
|
|
{
|
|
this.OutputHelper.WriteLine($"{DateTime.Now:HH:mm:ss.fff} [{process.ProcessName}(in)]: {input}");
|
|
await process.StandardInput.WriteLineAsync(input);
|
|
await process.StandardInput.FlushAsync(cancellationToken);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Reads the next Information-level log line from the queue.
|
|
/// Returns null if cancelled or collection is completed.
|
|
/// </summary>
|
|
protected string? ReadLogLine(BlockingCollection<OutputLog> logs, CancellationToken cancellationToken)
|
|
{
|
|
try
|
|
{
|
|
while (!cancellationToken.IsCancellationRequested)
|
|
{
|
|
OutputLog log = logs.Take(cancellationToken);
|
|
|
|
if (log.Message.Contains("Unhandled exception"))
|
|
{
|
|
Assert.Fail("Console app encountered an unhandled exception.");
|
|
}
|
|
|
|
if (log.Level == LogLevel.Information)
|
|
{
|
|
return log.Message;
|
|
}
|
|
}
|
|
}
|
|
catch (OperationCanceledException)
|
|
{
|
|
return null;
|
|
}
|
|
catch (InvalidOperationException)
|
|
{
|
|
return null;
|
|
}
|
|
|
|
return null;
|
|
}
|
|
|
|
/// <summary>
|
|
/// Creates a cancellation token source with the specified timeout for test operations.
|
|
/// </summary>
|
|
protected CancellationTokenSource CreateTestTimeoutCts(TimeSpan? timeout = null)
|
|
{
|
|
TimeSpan testTimeout = Debugger.IsAttached ? TimeSpan.FromMinutes(5) : timeout ?? TimeSpan.FromSeconds(120);
|
|
return new CancellationTokenSource(testTimeout);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Allows derived classes to set additional environment variables for the console app process.
|
|
/// </summary>
|
|
protected virtual void ConfigureAdditionalEnvironmentVariables(ProcessStartInfo startInfo, Action<string, string> setEnvVar)
|
|
{
|
|
}
|
|
|
|
private static string GetTargetFramework()
|
|
{
|
|
string filePath = new Uri(typeof(SamplesValidationBase).Assembly.Location).LocalPath;
|
|
string directory = Path.GetDirectoryName(filePath)!;
|
|
string tfm = Path.GetFileName(directory);
|
|
if (tfm.StartsWith("net", StringComparison.OrdinalIgnoreCase))
|
|
{
|
|
return tfm;
|
|
}
|
|
|
|
throw new InvalidOperationException($"Unable to find target framework in path: {filePath}");
|
|
}
|
|
|
|
private async Task StartDtsEmulatorAsync()
|
|
{
|
|
if (!await this.IsDtsEmulatorRunningAsync())
|
|
{
|
|
this.OutputHelper.WriteLine("Starting DTS emulator...");
|
|
await this.RunCommandAsync("docker", "run", "-d",
|
|
"--name", "dts-emulator",
|
|
"-p", $"{DtsPort}:8080",
|
|
"-e", "DTS_USE_DYNAMIC_TASK_HUBS=true",
|
|
"mcr.microsoft.com/dts/dts-emulator:latest");
|
|
}
|
|
}
|
|
|
|
private async Task StartRedisAsync()
|
|
{
|
|
if (!await this.IsRedisRunningAsync())
|
|
{
|
|
this.OutputHelper.WriteLine("Starting Redis...");
|
|
await this.RunCommandAsync("docker", "run", "-d",
|
|
"--name", "redis",
|
|
"-p", $"{RedisPort}:6379",
|
|
"redis:latest");
|
|
}
|
|
}
|
|
|
|
private async Task<bool> IsDtsEmulatorRunningAsync()
|
|
{
|
|
this.OutputHelper.WriteLine($"Checking if DTS emulator is running at http://localhost:{DtsPort}/healthz...");
|
|
|
|
using HttpClient http2Client = new()
|
|
{
|
|
DefaultRequestVersion = new Version(2, 0),
|
|
DefaultVersionPolicy = HttpVersionPolicy.RequestVersionExact
|
|
};
|
|
|
|
try
|
|
{
|
|
using CancellationTokenSource timeoutCts = new(TimeSpan.FromSeconds(30));
|
|
using HttpResponseMessage response = await http2Client.GetAsync(
|
|
new Uri($"http://localhost:{DtsPort}/healthz"), timeoutCts.Token);
|
|
|
|
if (response.Content.Headers.ContentLength > 0)
|
|
{
|
|
string content = await response.Content.ReadAsStringAsync(timeoutCts.Token);
|
|
this.OutputHelper.WriteLine($"DTS emulator health check response: {content}");
|
|
}
|
|
|
|
bool isRunning = response.IsSuccessStatusCode;
|
|
this.OutputHelper.WriteLine(isRunning ? "DTS emulator is running" : $"DTS emulator not running. Status: {response.StatusCode}");
|
|
return isRunning;
|
|
}
|
|
catch (HttpRequestException ex)
|
|
{
|
|
this.OutputHelper.WriteLine($"DTS emulator is not running: {ex.Message}");
|
|
return false;
|
|
}
|
|
}
|
|
|
|
private async Task<bool> IsRedisRunningAsync()
|
|
{
|
|
this.OutputHelper.WriteLine($"Checking if Redis is running at localhost:{RedisPort}...");
|
|
|
|
try
|
|
{
|
|
using CancellationTokenSource timeoutCts = new(TimeSpan.FromSeconds(30));
|
|
ProcessStartInfo startInfo = new()
|
|
{
|
|
FileName = "docker",
|
|
Arguments = "exec redis redis-cli ping",
|
|
UseShellExecute = false,
|
|
RedirectStandardOutput = true,
|
|
RedirectStandardError = true,
|
|
CreateNoWindow = true
|
|
};
|
|
|
|
using Process process = new() { StartInfo = startInfo };
|
|
if (!process.Start())
|
|
{
|
|
this.OutputHelper.WriteLine("Failed to start docker exec command");
|
|
return false;
|
|
}
|
|
|
|
string output = await process.StandardOutput.ReadToEndAsync(timeoutCts.Token);
|
|
await process.WaitForExitAsync(timeoutCts.Token);
|
|
|
|
bool isRunning = process.ExitCode == 0 && output.Contains("PONG", StringComparison.OrdinalIgnoreCase);
|
|
this.OutputHelper.WriteLine(isRunning ? "Redis is running" : $"Redis not running. Exit: {process.ExitCode}, Output: {output}");
|
|
return isRunning;
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
this.OutputHelper.WriteLine($"Redis is not running: {ex.Message}");
|
|
return false;
|
|
}
|
|
}
|
|
|
|
private async Task BuildSampleAsync(string samplePath)
|
|
{
|
|
this.OutputHelper.WriteLine($"Building sample at {samplePath}...");
|
|
|
|
ProcessStartInfo buildInfo = new()
|
|
{
|
|
FileName = "dotnet",
|
|
Arguments = $"build --framework {DotnetTargetFramework}",
|
|
WorkingDirectory = samplePath,
|
|
UseShellExecute = false,
|
|
RedirectStandardOutput = true,
|
|
RedirectStandardError = true,
|
|
};
|
|
|
|
using Process buildProcess = new() { StartInfo = buildInfo };
|
|
buildProcess.Start();
|
|
|
|
// Read both streams asynchronously to avoid deadlocks from filled pipe buffers
|
|
Task<string> stdoutTask = buildProcess.StandardOutput.ReadToEndAsync();
|
|
Task<string> stderrTask = buildProcess.StandardError.ReadToEndAsync();
|
|
|
|
using CancellationTokenSource buildCts = new(TimeSpan.FromMinutes(5));
|
|
try
|
|
{
|
|
await buildProcess.WaitForExitAsync(buildCts.Token);
|
|
}
|
|
catch (OperationCanceledException)
|
|
{
|
|
buildProcess.Kill(entireProcessTree: true);
|
|
throw new TimeoutException($"Build timed out after 5 minutes for sample at {samplePath}.");
|
|
}
|
|
|
|
await Task.WhenAll(stdoutTask, stderrTask);
|
|
|
|
string stdout = stdoutTask.Result;
|
|
string stderr = stderrTask.Result;
|
|
if (buildProcess.ExitCode != 0)
|
|
{
|
|
throw new InvalidOperationException($"Failed to build sample at {samplePath}:\n{stdout}\n{stderr}");
|
|
}
|
|
|
|
this.OutputHelper.WriteLine($"Build completed for {samplePath}.");
|
|
}
|
|
|
|
private Process StartConsoleApp(string samplePath, BlockingCollection<OutputLog> logs, string taskHubName)
|
|
{
|
|
ProcessStartInfo startInfo = new()
|
|
{
|
|
FileName = "dotnet",
|
|
Arguments = $"run --no-build --framework {DotnetTargetFramework}",
|
|
WorkingDirectory = samplePath,
|
|
UseShellExecute = false,
|
|
RedirectStandardOutput = true,
|
|
RedirectStandardError = true,
|
|
RedirectStandardInput = true,
|
|
};
|
|
|
|
string openAiEndpoint = Configuration["AZURE_OPENAI_ENDPOINT"] ??
|
|
throw new InvalidOperationException("The required AZURE_OPENAI_ENDPOINT env variable is not set.");
|
|
string openAiDeployment = Configuration["AZURE_OPENAI_CHAT_DEPLOYMENT_NAME"] ??
|
|
throw new InvalidOperationException("The required AZURE_OPENAI_CHAT_DEPLOYMENT_NAME env variable is not set.");
|
|
|
|
void SetAndLogEnvironmentVariable(string key, string value)
|
|
{
|
|
this.OutputHelper.WriteLine($"Setting environment variable for {startInfo.FileName} sub-process: {key}={value}");
|
|
startInfo.EnvironmentVariables[key] = value;
|
|
}
|
|
|
|
SetAndLogEnvironmentVariable("AZURE_OPENAI_ENDPOINT", openAiEndpoint);
|
|
SetAndLogEnvironmentVariable("AZURE_OPENAI_DEPLOYMENT", openAiDeployment);
|
|
SetAndLogEnvironmentVariable("DURABLE_TASK_SCHEDULER_CONNECTION_STRING",
|
|
$"Endpoint=http://localhost:{DtsPort};TaskHub={taskHubName};Authentication=None");
|
|
|
|
this.ConfigureAdditionalEnvironmentVariables(startInfo, SetAndLogEnvironmentVariable);
|
|
|
|
Process process = new() { StartInfo = startInfo, EnableRaisingEvents = true };
|
|
|
|
process.ErrorDataReceived += (sender, e) => this.HandleProcessOutput(e.Data, startInfo.FileName, "err", LogLevel.Error, logs);
|
|
process.OutputDataReceived += (sender, e) => this.HandleProcessOutput(e.Data, startInfo.FileName, "out", LogLevel.Information, logs);
|
|
|
|
// When the process exits unexpectedly (e.g. build failure), complete the log collection
|
|
// so that ReadLogLine returns null immediately instead of blocking until the test timeout.
|
|
process.Exited += (sender, e) =>
|
|
{
|
|
if (!logs.IsAddingCompleted)
|
|
{
|
|
logs.CompleteAdding();
|
|
}
|
|
};
|
|
|
|
if (!process.Start())
|
|
{
|
|
throw new InvalidOperationException("Failed to start the console app");
|
|
}
|
|
|
|
process.BeginErrorReadLine();
|
|
process.BeginOutputReadLine();
|
|
|
|
return process;
|
|
}
|
|
|
|
private void HandleProcessOutput(string? data, string processName, string stream, LogLevel level, BlockingCollection<OutputLog> logs)
|
|
{
|
|
if (data is null)
|
|
{
|
|
return;
|
|
}
|
|
|
|
string logMessage = $"{DateTime.Now:HH:mm:ss.fff} [{processName}({stream})]: {data}";
|
|
this.OutputHelper.WriteLine(logMessage);
|
|
Debug.WriteLine(logMessage);
|
|
|
|
try
|
|
{
|
|
logs.Add(new OutputLog(DateTime.Now, level, data));
|
|
}
|
|
catch (InvalidOperationException)
|
|
{
|
|
// Collection completed
|
|
}
|
|
}
|
|
|
|
private async Task RunCommandAsync(string command, params string[] args)
|
|
{
|
|
ProcessStartInfo startInfo = new()
|
|
{
|
|
FileName = command,
|
|
Arguments = string.Join(" ", args),
|
|
UseShellExecute = false,
|
|
RedirectStandardOutput = true,
|
|
RedirectStandardError = true,
|
|
CreateNoWindow = true
|
|
};
|
|
|
|
this.OutputHelper.WriteLine($"Running command: {command} {string.Join(" ", args)}");
|
|
|
|
using Process process = new() { StartInfo = startInfo };
|
|
process.ErrorDataReceived += (sender, e) => this.OutputHelper.WriteLine($"[{command}(err)]: {e.Data}");
|
|
process.OutputDataReceived += (sender, e) => this.OutputHelper.WriteLine($"[{command}(out)]: {e.Data}");
|
|
|
|
if (!process.Start())
|
|
{
|
|
throw new InvalidOperationException("Failed to start the command");
|
|
}
|
|
|
|
process.BeginErrorReadLine();
|
|
process.BeginOutputReadLine();
|
|
|
|
using CancellationTokenSource cts = new(TimeSpan.FromMinutes(1));
|
|
await process.WaitForExitAsync(cts.Token);
|
|
|
|
this.OutputHelper.WriteLine($"Command completed with exit code: {process.ExitCode}");
|
|
}
|
|
|
|
private async Task StopProcessAsync(Process process)
|
|
{
|
|
try
|
|
{
|
|
if (!process.HasExited)
|
|
{
|
|
this.OutputHelper.WriteLine($"{DateTime.Now:HH:mm:ss.fff} Killing process {process.ProcessName}#{process.Id}");
|
|
process.Kill(entireProcessTree: true);
|
|
|
|
using CancellationTokenSource cts = new(TimeSpan.FromSeconds(10));
|
|
await process.WaitForExitAsync(cts.Token);
|
|
this.OutputHelper.WriteLine($"{DateTime.Now:HH:mm:ss.fff} Process exited: {process.Id}");
|
|
}
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
this.OutputHelper.WriteLine($"{DateTime.Now:HH:mm:ss.fff} Failed to stop process: {ex.Message}");
|
|
}
|
|
}
|
|
}
|