Skip to content

feat: register HTML report as MTP session artifact #5270

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thomhurst merged 13 commits into from
Mar 27, 2026
Merged

feat: register HTML report as MTP session artifact #5270

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
8faac1e
feat: add NotifySession method to HtmlReporter for artifact publishing
thomhurst Mar 27, 2026
9c36c70
refactor: atomic session context assignment in HtmlReporter
thomhurst Mar 27, 2026
ce60f86
feat: HtmlReporter publishes SessionFileArtifact after writing report
thomhurst Mar 27, 2026
2efe378
test: add publish-path tests for HtmlReporter artifact publishing
thomhurst Mar 27, 2026
db087d0
feat: thread HtmlReporter into TUnitTestFramework for session context…
thomhurst Mar 27, 2026
5f6359a
feat: pass htmlReporter to TUnitTestFramework for artifact registration
thomhurst Mar 27, 2026
d38bd79
fix: use application-level IMessageBus to avoid drained-bus error in …
thomhurst Mar 27, 2026
327d298
fix: remove dead SetSessionContext and update tests to use SetMessageBus
thomhurst Mar 27, 2026
5cad693
refactor: revert HtmlReporter to internal, explicit SessionUid in tes…
thomhurst Mar 27, 2026
36f72f1
refactor: simplify and deduplicate — extract shared MockExtension, fi…
thomhurst Mar 27, 2026
aaedb4e
fix: rename test to MessageBus_Not_Injected, fix csproj indentation
thomhurst Mar 27, 2026
462c5b6
docs: document auto-registration, SetMessageBus ordering, and write-f…
thomhurst Mar 27, 2026
fae8544
refactor: extract CapturingMessageBus to TestInfrastructure
thomhurst Mar 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
feat: add NotifySession method to HtmlReporter for artifact publishing
  • Loading branch information
@thomhurst
thomhurst committed Mar 27, 2026
commit 8faac1e42a8c86f3067fa7708feef78c6cd7c054
10 changes: 10 additions & 0 deletions TUnit.Engine/Reporters/Html/HtmlReporter.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@
using System.Text.Json;
using Microsoft.Testing.Platform.Extensions;
using Microsoft.Testing.Platform.Extensions.Messages;
using Microsoft.Testing.Platform.Messages;
using Microsoft.Testing.Platform.Extensions.TestHost;
using Microsoft.Testing.Platform.TestHost;
using TUnit.Core;
using TUnit.Engine.Configuration;
using TUnit.Engine.Constants;
Expand All @@ -19,6 +21,8 @@ namespace TUnit.Engine.Reporters.Html;
internal sealed class HtmlReporter(IExtension extension) : IDataConsumer, ITestHostApplicationLifetime, IFilterReceiver, IDisposable
{
private string? _outputPath;
private IMessageBus? _messageBus;
private SessionUid? _sessionUid;
private readonly ConcurrentDictionary<string, ConcurrentQueue<TestNodeUpdateMessage>> _updates = [];

#if NET
Expand Down Expand Up @@ -135,6 +139,12 @@ internal void SetOutputPath(string path)
_outputPath = path;
}

internal void NotifySession(IMessageBus messageBus, SessionUid sessionUid)
{
_messageBus = messageBus;
_sessionUid = sessionUid;
}

private ReportData BuildReportData()
{
var assemblyName = Assembly.GetEntryAssembly()?.GetName().Name ?? "TestResults";
Expand Down