Feat/unit tests customization #68
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (methodResult.failures?.length) { | ||
| for (const failure of methodResult.failures) { | ||
| if (!failure.location) { | ||
| continue; | ||
| } |
There was a problem hiding this comment.
Legacy runner results often include failures with no location, but the new loop immediately continues when the location is missing, so those failures never produce a TestMessage. The test is marked failed with an empty message list, leaving the Test Explorer with no error text even though the runner reported a failure (the previous implementation displayed the message even without a location). Please keep the message when location is undefined so failures still show their reason.
Useful? React with 👍 / 👎.
Summary of Customizations
Changes
Added inline status decoration
Each test method now shows a clean inline decoration (status + duration) without modifying the file contents.
Improved test output mapping
Test output is now properly grouped per method and attached to the correct TestItem, ensuring that selecting a method in Test Results shows its console output and summary line.
Removed inline output artifacts
Eliminated the issue where summary/output lines were being rendered inside the editor along with decorations.
Added summary message for classes with failures
When a class-level result contains failures, a readable summary message is generated listing which methods failed.
Workspace root grouping
Fixed the issue where multiple test roots matched by wildcard patterns (*) produced duplicate namespace items (e.g., many DESENV75 entries).
Now all roots belonging to the same workspace/namespace are grouped under a single root node, while still respecting all configured relativeTestRoots.