target Android 13

Author: dzinad

CHANGELOG.md

@@ -9,6 +9,9 @@ Mapbox welcomes participation and contributions from everyone.
 - Fixed routes’ incorrect duration: now it relays on annotations if provided, otherwise on steps’s duration. [#6237](https://github.com/mapbox/mapbox-navigation-android/pull/6237)
 - Added `guideMapUri` to the `RestStop`. [#6237](https://github.com/mapbox/mapbox-navigation-android/pull/6237)
 - [TileStore Android Service] Fixed a crash when the service process is killed by the Android system. [#6237](https://github.com/mapbox/mapbox-navigation-android/pull/6237)
+- 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)
+- Bumped targetSdkVersion to 33. [#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
         }
     })