Skip to content

Commit 3e71148

Browse files
committed
test: Fix test failure by mirror existing tests
The mocks included in the test require usage of `uri` methods, otherwise the test fails due to unused mocks. These changes update the test and implementation to match pre-existing code.
1 parent 077abba commit 3e71148

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

WordPress/src/main/java/org/wordpress/android/ui/deeplinks/handlers/HomeLinkHandler.kt

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import org.wordpress.android.fluxc.store.AccountStore
44
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction
55
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenMySite
66
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenLoginPrologue
7+
import org.wordpress.android.ui.deeplinks.DeepLinkingIntentReceiverViewModel
78
import org.wordpress.android.util.UriWrapper
89
import javax.inject.Inject
910

@@ -25,7 +26,12 @@ class HomeLinkHandler
2526
}
2627

2728
override fun stripUrl(uri: UriWrapper): String {
28-
return HOME_PATH;
29+
return buildString {
30+
if (uri.host == HOME_PATH) {
31+
append(DeepLinkingIntentReceiverViewModel.APPLINK_SCHEME)
32+
}
33+
append(HOME_PATH)
34+
}
2935
}
3036

3137
companion object {

WordPress/src/test/java/org/wordpress/android/ui/deeplinks/handlers/HomeLinkHandlerTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,6 @@ class HomeLinkHandlerTest {
4747

4848
val strippedUrl = homeLinkHandler.stripUrl(homeUri)
4949

50-
assertThat(strippedUrl).isEqualTo("home")
50+
assertThat(strippedUrl).isEqualTo("wordpress://home")
5151
}
5252
}

0 commit comments

Comments
 (0)