Skip to content

fix: update parent node state when test completes#1799

Merged
wenytang-ms merged 1 commit into
microsoft:mainfrom
fbricon:update_parent_state
Oct 16, 2025
Merged

fix: update parent node state when test completes#1799
wenytang-ms merged 1 commit into
microsoft:mainfrom
fbricon:update_parent_state

Conversation

@fbricon

@fbricon fbricon commented Oct 13, 2025

Copy link
Copy Markdown
Contributor

Test class/package status doesn't update until the end of the full test run:
https://github.com/user-attachments/assets/2bc8c942-b407-447b-9c10-9d84c5403040

With this PR, completed tests update their parent node status, making updates/progress less frustrating:
https://github.com/user-attachments/assets/527e83da-0837-44a9-afa3-927aaa035741

Comment thread src/runners/baseRunner/RunnerResultAnalyzer.ts Outdated
@fbricon fbricon force-pushed the update_parent_state branch from faad8be to 78d1172 Compare October 15, 2025 08:16
Comment thread src/runners/baseRunner/RunnerResultAnalyzer.ts Outdated
@fbricon fbricon force-pushed the update_parent_state branch from 78d1172 to 002dcdd Compare October 15, 2025 13:34

@wenytang-ms wenytang-ms left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@wenytang-ms wenytang-ms merged commit 7c46caf into microsoft:main Oct 16, 2025
6 of 7 checks passed
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