NAVAND-552: nullify lastDownloadTime in onDetached

Author: dzinad

libnavui-voice/src/main/java/com/mapbox/navigation/ui/voice/api/VoiceInstructionsPrefetcher.kt

@@ -63,6 +63,7 @@ class VoiceInstructionsPrefetcher internal constructor(
      * See [MapboxNavigationObserver.onDetached].
      */
     override fun onDetached(mapboxNavigation: MapboxNavigation) {
+        lastDownloadTime = 0
         mapboxNavigation.unregisterRoutesObserver(routesObserver)
         mapboxNavigation.unregisterRouteProgressObserver(routeProgressObserver)
         speechApi.destroy()

libnavui-voice/src/test/java/com/mapbox/navigation/ui/voice/api/VoiceInstructionsPrefetcherTest.kt

@@ -336,6 +336,22 @@ class VoiceInstructionsPrefetcherTest {
         verify(exactly = 0) { speechAPI.predownload(any()) }
     }
 
+    @Test
+    fun `onDetached should nullify lastDownloadTime`() {
+        val mapboxNavigation = mockk<MapboxNavigation>(relaxed = true)
+        val route = validRoute()
+        onRoutesChanged(
+            routesUpdatedResult(listOf(route), RoutesExtra.ROUTES_UPDATE_REASON_NEW)
+        )
+        sut.onDetached(mapboxNavigation)
+        clearMocks(speechAPI, answers = false)
+        every { timeProvider.seconds() } returns currentTimeSeconds + 49
+
+        onRouteProgressChanged(validRouteProgress())
+
+        verify(exactly = 1) { speechAPI.predownload(any()) }
+    }
+
     private fun routesUpdatedResult(
         routes: List<NavigationRoute>,
         @RoutesExtra.RoutesUpdateReason mockReason: String