Skip to content

Fix test expectations for terminal output formatting after stdout/stderr removal#7601

Merged
Youssef1313 merged 2 commits intodev/ygerges/polluted-outputfrom
copilot/sub-pr-7600
Mar 24, 2026
Merged

Fix test expectations for terminal output formatting after stdout/stderr removal#7601
Youssef1313 merged 2 commits intodev/ygerges/polluted-outputfrom
copilot/sub-pr-7600

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 24, 2026

After removing StandardOutputProperty/StandardErrorProperty rendering from the terminal reporter, the test expectations for SimpleAnsiTerminal_OutputFormattingIsCorrect and AnsiTerminal_OutputFormattingIsCorrect were left with an empty blank line where the actual output emits an ANSI reset (\e[m) sequence on that line.

  • Test fix: Added ␛[m on the blank line following the stack trace in both test expected strings to match actual terminal output

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Fix console output for stdout/stderr properties Fix test expectations for terminal output formatting after stdout/stderr removal Mar 24, 2026
Copilot AI requested a review from Youssef1313 March 24, 2026 13:22
@Youssef1313 Youssef1313 marked this pull request as ready for review March 24, 2026 13:27
@Youssef1313 Youssef1313 merged commit bfd1562 into dev/ygerges/polluted-output Mar 24, 2026
2 of 11 checks passed
@Youssef1313 Youssef1313 deleted the copilot/sub-pr-7600 branch March 24, 2026 13:27
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.

2 participants