-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix an issue on android where popping in a nested navigation (through…
… Back Handler) would not pop the nested navigation
- Loading branch information
1 parent
f5be2e7
commit 05c84c4
Showing
5 changed files
with
105 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
84 changes: 84 additions & 0 deletions
84
...on/src/commonTest/kotlin/com/mirego/pilot/navigation/DefaultPilotNavigationManagerTest.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,84 @@ | ||
| package com.mirego.pilot.navigation | ||
|
|
||
| import assertk.assertThat | ||
| import assertk.assertions.containsExactly | ||
| import assertk.assertions.isEmpty | ||
| import kotlinx.coroutines.CoroutineScope | ||
| import kotlinx.coroutines.ExperimentalCoroutinesApi | ||
| import kotlinx.coroutines.test.UnconfinedTestDispatcher | ||
| import kotlin.test.Test | ||
|
|
||
| @OptIn(ExperimentalCoroutinesApi::class) | ||
| class DefaultPilotNavigationManagerTest { | ||
|
|
||
| private val testScope = CoroutineScope(UnconfinedTestDispatcher()) | ||
| private val parentListener = TestNavigationListener() | ||
| private val childListener = TestNavigationListener() | ||
| private val parentNavigationManager = DefaultPilotNavigationManager<TestNavigationRoute, Any>(testScope).apply { | ||
| listener = parentListener | ||
| } | ||
| private val childNavigationManager = DefaultPilotNavigationManager(testScope, parentNavigationManager = parentNavigationManager).apply { | ||
| listener = childListener | ||
| } | ||
|
|
||
| @Test | ||
| fun `when pushing and popping on the child not locally it should push and pop to the parent`() { | ||
| childNavigationManager.push(TestNavigationRoute.Route1, locally = false) | ||
| childNavigationManager.push(TestNavigationRoute.Route2, locally = false) | ||
| assertThat(parentListener.routes).containsExactly(TestNavigationRoute.Route1, TestNavigationRoute.Route2) | ||
| assertThat(childListener.routes).isEmpty() | ||
|
|
||
| childNavigationManager.pop(locally = false) | ||
| assertThat(parentListener.routes).containsExactly(TestNavigationRoute.Route1) | ||
| assertThat(childListener.routes).isEmpty() | ||
|
|
||
| childNavigationManager.pop(locally = false) | ||
| assertThat(parentListener.routes).isEmpty() | ||
| assertThat(childListener.routes).isEmpty() | ||
| } | ||
|
|
||
| @Test | ||
| fun `when popping locally and there is nothing to pop it should pop the parent`() { | ||
| parentNavigationManager.push(TestNavigationRoute.Route1, locally = true) | ||
| childNavigationManager.push(TestNavigationRoute.Route2, locally = true) | ||
| assertThat(parentListener.routes).containsExactly(TestNavigationRoute.Route1) | ||
| assertThat(childListener.routes).containsExactly(TestNavigationRoute.Route2) | ||
|
|
||
| childNavigationManager.pop(locally = true) | ||
| assertThat(parentListener.routes).containsExactly(TestNavigationRoute.Route1) | ||
| assertThat(childListener.routes).isEmpty() | ||
|
|
||
| childNavigationManager.pop(locally = true) | ||
| assertThat(parentListener.routes).isEmpty() | ||
| assertThat(childListener.routes).isEmpty() | ||
| } | ||
|
|
||
| private enum class TestNavigationRouteName { | ||
| ROUTE1, | ||
| ROUTE2, | ||
| } | ||
|
|
||
| private sealed class TestNavigationRoute(routeName: TestNavigationRouteName) : EnumPilotNavigationRoute(routeName) { | ||
| data object Route1 : TestNavigationRoute(TestNavigationRouteName.ROUTE1) | ||
| data object Route2 : TestNavigationRoute(TestNavigationRouteName.ROUTE2) | ||
| } | ||
|
|
||
| private class TestNavigationListener : PilotNavigationListener<TestNavigationRoute>() { | ||
| val routes = mutableListOf<TestNavigationRoute>() | ||
|
|
||
| override fun push(route: TestNavigationRoute): Boolean { | ||
| routes.add(route) | ||
| return true | ||
| } | ||
|
|
||
| override fun pop() { | ||
| routes.removeLastOrNull() | ||
| } | ||
|
|
||
| override fun popTo(route: TestNavigationRoute, inclusive: Boolean) { | ||
| while (routes.last() != route) { | ||
| routes.removeLast() | ||
| } | ||
| } | ||
| } | ||
| } |