Skip to content

Remove printing _attachments content to console#15520

Merged
Evangelink merged 1 commit intomicrosoft:mainfrom
nohwnd:remove-attachments-print
Mar 20, 2026
Merged

Remove printing _attachments content to console#15520
Evangelink merged 1 commit intomicrosoft:mainfrom
nohwnd:remove-attachments-print

Conversation

@nohwnd
Copy link
Copy Markdown
Member

@nohwnd nohwnd commented Mar 19, 2026

Stop dumping attachment file contents to stdout during test cleanup.

Files are still attached via \TestContext.AddResultFile\ for test result reporting, but their contents are no longer printed to the console output.

Stop dumping attachment file contents to stdout during test cleanup.
Files are still attached via TestContext.AddResultFile for test result
reporting, but their contents are no longer printed to the console.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 19, 2026 17:07
Copy link
Copy Markdown
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

Removes console output that printed attachment file contents during integration test cleanup while still attaching files to test results.

Changes:

  • Stop printing .txt/.log attachment contents to stdout during cleanup.
  • Continue attaching files via TestContext.AddResultFile for result reporting.

Comment thread test/Microsoft.TestPlatform.TestUtilities/IntegrationTestBase.cs
@nohwnd nohwnd added the 🚢 Ship it! Add to PRs where owner approves automated PR, but cannot approve because they "wrote it". label Mar 20, 2026
@Evangelink Evangelink merged commit a5bb999 into microsoft:main Mar 20, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚢 Ship it! Add to PRs where owner approves automated PR, but cannot approve because they "wrote it".

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants