Introduce Announcement enum.

Author: bmarty

features/announcement/api/src/main/kotlin/io/element/android/features/announcement/api/Announcement.kt

@@ -0,0 +1,12 @@
+/*
+ * Copyright 2025 New Vector Ltd.
+ *
+ * SPDX-License-Identifier: AGPL-3.0-only OR LicenseRef-Element-Commercial
+ * Please see LICENSE files in the repository root for full details.
+ */
+
+package io.element.android.features.announcement.api
+
+enum class Announcement {
+    Space,
+}

features/announcement/api/src/main/kotlin/io/element/android/features/announcement/api/AnnouncementService.kt

@@ -11,7 +11,7 @@ import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
 
 interface AnnouncementService {
-    suspend fun onEnteringSpaceTab()
+    suspend fun showAnnouncement(announcement: Announcement)
 
     @Composable
     fun Render(

features/announcement/impl/src/main/kotlin/io/element/android/features/announcement/impl/DefaultAnnouncementService.kt

@@ -17,6 +17,7 @@ import androidx.compose.ui.Modifier
 import dev.zacsweers.metro.AppScope
 import dev.zacsweers.metro.ContributesBinding
 import dev.zacsweers.metro.Inject
+import io.element.android.features.announcement.api.Announcement
 import io.element.android.features.announcement.api.AnnouncementService
 import io.element.android.features.announcement.impl.spaces.SpaceAnnouncementState
 import io.element.android.features.announcement.impl.spaces.SpaceAnnouncementView
@@ -31,7 +32,13 @@ class DefaultAnnouncementService(
     private val announcementPresenter: Presenter<AnnouncementState>,
     private val spaceAnnouncementPresenter: Presenter<SpaceAnnouncementState>,
 ) : AnnouncementService {
-    override suspend fun onEnteringSpaceTab() {
+    override suspend fun showAnnouncement(announcement: Announcement) {
+        when (announcement) {
+            Announcement.Space -> showSpaceAnnouncement()
+        }
+    }
+
+    private suspend fun showSpaceAnnouncement() {
         val currentValue = announcementStore.spaceAnnouncementFlow().first()
         if (currentValue == AnnouncementStore.SpaceAnnouncement.NeverShown) {
             announcementStore.setSpaceAnnouncementValue(AnnouncementStore.SpaceAnnouncement.Show)

features/announcement/impl/src/test/kotlin/io/element/android/features/announcement/impl/DefaultAnnouncementServiceTest.kt

@@ -8,6 +8,7 @@
 package io.element.android.features.announcement.impl
 
 import com.google.common.truth.Truth.assertThat
+import io.element.android.features.announcement.api.Announcement
 import io.element.android.features.announcement.impl.spaces.SpaceAnnouncementState
 import io.element.android.features.announcement.impl.spaces.aSpaceAnnouncementState
 import io.element.android.features.announcement.impl.store.AnnouncementStore
@@ -19,18 +20,18 @@ import org.junit.Test
 
 class DefaultAnnouncementServiceTest {
     @Test
-    fun `when entering space tab, space announcement is set to show only if it was never shown`() = runTest {
+    fun `when showing Space announcement, space announcement is set to show only if it was never shown`() = runTest {
         val announcementStore = InMemoryAnnouncementStore()
         val sut = createDefaultAnnouncementService(
             announcementStore = announcementStore,
         )
         assertThat(announcementStore.spaceAnnouncementFlow().first()).isEqualTo(AnnouncementStore.SpaceAnnouncement.NeverShown)
-        sut.onEnteringSpaceTab()
+        sut.showAnnouncement(Announcement.Space)
         assertThat(announcementStore.spaceAnnouncementFlow().first()).isEqualTo(AnnouncementStore.SpaceAnnouncement.Show)
         // Simulate user close the announcement
         announcementStore.setSpaceAnnouncementValue(AnnouncementStore.SpaceAnnouncement.Shown)
         // Entering again the space tab should not change the value
-        sut.onEnteringSpaceTab()
+        sut.showAnnouncement(Announcement.Space)
         assertThat(announcementStore.spaceAnnouncementFlow().first()).isEqualTo(AnnouncementStore.SpaceAnnouncement.Shown)
     }
 

features/announcement/test/src/main/kotlin/io/element/android/features/rageshake/test/logs/FakeAnnouncementService.kt

@@ -9,15 +9,16 @@ package io.element.android.features.rageshake.test.logs
 
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
+import io.element.android.features.announcement.api.Announcement
 import io.element.android.features.announcement.api.AnnouncementService
 import io.element.android.tests.testutils.lambda.lambdaError
 
 class FakeAnnouncementService(
-    val onEnteringSpaceTabResult: () -> Unit = { lambdaError() },
+    val showAnnouncementResult: (Announcement) -> Unit = { lambdaError() },
     val renderResult: (Modifier) -> Unit = { lambdaError() },
 ) : AnnouncementService {
-    override suspend fun onEnteringSpaceTab() {
-        onEnteringSpaceTabResult()
+    override suspend fun showAnnouncement(announcement: Announcement) {
+        showAnnouncementResult(announcement)
     }
 
     @Composable

features/home/impl/src/main/kotlin/io/element/android/features/home/impl/HomePresenter.kt

@@ -18,6 +18,7 @@ import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.saveable.rememberSaveable
 import androidx.compose.runtime.setValue
 import dev.zacsweers.metro.Inject
+import io.element.android.features.announcement.api.Announcement
 import io.element.android.features.announcement.api.AnnouncementService
 import io.element.android.features.home.impl.roomlist.RoomListState
 import io.element.android.features.home.impl.spaces.HomeSpacesState
@@ -88,7 +89,7 @@ class HomePresenter(
             when (event) {
                 is HomeEvents.SelectHomeNavigationBarItem -> coroutineState.launch {
                     if (event.item == HomeNavigationBarItem.Spaces) {
-                        announcementService.onEnteringSpaceTab()
+                        announcementService.showAnnouncement(Announcement.Space)
                     }
                     currentHomeNavigationBarItemOrdinal = event.item.ordinal
                 }

features/home/impl/src/test/kotlin/io/element/android/features/home/impl/HomePresenterTest.kt

@@ -11,6 +11,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.announcement.api.Announcement
 import io.element.android.features.announcement.api.AnnouncementService
 import io.element.android.features.home.impl.roomlist.aRoomListState
 import io.element.android.features.home.impl.spaces.HomeSpacesState
@@ -40,6 +41,7 @@ import io.element.android.libraries.sessionstorage.test.aSessionData
 import io.element.android.tests.testutils.MutablePresenter
 import io.element.android.tests.testutils.WarmUpRule
 import io.element.android.tests.testutils.lambda.lambdaRecorder
+import io.element.android.tests.testutils.lambda.value
 import io.element.android.tests.testutils.test
 import kotlinx.coroutines.flow.flowOf
 import kotlinx.coroutines.test.runTest
@@ -168,13 +170,13 @@ class HomePresenterTest {
 
     @Test
     fun `present - NavigationBar change`() = runTest {
-        val onEnteringSpaceTabResult = lambdaRecorder<Unit> { }
+        val showAnnouncementResult = lambdaRecorder<Announcement, Unit> { }
         val presenter = createHomePresenter(
             sessionStore = InMemorySessionStore(
                 updateUserProfileResult = { _, _, _ -> },
             ),
             announcementService = FakeAnnouncementService(
-                onEnteringSpaceTabResult = onEnteringSpaceTabResult,
+                showAnnouncementResult = showAnnouncementResult,
             )
         )
         moleculeFlow(RecompositionMode.Immediate) {
@@ -185,7 +187,8 @@ class HomePresenterTest {
             initialState.eventSink(HomeEvents.SelectHomeNavigationBarItem(HomeNavigationBarItem.Spaces))
             val finalState = awaitItem()
             assertThat(finalState.currentHomeNavigationBarItem).isEqualTo(HomeNavigationBarItem.Spaces)
-            onEnteringSpaceTabResult.assertions().isCalledOnce()
+            showAnnouncementResult.assertions().isCalledOnce()
+                .with(value(Announcement.Space))
         }
     }
 
@@ -201,7 +204,7 @@ class HomePresenterTest {
             ),
             homeSpacesPresenter = homeSpacesPresenter,
             announcementService = FakeAnnouncementService(
-                onEnteringSpaceTabResult = {},
+                showAnnouncementResult = {},
             )
         )
         presenter.test {