Skip to content

Print out-of-process artifacts#7534

Merged
Evangelink merged 1 commit intomainfrom
dev/ygerges/print-oop-artifacts
Mar 12, 2026
Merged

Print out-of-process artifacts#7534
Evangelink merged 1 commit intomainfrom
dev/ygerges/print-oop-artifacts

Conversation

@Youssef1313
Copy link
Member

@Youssef1313 Youssef1313 commented Mar 12, 2026

Fixes #7471

image

It's a little bit ugly because Out of process artifacts are printed at the very end, but it's better than not showing the info at all.

Copilot AI review requested due to automatic review settings March 12, 2026 14:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds terminal output for artifacts produced in non-TestHost (out-of-process) scenarios so they’re visible at the end of a test session.

Changes:

  • Invoke artifact printing on session end when running outside the TestHost role.
  • Add TerminalTestReporter.PrintOutOfProcessArtifacts() to render collected artifact paths to the terminal.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/Platform/Microsoft.Testing.Platform/OutputDevice/TerminalOutputDevice.cs Calls artifact printing during session teardown for non-TestHost process roles.
src/Platform/Microsoft.Testing.Platform/OutputDevice/Terminal/TerminalTestReporter.cs Adds a new method to print artifact links collected during execution.

You can also share your feedback on Copilot code review. Take the survey.

@Evangelink Evangelink enabled auto-merge March 12, 2026 15:29
@Evangelink Evangelink merged commit c2e9469 into main Mar 12, 2026
15 checks passed
@Evangelink Evangelink deleted the dev/ygerges/print-oop-artifacts branch March 12, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

File artifacts message is not printed when running with an out-of-proc extension

3 participants