Skip to content

feat: add coroutine Maps SDK initialization - #423

Open
arpitagarwal1301 wants to merge 1 commit into
googlemaps:mainfrom
arpitagarwal1301:fix/228-add-coroutine-version-of
Open

feat: add coroutine Maps SDK initialization#423
arpitagarwal1301 wants to merge 1 commit into
googlemaps:mainfrom
arpitagarwal1301:fix/228-add-coroutine-version-of

Conversation

@arpitagarwal1301

Copy link
Copy Markdown

Fixes #228

  • Add Context.awaitMapsSdkInitialized() for coroutine-based Maps SDK initialization.
  • Return the renderer actually loaded and preserve the optional renderer preference.
  • Throw GooglePlayServicesNotAvailableException with the SDK status code on initialization failure.
  • Add focused success and failure coverage.

Verification

  • ./gradlew :maps-ktx:testDebugUnitTest --tests 'com.google.maps.android.ktx.MapsInitializerTest'
  • ./gradlew :maps-ktx:testDebugUnitTest :maps-ktx:lintDebug
  • ./gradlew build jacocoTestReport
  • git diff --check

Limitations

  • Live Play Services initialization was not exercised on a device or emulator; the callback and status contracts are covered with static mocks and verified against Maps SDK 20.0.0 bytecode.

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.

Add coroutine version of MapsInitializer.initialize()

2 participants