Skip to content

[ Method Browser ] Test result announcement refresh#1373

Merged
jecisc merged 2 commits intopharo-spec:Pharo14from
AlexisCnockaert:methodBrowser/testFeature
Apr 7, 2026
Merged

[ Method Browser ] Test result announcement refresh#1373
jecisc merged 2 commits intopharo-spec:Pharo14from
AlexisCnockaert:methodBrowser/testFeature

Conversation

@AlexisCnockaert
Copy link
Copy Markdown
Contributor

Now the method browser listen to Test announcements, meaning if we browse some test method and any of them is run elsewhere, the icons in the method browser will refresh based on test result.
Also cleaned some code !

Now the method browser listen to Test announcements, meaning if we browse some test method and any of them is run elsewhere, the icons in the method browser will refresh based on test result.
Also cleaned some code !
@AlexisCnockaert
Copy link
Copy Markdown
Contributor Author

Also a good improvement that could be done in the future is to only refresh the methods that were impacted by the announcement.
Because in case we have a method browser with a ton of methods it could be slow and bring problems

@AlexisCnockaert AlexisCnockaert requested a review from Ducasse April 7, 2026 13:59
Copy link
Copy Markdown
Member

@jecisc jecisc left a comment

Choose a reason for hiding this comment

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

This seems nice!

@jecisc jecisc merged commit 1ebe3c5 into pharo-spec:Pharo14 Apr 7, 2026
1 of 2 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