- Notifications
You must be signed in to change notification settings - Fork548
[tests] Improve test summary to include up to 3 failing unit tests.#24282
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR improves test result summaries by adding detailed information about up to 3 failing unit tests from TRX test result files. The changes introduce a new TRX parser utility that extracts test failure details and integrates it into multiple reporting systems.
- Adds a new
TrxParserutility class to parse TRX test result files and extract failed test information - Updates markdown and HTML report writers to display the first 3 failing tests with truncated names and messages
- Modifies PowerShell test results script to include parsed TRX content in summaries
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/common/ParseTrxFile.cs | New TRX parser utility class that extracts failed test information from TRX XML files |
| tests/xharness/xharness.csproj | Links the new ParseTrxFile.cs into the xharness project |
| tests/xharness/Jenkins/Reports/MarkdownReportWriter.cs | Integrates TRX parsing to display up to 3 failing unit tests in markdown reports with aCap string extension method |
| scripts/create-windows-html-report/create-windows-html-report.csproj | Links ParseTrxFile.cs into the HTML report generator project |
| scripts/create-windows-html-report/create-windows-html-report.cs | Refactors to use TrxParser instead of inline XML parsing for cleaner code |
| tools/devops/automation/scripts/TestResults.psm1 | Updates PowerShell script to parse and include TRX results in test summaries |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.