target Android 13

Author: dzinad

CHANGELOG.md

@@ -12,6 +12,9 @@ Mapbox welcomes participation and contributions from everyone.
 - Fixed `MapboxNavigationApp#detach` will not fully detach. This causes `MapboxNavigation` to continue to be accessible, and causes `MapboxNavigationObserver.onDetached` to be called multiple times. [#6245](https://github.com/mapbox/mapbox-navigation-android/pull/6245)
 - Introduced `NavigationViewListener#onInfoPanelDragging` to inform user when `InfoPanel` is dragging. [#6249](https://github.com/mapbox/mapbox-navigation-android/pull/6249)
 - Introduced `NavigationViewListener#onInfoPanelSettling` to inform user when `InfoPanel` is settling. [#6249](https://github.com/mapbox/mapbox-navigation-android/pull/6249)
+- Fixed `MapboxNavigationApp#detach` will not fully detach. This causes `MapboxNavigation` to continue to be accessible, and causes `MapboxNavigationObserver.onDetached` to be called multiple times.
+- Added Android 13 support. [#6196](https://github.com/mapbox/mapbox-navigation-android/pull/6196)
+- Declared [POST_NOTIFICATIONS](https://developer.android.com/reference/android/Manifest.permission#POST_NOTIFICATIONS) permission in SDK's AndroidManifest.xml. It is highly recommended for apps to request the permission in runtime. Without it, the SDK will not be able to show the notification with trip progress in the notification drawer for apps that target Android 13 or higher. [#6196](https://github.com/mapbox/mapbox-navigation-android/pull/6196)
 
 ## Mapbox Navigation SDK 2.8.0-beta.1 - 25 August, 2022
 ### Changelog

gradle/dependencies.gradle

@@ -3,8 +3,8 @@ ext {
 
   androidVersions = [
       minSdkVersion           : 21,
-      targetSdkVersion        : 31,
-      compileSdkVersion       : 31,
+      targetSdkVersion        : 33,
+      compileSdkVersion       : 33,
       carMinSdkVersion        : 23
   ]
 

libnavigation-core/src/main/AndroidManifest.xml

@@ -5,6 +5,7 @@
     <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
     <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
+    <uses-permission android:name="android.permission.POST_NOTIFICATIONS" />
 
     <application>
 

libnavigation-core/src/main/java/com/mapbox/navigation/core/trip/service/NavigationNotificationService.kt

@@ -5,6 +5,7 @@ import android.app.Service
 import android.content.Context
 import android.content.Intent
 import android.os.IBinder
+import androidx.core.app.ServiceCompat
 import com.mapbox.navigation.core.telemetry.MapboxNavigationTelemetry
 
 /**
@@ -43,7 +44,7 @@ internal class NavigationNotificationService : Service() {
      */
     override fun onDestroy() {
         super.onDestroy()
-        stopForeground(true)
+        ServiceCompat.stopForeground(this, ServiceCompat.STOP_FOREGROUND_REMOVE)
         MapboxTripService.unregisterOneTimeNotificationDataObserver(notificationDataObserver)
     }
 }

libnavigation-core/src/test/java/com/mapbox/navigation/core/navigator/LocationExTest.kt

@@ -141,7 +141,7 @@ class LocationExTest {
             assertEquals(ZERO_VALUE, bearingAccuracyDegrees, .0f)
             assertEquals(ZERO_VALUE, speedAccuracyMetersPerSecond, .0f)
             assertEquals(ZERO_VALUE, verticalAccuracyMeters, .0f)
-            assertEquals(EMPTY_EXTRAS.toString(), extras.toMap().toString())
+            assertEquals(EMPTY_EXTRAS.toString(), extras!!.toMap().toString())
             assertEquals(IS_MOCK, isFromMockProvider)
         }
     }

libnavigation-metrics/src/test/java/com/mapbox/navigation/metrics/MapboxMetricsReporterTest.kt

@@ -110,7 +110,7 @@ class MapboxMetricsReporterTest {
         override val metricName: String
     ) : Event(), MetricEvent {
 
-        override fun writeToParcel(dest: Parcel?, flags: Int) {}
+        override fun writeToParcel(dest: Parcel, flags: Int) {}
 
         override fun describeContents(): Int = 0
 

libnavui-dropin/src/test/resources/robolectric.properties

@@ -0,0 +1 @@
+sdk=31

libnavui-maps/src/main/java/com/mapbox/navigation/ui/maps/camera/NavigationCamera.kt

@@ -436,12 +436,12 @@ class NavigationCamera(
 
         private var isCanceled = false
 
-        override fun onAnimationStart(animation: Animator?) {
+        override fun onAnimationStart(animation: Animator) {
             this@NavigationCamera.frameTransitionOptions = DEFAULT_FRAME_TRANSITION_OPT
             state = progressState
         }
 
-        override fun onAnimationEnd(animation: Animator?) {
+        override fun onAnimationEnd(animation: Animator) {
             if (!isCanceled) {
                 this@NavigationCamera.frameTransitionOptions = frameTransitionOptions
                 state = finalState
@@ -453,29 +453,29 @@ class NavigationCamera(
             updateFrame(viewportDataSource.getViewportData(), instant = false)
         }
 
-        override fun onAnimationCancel(animation: Animator?) {
+        override fun onAnimationCancel(animation: Animator) {
             isCanceled = true
         }
 
-        override fun onAnimationRepeat(animation: Animator?) {
+        override fun onAnimationRepeat(animation: Animator) {
         }
     }
 
     private fun createFrameListener() = object : Animator.AnimatorListener {
 
-        override fun onAnimationStart(animation: Animator?) {
+        override fun onAnimationStart(animation: Animator) {
             // no impl
         }
 
-        override fun onAnimationEnd(animation: Animator?) {
+        override fun onAnimationEnd(animation: Animator) {
             finishAnimation(animation as AnimatorSet)
         }
 
-        override fun onAnimationCancel(animation: Animator?) {
+        override fun onAnimationCancel(animation: Animator) {
             // no impl
         }
 
-        override fun onAnimationRepeat(animation: Animator?) {
+        override fun onAnimationRepeat(animation: Animator) {
             // no impl
         }
     }

libnavui-shield/src/test/resources/robolectric.properties

@@ -0,0 +1 @@
+sdk=31

libnavui-util/src/main/java/com/mapbox/navigation/ui/utils/internal/extensions/ViewEx.kt

@@ -40,19 +40,19 @@ fun ValueAnimator.play(
     set.play(this)
     set.interpolator = AccelerateDecelerateInterpolator()
     set.addListener(object : Animator.AnimatorListener {
-        override fun onAnimationStart(animation: Animator?) {
+        override fun onAnimationStart(animation: Animator) {
             doOnStart?.invoke()
         }
 
-        override fun onAnimationEnd(animation: Animator?) {
+        override fun onAnimationEnd(animation: Animator) {
             doOnEnd?.invoke()
         }
 
-        override fun onAnimationCancel(animation: Animator?) {
+        override fun onAnimationCancel(animation: Animator) {
             // No implementation
         }
 
-        override fun onAnimationRepeat(animation: Animator?) {
+        override fun onAnimationRepeat(animation: Animator) {
             // No implementation
         }
     })