[fenix] No issue: Correct TabsTrayAction naming

pull/600/head
Jonathan Almeida 3 years ago committed by mergify[bot]
parent 199547f94f
commit 7222c0220c

@ -134,7 +134,7 @@ sealed class TabsTrayAction : Action {
/** /**
* Removes the [TabsTrayState.focusGroupTabId] of the [TabsTrayState]. * Removes the [TabsTrayState.focusGroupTabId] of the [TabsTrayState].
*/ */
object ConsumeFocusGroupTabIdAction : TabsTrayAction() object ConsumeFocusGroupTabId : TabsTrayAction()
/** /**
* Updates the list of tabs in [TabsTrayState.inactiveTabs]. * Updates the list of tabs in [TabsTrayState.inactiveTabs].
@ -185,7 +185,7 @@ internal object TabsTrayReducer {
state.copy(syncing = true) state.copy(syncing = true)
is TabsTrayAction.SyncCompleted -> is TabsTrayAction.SyncCompleted ->
state.copy(syncing = false) state.copy(syncing = false)
is TabsTrayAction.ConsumeFocusGroupTabIdAction -> is TabsTrayAction.ConsumeFocusGroupTabId ->
state.copy(focusGroupTabId = null) state.copy(focusGroupTabId = null)
is TabsTrayAction.UpdateInactiveTabs -> is TabsTrayAction.UpdateInactiveTabs ->
state.copy(inactiveTabs = action.tabs) state.copy(inactiveTabs = action.tabs)

@ -135,7 +135,7 @@ class NormalBrowserPageViewHolder(
layoutManager.scrollToPosition(indexToScrollTo) layoutManager.scrollToPosition(indexToScrollTo)
if (focusGroupTabId != null) { if (focusGroupTabId != null) {
tabsTrayStore.dispatch(TabsTrayAction.ConsumeFocusGroupTabIdAction) tabsTrayStore.dispatch(TabsTrayAction.ConsumeFocusGroupTabId)
} }
return@observeFirstInsert return@observeFirstInsert
} }

@ -16,7 +16,7 @@ class TabsTrayStoreReducerTest {
val resultState = TabsTrayReducer.reduce( val resultState = TabsTrayReducer.reduce(
initialState, initialState,
TabsTrayAction.ConsumeFocusGroupTabIdAction TabsTrayAction.ConsumeFocusGroupTabId
) )
assertEquals(expectedState, resultState) assertEquals(expectedState, resultState)

Loading…
Cancel
Save