diff --git a/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/AnalyticsOptInEvents.kt b/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/AnalyticsOptInEvent.kt similarity index 86% rename from features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/AnalyticsOptInEvents.kt rename to features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/AnalyticsOptInEvent.kt index 4181f52ba8..6095bc67da 100644 --- a/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/AnalyticsOptInEvents.kt +++ b/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/AnalyticsOptInEvent.kt @@ -8,6 +8,6 @@ package io.element.android.features.analytics.api -sealed interface AnalyticsOptInEvents { - data class EnableAnalytics(val isEnabled: Boolean) : AnalyticsOptInEvents +sealed interface AnalyticsOptInEvent { + data class EnableAnalytics(val isEnabled: Boolean) : AnalyticsOptInEvent } diff --git a/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesState.kt b/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesState.kt index 20a180446c..b5f049460d 100644 --- a/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesState.kt +++ b/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesState.kt @@ -8,11 +8,11 @@ package io.element.android.features.analytics.api.preferences -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent data class AnalyticsPreferencesState( val applicationName: String, val isEnabled: Boolean, val policyUrl: String, - val eventSink: (AnalyticsOptInEvents) -> Unit, + val eventSink: (AnalyticsOptInEvent) -> Unit, ) diff --git a/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesView.kt b/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesView.kt index 0d141e5ed2..31891a0162 100644 --- a/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesView.kt +++ b/features/analytics/api/src/main/kotlin/io/element/android/features/analytics/api/preferences/AnalyticsPreferencesView.kt @@ -13,7 +13,7 @@ import androidx.compose.runtime.Composable import androidx.compose.ui.Modifier import androidx.compose.ui.res.stringResource import androidx.compose.ui.tooling.preview.PreviewParameter -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent import io.element.android.features.analytics.api.R import io.element.android.libraries.designsystem.components.LINK_TAG import io.element.android.libraries.designsystem.components.list.ListItemContent @@ -30,7 +30,7 @@ fun AnalyticsPreferencesView( modifier: Modifier = Modifier, ) { fun onEnabledChanged(isEnabled: Boolean) { - state.eventSink(AnalyticsOptInEvents.EnableAnalytics(isEnabled = isEnabled)) + state.eventSink(AnalyticsOptInEvent.EnableAnalytics(isEnabled = isEnabled)) } val supportingText = stringResource( diff --git a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenter.kt b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenter.kt index d7bdb85e35..05d2c2fefd 100644 --- a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenter.kt +++ b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenter.kt @@ -12,7 +12,7 @@ import androidx.compose.runtime.Composable import androidx.compose.runtime.rememberCoroutineScope import dev.zacsweers.metro.Inject import io.element.android.appconfig.AnalyticsConfig -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent import io.element.android.libraries.architecture.Presenter import io.element.android.libraries.core.meta.BuildMeta import io.element.android.services.analytics.api.AnalyticsService @@ -28,9 +28,9 @@ class AnalyticsOptInPresenter( override fun present(): AnalyticsOptInState { val localCoroutineScope = rememberCoroutineScope() - fun handleEvent(event: AnalyticsOptInEvents) { + fun handleEvent(event: AnalyticsOptInEvent) { when (event) { - is AnalyticsOptInEvents.EnableAnalytics -> localCoroutineScope.setIsEnabled(event.isEnabled) + is AnalyticsOptInEvent.EnableAnalytics -> localCoroutineScope.setIsEnabled(event.isEnabled) } localCoroutineScope.launch { analyticsService.setDidAskUserConsent() diff --git a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInState.kt b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInState.kt index b2ebd37e20..063ed316c8 100644 --- a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInState.kt +++ b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInState.kt @@ -8,10 +8,10 @@ package io.element.android.features.analytics.impl -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent data class AnalyticsOptInState( val applicationName: String, val hasPolicyLink: Boolean, - val eventSink: (AnalyticsOptInEvents) -> Unit + val eventSink: (AnalyticsOptInEvent) -> Unit ) diff --git a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInView.kt b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInView.kt index 678d28c5c9..3ead26ea48 100644 --- a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInView.kt +++ b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInView.kt @@ -28,7 +28,7 @@ import androidx.compose.ui.unit.dp import io.element.android.appconfig.AnalyticsConfig import io.element.android.compound.theme.ElementTheme import io.element.android.compound.tokens.generated.CompoundIcons -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent import io.element.android.libraries.designsystem.atomic.molecules.ButtonColumnMolecule import io.element.android.libraries.designsystem.atomic.molecules.IconTitleSubtitleMolecule import io.element.android.libraries.designsystem.atomic.pages.HeaderFooterPage @@ -55,11 +55,11 @@ fun AnalyticsOptInView( val eventSink = state.eventSink fun onAcceptTerms() { - eventSink(AnalyticsOptInEvents.EnableAnalytics(true)) + eventSink(AnalyticsOptInEvent.EnableAnalytics(true)) } fun onDeclineTerms() { - eventSink(AnalyticsOptInEvents.EnableAnalytics(false)) + eventSink(AnalyticsOptInEvent.EnableAnalytics(false)) } BackHandler(onBack = ::onDeclineTerms) diff --git a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenter.kt b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenter.kt index 110cb98d41..ca3388ee3c 100644 --- a/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenter.kt +++ b/features/analytics/impl/src/main/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenter.kt @@ -13,7 +13,7 @@ import androidx.compose.runtime.collectAsState import androidx.compose.runtime.rememberCoroutineScope import dev.zacsweers.metro.Inject import io.element.android.appconfig.AnalyticsConfig -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent import io.element.android.features.analytics.api.preferences.AnalyticsPreferencesState import io.element.android.libraries.architecture.Presenter import io.element.android.libraries.core.meta.BuildMeta @@ -31,9 +31,9 @@ class AnalyticsPreferencesPresenter( val localCoroutineScope = rememberCoroutineScope() val isEnabled = analyticsService.userConsentFlow.collectAsState(initial = false) - fun handleEvent(event: AnalyticsOptInEvents) { + fun handleEvent(event: AnalyticsOptInEvent) { when (event) { - is AnalyticsOptInEvents.EnableAnalytics -> localCoroutineScope.setIsEnabled(event.isEnabled) + is AnalyticsOptInEvent.EnableAnalytics -> localCoroutineScope.setIsEnabled(event.isEnabled) } } diff --git a/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenterTest.kt b/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenterTest.kt index 6521c7f597..f2093dfa72 100644 --- a/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenterTest.kt +++ b/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/AnalyticsOptInPresenterTest.kt @@ -12,7 +12,7 @@ import app.cash.molecule.RecompositionMode import app.cash.molecule.moleculeFlow import app.cash.turbine.test import com.google.common.truth.Truth.assertThat -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent import io.element.android.libraries.matrix.test.core.aBuildMeta import io.element.android.services.analytics.test.FakeAnalyticsService import io.element.android.tests.testutils.WarmUpRule @@ -37,7 +37,7 @@ class AnalyticsOptInPresenterTest { }.test { val initialState = awaitItem() assertThat(analyticsService.didAskUserConsentFlow.first()).isFalse() - initialState.eventSink.invoke(AnalyticsOptInEvents.EnableAnalytics(true)) + initialState.eventSink.invoke(AnalyticsOptInEvent.EnableAnalytics(true)) assertThat(analyticsService.didAskUserConsentFlow.first()).isTrue() assertThat(analyticsService.userConsentFlow.first()).isTrue() } @@ -55,7 +55,7 @@ class AnalyticsOptInPresenterTest { }.test { val initialState = awaitItem() assertThat(analyticsService.didAskUserConsentFlow.first()).isFalse() - initialState.eventSink.invoke(AnalyticsOptInEvents.EnableAnalytics(false)) + initialState.eventSink.invoke(AnalyticsOptInEvent.EnableAnalytics(false)) assertThat(analyticsService.didAskUserConsentFlow.first()).isTrue() assertThat(analyticsService.userConsentFlow.first()).isFalse() } diff --git a/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenterTest.kt b/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenterTest.kt index 174935a1ad..29f7d9af03 100644 --- a/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenterTest.kt +++ b/features/analytics/impl/src/test/kotlin/io/element/android/features/analytics/impl/preferences/AnalyticsPreferencesPresenterTest.kt @@ -13,7 +13,7 @@ import app.cash.molecule.moleculeFlow import app.cash.turbine.test import com.google.common.truth.Truth.assertThat import io.element.android.appconfig.AnalyticsConfig -import io.element.android.features.analytics.api.AnalyticsOptInEvents +import io.element.android.features.analytics.api.AnalyticsOptInEvent import io.element.android.libraries.matrix.test.core.aBuildMeta import io.element.android.services.analytics.test.FakeAnalyticsService import io.element.android.tests.testutils.WarmUpRule @@ -67,9 +67,9 @@ class AnalyticsPreferencesPresenterTest { skipItems(1) val initialState = awaitItem() assertThat(initialState.isEnabled).isTrue() - initialState.eventSink.invoke(AnalyticsOptInEvents.EnableAnalytics(false)) + initialState.eventSink.invoke(AnalyticsOptInEvent.EnableAnalytics(false)) assertThat(awaitItem().isEnabled).isFalse() - initialState.eventSink.invoke(AnalyticsOptInEvents.EnableAnalytics(true)) + initialState.eventSink.invoke(AnalyticsOptInEvent.EnableAnalytics(true)) assertThat(awaitItem().isEnabled).isTrue() } }