Bug 1818015 - Use custom tab to show privacy notice during onboarding

(cherry picked from commit 8661a63a378da6c76fede176824e8c8655d9d560)
pull/600/head
Alexandru2909 1 year ago committed by mergify[bot]
parent 17a74db488
commit 3c261a98f5

@ -6,6 +6,7 @@ import androidx.test.uiautomator.UiDevice
import okhttp3.mockwebserver.MockWebServer import okhttp3.mockwebserver.MockWebServer
import org.junit.After import org.junit.After
import org.junit.Before import org.junit.Before
import org.junit.Ignore
import org.junit.Rule import org.junit.Rule
import org.junit.Test import org.junit.Test
import org.mozilla.fenix.customannotations.SmokeTest import org.mozilla.fenix.customannotations.SmokeTest
@ -211,6 +212,7 @@ class OnboardingTest {
} }
@Test @Test
@Ignore("Failing due to changes from https://github.com/mozilla-mobile/firefox-android/pull/969")
fun youControlYourDataCardTest() { fun youControlYourDataCardTest() {
homeScreen { homeScreen {
verifyPrivacyNoticeCard() verifyPrivacyNoticeCard()

@ -516,10 +516,11 @@ class DefaultSessionControlController(
} }
override fun handleReadPrivacyNoticeClicked() { override fun handleReadPrivacyNoticeClicked() {
activity.openToBrowserAndLoad( activity.startActivity(
searchTermOrURL = SupportUtils.getMozillaPageUrl(SupportUtils.MozillaPage.PRIVATE_NOTICE), SupportUtils.createCustomTabIntent(
newTab = true, activity,
from = BrowserDirection.FromHome, SupportUtils.getMozillaPageUrl(SupportUtils.MozillaPage.PRIVATE_NOTICE),
),
) )
} }

@ -11,8 +11,11 @@ import io.mockk.Runs
import io.mockk.every import io.mockk.every
import io.mockk.just import io.mockk.just
import io.mockk.mockk import io.mockk.mockk
import io.mockk.mockkObject
import io.mockk.mockkStatic import io.mockk.mockkStatic
import io.mockk.slot
import io.mockk.spyk import io.mockk.spyk
import io.mockk.unmockkObject
import io.mockk.unmockkStatic import io.mockk.unmockkStatic
import io.mockk.verify import io.mockk.verify
import kotlinx.coroutines.ExperimentalCoroutinesApi import kotlinx.coroutines.ExperimentalCoroutinesApi
@ -963,14 +966,22 @@ class DefaultSessionControlControllerTest {
@Test @Test
fun handleReadPrivacyNoticeClicked() { fun handleReadPrivacyNoticeClicked() {
mockkObject(SupportUtils)
val urlCaptor = slot<String>()
every { SupportUtils.createCustomTabIntent(any(), capture(urlCaptor)) } returns mockk()
createController().handleReadPrivacyNoticeClicked() createController().handleReadPrivacyNoticeClicked()
verify { verify {
activity.openToBrowserAndLoad( activity.startActivity(
searchTermOrURL = SupportUtils.getMozillaPageUrl(SupportUtils.MozillaPage.PRIVATE_NOTICE), any(),
newTab = true,
from = BrowserDirection.FromHome,
) )
} }
assertEquals(
SupportUtils.getMozillaPageUrl(SupportUtils.MozillaPage.PRIVATE_NOTICE),
urlCaptor.captured,
)
unmockkObject(SupportUtils)
} }
@Test @Test

Loading…
Cancel
Save