Rename Event for clarity
This commit is contained in:
parent
f24036d688
commit
f31c91a417
5 changed files with 7 additions and 7 deletions
|
|
@ -19,7 +19,7 @@ package io.element.android.features.roomdetails.impl.notificationsettings
|
||||||
import io.element.android.libraries.matrix.api.room.RoomNotificationMode
|
import io.element.android.libraries.matrix.api.room.RoomNotificationMode
|
||||||
|
|
||||||
sealed interface RoomNotificationSettingsEvents {
|
sealed interface RoomNotificationSettingsEvents {
|
||||||
data class RoomNotificationModeChanged(val mode: RoomNotificationMode) : RoomNotificationSettingsEvents
|
data class ChangeRoomNotificationMode(val mode: RoomNotificationMode) : RoomNotificationSettingsEvents
|
||||||
data class SetNotificationMode(val isDefault: Boolean) : RoomNotificationSettingsEvents
|
data class SetNotificationMode(val isDefault: Boolean) : RoomNotificationSettingsEvents
|
||||||
data object DeleteCustomNotification : RoomNotificationSettingsEvents
|
data object DeleteCustomNotification : RoomNotificationSettingsEvents
|
||||||
data object ClearSetNotificationError : RoomNotificationSettingsEvents
|
data object ClearSetNotificationError : RoomNotificationSettingsEvents
|
||||||
|
|
|
||||||
|
|
@ -95,7 +95,7 @@ class RoomNotificationSettingsPresenter @AssistedInject constructor(
|
||||||
|
|
||||||
fun handleEvents(event: RoomNotificationSettingsEvents) {
|
fun handleEvents(event: RoomNotificationSettingsEvents) {
|
||||||
when (event) {
|
when (event) {
|
||||||
is RoomNotificationSettingsEvents.RoomNotificationModeChanged -> {
|
is RoomNotificationSettingsEvents.ChangeRoomNotificationMode -> {
|
||||||
localCoroutineScope.setRoomNotificationMode(event.mode, pendingRoomNotificationMode, pendingSetDefault, setNotificationSettingAction)
|
localCoroutineScope.setRoomNotificationMode(event.mode, pendingRoomNotificationMode, pendingSetDefault, setNotificationSettingAction)
|
||||||
}
|
}
|
||||||
is RoomNotificationSettingsEvents.SetNotificationMode -> {
|
is RoomNotificationSettingsEvents.SetNotificationMode -> {
|
||||||
|
|
|
||||||
|
|
@ -149,7 +149,7 @@ private fun RoomSpecificNotificationSettingsView(
|
||||||
enabled = !state.displayIsDefault.orTrue(),
|
enabled = !state.displayIsDefault.orTrue(),
|
||||||
displayMentionsOnlyDisclaimer = state.displayMentionsOnlyDisclaimer,
|
displayMentionsOnlyDisclaimer = state.displayMentionsOnlyDisclaimer,
|
||||||
onSelectOption = {
|
onSelectOption = {
|
||||||
state.eventSink(RoomNotificationSettingsEvents.RoomNotificationModeChanged(it.mode))
|
state.eventSink(RoomNotificationSettingsEvents.ChangeRoomNotificationMode(it.mode))
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -68,7 +68,7 @@ fun UserDefinedRoomNotificationSettingsView(
|
||||||
enabled = !state.displayIsDefault.orTrue(),
|
enabled = !state.displayIsDefault.orTrue(),
|
||||||
displayMentionsOnlyDisclaimer = state.displayMentionsOnlyDisclaimer,
|
displayMentionsOnlyDisclaimer = state.displayMentionsOnlyDisclaimer,
|
||||||
onSelectOption = {
|
onSelectOption = {
|
||||||
state.eventSink(RoomNotificationSettingsEvents.RoomNotificationModeChanged(it.mode))
|
state.eventSink(RoomNotificationSettingsEvents.ChangeRoomNotificationMode(it.mode))
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -55,7 +55,7 @@ class RoomNotificationSettingsPresenterTest {
|
||||||
moleculeFlow(RecompositionMode.Immediate) {
|
moleculeFlow(RecompositionMode.Immediate) {
|
||||||
presenter.present()
|
presenter.present()
|
||||||
}.test {
|
}.test {
|
||||||
awaitItem().eventSink(RoomNotificationSettingsEvents.RoomNotificationModeChanged(RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY))
|
awaitItem().eventSink(RoomNotificationSettingsEvents.ChangeRoomNotificationMode(RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY))
|
||||||
val updatedState = consumeItemsUntilPredicate {
|
val updatedState = consumeItemsUntilPredicate {
|
||||||
it.roomNotificationSettings.dataOrNull()?.mode == RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY
|
it.roomNotificationSettings.dataOrNull()?.mode == RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY
|
||||||
}.last()
|
}.last()
|
||||||
|
|
@ -129,7 +129,7 @@ class RoomNotificationSettingsPresenterTest {
|
||||||
presenter.present()
|
presenter.present()
|
||||||
}.test {
|
}.test {
|
||||||
val initialState = awaitItem()
|
val initialState = awaitItem()
|
||||||
initialState.eventSink(RoomNotificationSettingsEvents.RoomNotificationModeChanged(RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY))
|
initialState.eventSink(RoomNotificationSettingsEvents.ChangeRoomNotificationMode(RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY))
|
||||||
initialState.eventSink(RoomNotificationSettingsEvents.SetNotificationMode(true))
|
initialState.eventSink(RoomNotificationSettingsEvents.SetNotificationMode(true))
|
||||||
val defaultState = consumeItemsUntilPredicate {
|
val defaultState = consumeItemsUntilPredicate {
|
||||||
it.roomNotificationSettings.dataOrNull()?.mode == RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY
|
it.roomNotificationSettings.dataOrNull()?.mode == RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY
|
||||||
|
|
@ -148,7 +148,7 @@ class RoomNotificationSettingsPresenterTest {
|
||||||
presenter.present()
|
presenter.present()
|
||||||
}.test {
|
}.test {
|
||||||
val initialState = awaitItem()
|
val initialState = awaitItem()
|
||||||
initialState.eventSink(RoomNotificationSettingsEvents.RoomNotificationModeChanged(RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY))
|
initialState.eventSink(RoomNotificationSettingsEvents.ChangeRoomNotificationMode(RoomNotificationMode.MENTIONS_AND_KEYWORDS_ONLY))
|
||||||
initialState.eventSink(RoomNotificationSettingsEvents.SetNotificationMode(true))
|
initialState.eventSink(RoomNotificationSettingsEvents.SetNotificationMode(true))
|
||||||
val failedState = consumeItemsUntilPredicate {
|
val failedState = consumeItemsUntilPredicate {
|
||||||
it.restoreDefaultAction.isFailure()
|
it.restoreDefaultAction.isFailure()
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue