Skip to content

Commit 733f35f

Browse files
committed
added unit test for unsubscribing on destroy
1 parent 1ecfba5 commit 733f35f

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

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

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ import com.mapbox.navigation.core.accounts.BillingController
2020
import com.mapbox.navigation.core.arrival.ArrivalProgressObserver
2121
import com.mapbox.navigation.core.directions.session.DirectionsSession
2222
import com.mapbox.navigation.core.navigator.CacheHandleWrapper
23+
import com.mapbox.navigation.core.preview.RoutesPreviewController
2324
import com.mapbox.navigation.core.reroute.RerouteController
2425
import com.mapbox.navigation.core.reroute.RerouteState
2526
import com.mapbox.navigation.core.routealternatives.RouteAlternativesController
@@ -100,6 +101,7 @@ internal open class MapboxNavigationBaseTest {
100101
val developerMetadataAggregator: DeveloperMetadataAggregator = mockk(relaxUnitFun = true)
101102
val threadController = mockk<ThreadController>(relaxed = true)
102103
val routeProgressDataProvider = mockk<RouteProgressDataProvider>(relaxed = true)
104+
val routesPreviewController = mockk<RoutesPreviewController>(relaxed = true)
103105

104106
val applicationContext: Context = mockk(relaxed = true) {
105107
every { inferDeviceLocale() } returns Locale.US
@@ -206,6 +208,9 @@ internal open class MapboxNavigationBaseTest {
206208
every {
207209
NavigationComponentProvider.createRouteRefreshRequestDataProvider()
208210
} returns routeProgressDataProvider
211+
every {
212+
NavigationComponentProvider.createRoutesPreviewController(any())
213+
} returns routesPreviewController
209214

210215
every {
211216
navigator.create(

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

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,14 @@ internal class MapboxNavigationTest : MapboxNavigationBaseTest() {
156156
verify(exactly = 1) { tripSession.unregisterAllOffRouteObservers() }
157157
}
158158

159+
@Test
160+
fun destroy_unregisterAllRoutesPreviewObservers() {
161+
createMapboxNavigation()
162+
mapboxNavigation.onDestroy()
163+
164+
verify(exactly = 1) { routesPreviewController.unregisterAllRoutesPreviewObservers() }
165+
}
166+
159167
@Test
160168
fun init_registerOffRouteObserver_MapboxNavigation_recreated() {
161169
createMapboxNavigation()

0 commit comments

Comments
 (0)