Skip to content

feat: add constructor parameter for custom Executor in cluster renderers#1682

Merged
dkhawk merged 1 commit intomainfrom
feat/injectable-executor
Apr 24, 2026
Merged

feat: add constructor parameter for custom Executor in cluster renderers#1682
dkhawk merged 1 commit intomainfrom
feat/injectable-executor

Conversation

@kikoso
Copy link
Copy Markdown
Collaborator

@kikoso kikoso commented Apr 24, 2026

Adds overloaded constructors to ClusterRendererMultipleItems, DefaultClusterRenderer, and DefaultAdvancedMarkersClusterRenderer that accept an Executor parameter. This allows injecting a synchronous executor (or any custom executor) during testing, resolving issues with Executors.newSingleThreadExecutor() creating background threads in test environments.

Adds overloaded constructors to `ClusterRendererMultipleItems`, `DefaultClusterRenderer`, and `DefaultAdvancedMarkersClusterRenderer` that accept an `Executor` parameter. This allows injecting a synchronous executor (or any custom executor) during testing, resolving issues with `Executors.newSingleThreadExecutor()` creating background threads in test environments.
@googlemaps-bot
Copy link
Copy Markdown
Contributor

Code Coverage

Overall Project 41.07% -0.14% 🍏
Files changed 0%

File Coverage
ClusterRendererMultipleItems.java 0% -0.44%
DefaultAdvancedMarkersClusterRenderer.java 0% -0.59%
DefaultClusterRenderer.java 0% -0.59%

@dkhawk dkhawk merged commit b803dc0 into main Apr 24, 2026
8 checks passed
@dkhawk dkhawk deleted the feat/injectable-executor branch April 24, 2026 16:57
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.

3 participants