Archives/iceraven-browser
2
0
Fork 0
mirror of https://github.com/fork-maintainers/iceraven-browser synced 2024-11-11 13:11:01 +00:00
Code Issues Releases Wiki Activity

Close #20792: Fix intermittent test failures in QuickSettingsFragmentReducerTest

Browse Source
This commit is contained in:
Jonathan Almeida 2021-08-11 17:12:48 -04:00 committed by Jonathan Almeida
parent 8b80cccd1b
commit 545a59ac97
2 changed files with 19 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/src/main/java/org/mozilla/fenix/settings/quicksettings/QuickSettingsFragmentReducer.kt
View File

@ -7,7 +7,7 @@ package org.mozilla.fenix.settings.quicksettings
/**
* Parent Reducer for all [QuickSettingsFragmentState]s of all Views shown in this Fragment.
*/
fun quickSettingsFragmentReducer(
internal fun quickSettingsFragmentReducer(
state: QuickSettingsFragmentState,
action: QuickSettingsFragmentAction
): QuickSettingsFragmentState {

27
app/src/test/java/org/mozilla/fenix/settings/quicksettings/QuickSettingsFragmentReducerTest.kt
View File

@ -4,16 +4,14 @@
package org.mozilla.fenix.settings.quicksettings
import mozilla.components.support.test.mock
import mozilla.components.feature.sitepermissions.SitePermissionsRules
import org.junit.Assert.assertEquals
import org.junit.Assert.assertTrue
import org.junit.Ignore
import org.junit.Test
import org.mozilla.fenix.settings.PhoneFeature
class QuickSettingsFragmentReducerTest {
@Ignore("See https://github.com/mozilla-mobile/fenix/issues/20792")
@Test
fun `WebsitePermissionAction - TogglePermission`() {
val toggleablePermission = WebsitePermission.Toggleable(
@ -26,8 +24,8 @@ class QuickSettingsFragmentReducerTest {
val map =
mapOf<PhoneFeature, WebsitePermission>(PhoneFeature.CAMERA to toggleablePermission)
val state = QuickSettingsFragmentState(mock(), map)
val infoState = WebsiteInfoState("", "", WebsiteSecurityUiValues.SECURE, "")
val state = QuickSettingsFragmentState(infoState, map)
val newState = quickSettingsFragmentReducer(
state,
WebsitePermissionAction.TogglePermission(
@ -46,7 +44,7 @@ class QuickSettingsFragmentReducerTest {
val permissionPermission = WebsitePermission.Autoplay(
autoplayValue = AutoplayValue.BlockAll(
label = "label",
rules = mock(),
rules = createTestRule(),
sitePermission = null
),
options = emptyList(),
@ -55,11 +53,11 @@ class QuickSettingsFragmentReducerTest {
val map =
mapOf<PhoneFeature, WebsitePermission>(PhoneFeature.AUTOPLAY to permissionPermission)
val state = QuickSettingsFragmentState(mock(), map)
val infoState = WebsiteInfoState("", "", WebsiteSecurityUiValues.SECURE, "")
val state = QuickSettingsFragmentState(infoState, map)
val autoplayValue = AutoplayValue.AllowAll(
label = "newLabel",
rules = mock(),
rules = createTestRule(),
sitePermission = null
)
val newState = quickSettingsFragmentReducer(
@ -71,4 +69,15 @@ class QuickSettingsFragmentReducerTest {
newState.websitePermissionsState[PhoneFeature.AUTOPLAY] as WebsitePermission.Autoplay
assertEquals(autoplayValue, result.autoplayValue)
}
private fun createTestRule() = SitePermissionsRules(
SitePermissionsRules.Action.ALLOWED,
SitePermissionsRules.Action.ALLOWED,
SitePermissionsRules.Action.ALLOWED,
SitePermissionsRules.Action.ALLOWED,
SitePermissionsRules.AutoplayAction.ALLOWED,
SitePermissionsRules.AutoplayAction.ALLOWED,
SitePermissionsRules.Action.ALLOWED,
SitePermissionsRules.Action.ALLOWED,
)
}