Bug 1881238 - Convert private variables to functions so they don't get initialised

fenix/125.0
AndiAJ 4 months ago committed by mergify[bot]
parent 2dd692a682
commit 7907e8e606

@ -39,7 +39,7 @@ class SettingsSubMenuHttpsOnlyModeRobot {
} }
fun verifyHttpsOnlyModeIsEnabled(shouldBeEnabled: Boolean) { fun verifyHttpsOnlyModeIsEnabled(shouldBeEnabled: Boolean) {
httpsModeOnlySwitch.check( httpsModeOnlySwitch().check(
matches( matches(
if (shouldBeEnabled) { if (shouldBeEnabled) {
isChecked(true) isChecked(true)
@ -50,36 +50,36 @@ class SettingsSubMenuHttpsOnlyModeRobot {
) )
} }
fun clickHttpsOnlyModeSwitch() = httpsModeOnlySwitch.click() fun clickHttpsOnlyModeSwitch() = httpsModeOnlySwitch().click()
fun verifyHttpsOnlyModeOptionsEnabled(shouldBeEnabled: Boolean) { fun verifyHttpsOnlyModeOptionsEnabled(shouldBeEnabled: Boolean) {
allTabsOption.assertIsEnabled(shouldBeEnabled) allTabsOption().assertIsEnabled(shouldBeEnabled)
onlyPrivateTabsOption.assertIsEnabled(shouldBeEnabled) onlyPrivateTabsOption().assertIsEnabled(shouldBeEnabled)
} }
fun verifyHttpsOnlyOptionSelected(allTabsOptionSelected: Boolean, privateTabsOptionSelected: Boolean) { fun verifyHttpsOnlyOptionSelected(allTabsOptionSelected: Boolean, privateTabsOptionSelected: Boolean) {
if (allTabsOptionSelected) { if (allTabsOptionSelected) {
allTabsOption.assertIsChecked(true) allTabsOption().assertIsChecked(true)
onlyPrivateTabsOption.assertIsChecked(false) onlyPrivateTabsOption().assertIsChecked(false)
} else if (privateTabsOptionSelected) { } else if (privateTabsOptionSelected) {
allTabsOption.assertIsChecked(false) allTabsOption().assertIsChecked(false)
onlyPrivateTabsOption.assertIsChecked(true) onlyPrivateTabsOption().assertIsChecked(true)
} }
} }
fun selectHttpsOnlyModeOption(allTabsOptionSelected: Boolean, privateTabsOptionSelected: Boolean) { fun selectHttpsOnlyModeOption(allTabsOptionSelected: Boolean, privateTabsOptionSelected: Boolean) {
if (allTabsOptionSelected) { if (allTabsOptionSelected) {
allTabsOption.click() allTabsOption().click()
allTabsOption.assertIsChecked(true) allTabsOption().assertIsChecked(true)
} else if (privateTabsOptionSelected) { } else if (privateTabsOptionSelected) {
onlyPrivateTabsOption.click() onlyPrivateTabsOption().click()
onlyPrivateTabsOption.assertIsChecked(true) onlyPrivateTabsOption().assertIsChecked(true)
} }
} }
class Transition { class Transition {
fun goBack(interact: SettingsRobot.() -> Unit): SettingsRobot.Transition { fun goBack(interact: SettingsRobot.() -> Unit): SettingsRobot.Transition {
goBackButton.perform(click()) goBackButton().perform(click())
SettingsRobot().interact() SettingsRobot().interact()
return SettingsRobot.Transition() return SettingsRobot.Transition()
@ -87,9 +87,9 @@ class SettingsSubMenuHttpsOnlyModeRobot {
} }
} }
private val httpsModeOnlySwitch = onView(withId(R.id.https_only_switch)) private fun httpsModeOnlySwitch() = onView(withId(R.id.https_only_switch))
private val allTabsOption = private fun allTabsOption() =
onView( onView(
allOf( allOf(
withId(R.id.https_only_all_tabs), withId(R.id.https_only_all_tabs),
@ -97,7 +97,7 @@ private val allTabsOption =
), ),
) )
private val onlyPrivateTabsOption = private fun onlyPrivateTabsOption() =
onView( onView(
allOf( allOf(
withId(R.id.https_only_private_tabs), withId(R.id.https_only_private_tabs),
@ -105,4 +105,4 @@ private val onlyPrivateTabsOption =
), ),
) )
private val goBackButton = onView(withContentDescription("Navigate up")) private fun goBackButton() = onView(withContentDescription("Navigate up"))

Loading…
Cancel
Save