fix coroutine tests

Author: kmadsen

libnavigation-core/api/current.txt

@@ -1050,7 +1050,7 @@ package com.mapbox.navigation.core.trip {
   }
 
   public final class MapboxTripStarterKt {
-    method @com.mapbox.navigation.base.ExperimentalPreviewMapboxNavigationAPI public static void update(com.mapbox.navigation.core.trip.MapboxTripStarter, kotlin.jvm.functions.Function1<? super com.mapbox.navigation.core.trip.MapboxTripStarterOptions.Builder,com.mapbox.navigation.core.trip.MapboxTripStarterOptions.Builder> function);
+    method @com.mapbox.navigation.base.ExperimentalPreviewMapboxNavigationAPI public static com.mapbox.navigation.core.trip.MapboxTripStarter update(com.mapbox.navigation.core.trip.MapboxTripStarter, kotlin.jvm.functions.Function1<? super com.mapbox.navigation.core.trip.MapboxTripStarterOptions.Builder,com.mapbox.navigation.core.trip.MapboxTripStarterOptions.Builder> function);
   }
 
   @com.mapbox.navigation.base.ExperimentalPreviewMapboxNavigationAPI public final class MapboxTripStarterOptions {

libnavigation-core/src/test/java/com/mapbox/navigation/core/trip/MapboxTripStarterTest.kt

@@ -74,6 +74,7 @@ class MapboxTripStarterTest {
         sut.onAttached(mapboxNavigation)
         every { PermissionsManager.areLocationPermissionsGranted(any()) } returns true
         sut.setLocationPermissionGranted(true)
+        testCoroutineScope.cleanupTestCoroutines()
 
         verify(exactly = 1) { mapboxNavigation.startTripSession() }
     }
@@ -84,13 +85,10 @@ class MapboxTripStarterTest {
 
         val mapboxNavigation = mockMapboxNavigation()
         sut.onAttached(mapboxNavigation)
-        sut.setLocationPermissionGranted(true)
 
-        // Clean up coroutines early so that the error is caught by the tests.
         assertThrows(Exception::class.java) {
-            coroutineRule.createdScopes.forEach { it.cleanupTestCoroutines() }
+            sut.setLocationPermissionGranted(true)
         }
-        coroutineRule.createdScopes.clear()
     }
 
     @Test
@@ -110,9 +108,10 @@ class MapboxTripStarterTest {
 
         val mapboxNavigation = mockMapboxNavigation()
         sut.onAttached(mapboxNavigation)
-        sut.update { it.tripType(MapboxTripStarterExtra.MAPBOX_TRIP_STARTER_REPLAY_ROUTE) }
         every { PermissionsManager.areLocationPermissionsGranted(any()) } returns true
         sut.setLocationPermissionGranted(true)
+        sut.update { it.tripType(MapboxTripStarterExtra.MAPBOX_TRIP_STARTER_REPLAY_ROUTE) }
+        testCoroutineScope.cleanupTestCoroutines()
 
         verify(exactly = 1) { mapboxNavigation.startReplayTripSession() }
     }
@@ -125,6 +124,7 @@ class MapboxTripStarterTest {
 
         sut.onAttached(mapboxNavigation)
         sut.update { it.tripType(MapboxTripStarterExtra.MAPBOX_TRIP_STARTER_REPLAY_ROUTE) }
+        testCoroutineScope.cleanupTestCoroutines()
 
         verify(exactly = 0) { mapboxNavigation.stopTripSession() }
         verify(exactly = 1) { mapboxNavigation.startReplayTripSession() }