fix test_router_callback_is_not_called

GitOrigin-RevId: 1a6515e59b838087e2f92dc8496945dbe2c93b31

Author: dzinad

instrumentation-tests/src/androidTest/java/com/mapbox/navigation/instrumentation_tests/core/CustomRouterTest.kt

@@ -56,7 +56,6 @@ import kotlinx.coroutines.flow.first
 import okhttp3.mockwebserver.MockResponse
 import org.junit.Assert.assertEquals
 import org.junit.Assert.assertTrue
-import org.junit.Assert.fail
 import org.junit.Rule
 import org.junit.Test
 
@@ -169,14 +168,15 @@ class CustomRouterTest : BaseCoreNoCleanUpTest() {
             withMapboxNavigation(
                 historyRecorderRule = historyRecorderRule,
             ) { navigation ->
+                val clientCallbackInvocationErrors = mutableListOf<String>()
                 navigation.requestRoutes(
                     testRouteOptions,
                     object : NavigationRouterCallback {
                         override fun onRoutesReady(
                             routes: List<NavigationRoute>,
                             @RouterOrigin routerOrigin: String,
                         ) {
-                            fail(
+                            clientCallbackInvocationErrors.add(
                                 "request shouldn't be completed successfully " +
                                     "if user doesn't call test router callback",
                             )
@@ -186,7 +186,7 @@ class CustomRouterTest : BaseCoreNoCleanUpTest() {
                             reasons: List<RouterFailure>,
                             routeOptions: RouteOptions,
                         ) {
-                            fail(
+                            clientCallbackInvocationErrors.add(
                                 "request shouldn't fail" +
                                     "if user doesn't call test router callback",
                             )
@@ -196,14 +196,15 @@ class CustomRouterTest : BaseCoreNoCleanUpTest() {
                             routeOptions: RouteOptions,
                             @RouterOrigin routerOrigin: String,
                         ) {
-                            fail(
+                            clientCallbackInvocationErrors.add(
                                 "request shouldn't be cancelled" +
                                     "if user doesn't call test router callback",
                             )
                         }
                     },
                 )
                 testRouteRequested.await()
+                assertEquals(emptyList<String>(), clientCallbackInvocationErrors)
             }
         }
     }