search

getappmap / appmap-intellij-plugin

MIT License
38 stars 9 forks source link

Debounce requests for a filesystem refresh #805

Closed jansorg closed 1 month ago

jansorg commented 1 month ago

Closes https://github.com/getappmap/appmap-intellij-plugin/issues/665

This reduces the flickering of the "Indexing..." progress indicator in the status bar when a lot of AppMap data changes in a short time interval. This happens when multiple test cases are executed, for example.

I extracted the refresh logic from the command line service into a new service for better separation and easier testing. There's already a test case covering the refresh request handling. There's no test to test the debouncing of the requests, I don't think that there's a way to do this.

github-actions[bot] commented 1 month ago

AppMap runtime code review

Summary Status
Failed tests :white_check_mark: All tests passed
API changes :zero: No API changes
Security flaws :white_check_mark: None detected
Performance problems :white_check_mark: None detected
Code anti-patterns :white_check_mark: None detected
New AppMaps :zero: No new AppMaps
appland-release commented 3 weeks ago

:tada: This PR is included in version 0.75.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: