Force re-decoration on preferences change#246
Conversation
- Refactor UnitDecorator.refreshAll() to be static and thread-safe using asyncExec. - Fix TEST_CASE_DECORATOR constant to match the ID in plugin.xml. - Add IPropertyChangeListener in MoreUnitPlugin to trigger refresh on global preference changes. - Explicitly trigger refresh in MoreUnitPropertyPage when project-specific properties are saved. Co-authored-by: RoiSoleil <3462260+RoiSoleil@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #246 +/- ##
============================================
+ Coverage 73.28% 73.34% +0.06%
- Complexity 3234 3239 +5
============================================
Files 420 420
Lines 14435 14442 +7
Branches 1266 1267 +1
============================================
+ Hits 10578 10593 +15
+ Misses 3325 3315 -10
- Partials 532 534 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Implemented forced re-decoration of Java elements when MoreUnit preferences or project properties change. This ensures that the test-case overlay icons are updated immediately without requiring a manual refresh or editor restart.
Changes:
UnitDecorator: MaderefreshAll()static and wrapped the label provider change event inasyncExecto ensure it runs on the UI thread.MoreUnitContants: CorrectedTEST_CASE_DECORATORto"org.moreunit.testdecorator".MoreUnitPlugin: Registered apropertyChangeListeneron the plugin's preference store.MoreUnitPropertyPage: Added a call toUnitDecorator.refreshAll()after saving project-specific properties.PR created automatically by Jules for task 102200140677153220 started by @RoiSoleil