Skip to content

fix(app): fit store locator camera onto displayed stores - #22

Draft
dkhawk wants to merge 1 commit into
mainfrom
fix/store-locator-camera
Draft

dkhawk wants to merge 1 commit into
mainfrom
fix/store-locator-camera

Conversation

@dkhawk

@dkhawk dkhawk commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Moves the sample app's map camera to the displayed Colorado store locations (renderStoresOnMap) and invokes StoreLocatorMapManager.fitAllStores() once the MapView is laid out, instead of leaving the camera at LatLng(0.0, 0.0).
  • Adds MainScreenMapRenderTest.renderStoresOnMap_displaysMarkersAndFitsAllStoresInViewport to verify via Robolectric that all store markers are rendered and visible within the map viewport (showsOnScreen).

Testing

  • ./gradlew test lint spotlessCheck apiCheck docs:dokkaGenerate
  • CI instrumentation suite (VisualMapSampleTest.verifyStoreLocatorMapRendering) with updated ACTIONS_API_KEY secret.

@github-actions

Copy link
Copy Markdown

Coverage (unit tests)

No unit baseline recorded in coverage/history.csv yet, so this run only reports absolute numbers.

Module Line % Change Branch % Change
golden-testing 90.42% new 58.00% new
robolectric-testing 84.06% new 51.87% new
visual-testing 63.58% new 35.16% new
TOTAL 82.12% new 47.11% new

Line and branch coverage from unit test reports. History is recorded in coverage/history.csv after each merge to main.

This branch has not been deployed

No deployments
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.

1 participant