Rename method storeData to addSession.

Author: bmarty

libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/auth/RustMatrixAuthenticationService.kt

@@ -158,7 +158,7 @@ class RustMatrixAuthenticationService(
                     )
                 val matrixClient = rustMatrixClientFactory.create(client)
                 newMatrixClientObservers.forEach { it.invoke(matrixClient) }
-                sessionStore.storeData(sessionData)
+                sessionStore.addSession(sessionData)
 
                 // Clean up the strong reference held here since it's no longer necessary
                 currentClient = null
@@ -182,7 +182,7 @@ class RustMatrixAuthenticationService(
                     sessionPaths = currentSessionPaths,
                 )
                 clear()
-                sessionStore.storeData(sessionData)
+                sessionStore.addSession(sessionData)
                 SessionId(sessionData.userId)
             }
         }
@@ -250,7 +250,7 @@ class RustMatrixAuthenticationService(
 
                 val matrixClient = rustMatrixClientFactory.create(client)
                 newMatrixClientObservers.forEach { it.invoke(matrixClient) }
-                sessionStore.storeData(sessionData)
+                sessionStore.addSession(sessionData)
 
                 // Clean up the strong reference held here since it's no longer necessary
                 currentClient = null
@@ -295,7 +295,7 @@ class RustMatrixAuthenticationService(
                     )
                 val matrixClient = rustMatrixClientFactory.create(client)
                 newMatrixClientObservers.forEach { it.invoke(matrixClient) }
-                sessionStore.storeData(sessionData)
+                sessionStore.addSession(sessionData)
 
                 // Clean up the strong reference held here since it's no longer necessary
                 currentClient = null

libraries/session-storage/api/src/main/kotlin/io/element/android/libraries/sessionstorage/api/SessionStore.kt

@@ -13,7 +13,7 @@ import kotlinx.coroutines.flow.map
 interface SessionStore {
     fun isLoggedIn(): Flow<LoggedInState>
     fun sessionsFlow(): Flow<List<SessionData>>
-    suspend fun storeData(sessionData: SessionData)
+    suspend fun addSession(sessionData: SessionData)
 
     /**
      * Will update the session data matching the userId, except the value of loginTimestamp.

libraries/session-storage/impl/src/main/kotlin/io/element/android/libraries/sessionstorage/impl/DatabaseSessionStore.kt

@@ -49,7 +49,7 @@ class DatabaseSessionStore(
             }
     }
 
-    override suspend fun storeData(sessionData: SessionData) {
+    override suspend fun addSession(sessionData: SessionData) {
         sessionDataMutex.withLock {
             database.sessionDataQueries.insertSessionData(sessionData.toDbModel())
         }

libraries/session-storage/impl/src/test/kotlin/io/element/android/libraries/sessionstorage/impl/DatabaseSessionStoreTest.kt

@@ -44,10 +44,10 @@ class DatabaseSessionStoreTest {
     }
 
     @Test
-    fun `storeData persists the SessionData into the DB`() = runTest {
+    fun `addSession persists the SessionData into the DB`() = runTest {
         assertThat(database.sessionDataQueries.selectFirst().executeAsOneOrNull()).isNull()
 
-        databaseSessionStore.storeData(aSessionData.toApiModel())
+        databaseSessionStore.addSession(aSessionData.toApiModel())
 
         assertThat(database.sessionDataQueries.selectFirst().executeAsOneOrNull()).isEqualTo(aSessionData)
         assertThat(database.sessionDataQueries.selectAll().executeAsList().size).isEqualTo(1)

libraries/session-storage/impl/src/test/kotlin/io/element/android/libraries/sessionstorage/impl/observer/DefaultSessionObserverTest.kt

@@ -51,7 +51,7 @@ import org.junit.Test
         runCurrent()
         val listener = TestSessionListener()
         sut.addListener(listener)
-        databaseSessionStore.storeData(sessionData.toApiModel())
+        databaseSessionStore.addSession(sessionData.toApiModel())
         listener.assertEvents(TestSessionListener.Event.Created(sessionData.userId))
         sut.removeListener(listener)
         coroutineContext.cancelChildren()
@@ -64,7 +64,7 @@ import org.junit.Test
         runCurrent()
         val listener = TestSessionListener()
         sut.addListener(listener)
-        databaseSessionStore.storeData(sessionData.toApiModel())
+        databaseSessionStore.addSession(sessionData.toApiModel())
         listener.assertEvents(TestSessionListener.Event.Created(sessionData.userId))
         databaseSessionStore.removeSession(sessionData.userId)
         listener.assertEvents(

libraries/session-storage/test/src/main/kotlin/io/element/android/libraries/sessionstorage/test/InMemorySessionStore.kt

@@ -37,7 +37,7 @@ class InMemorySessionStore(
 
     override fun sessionsFlow(): Flow<List<SessionData>> = sessionDataListFlow.asStateFlow()
 
-    override suspend fun storeData(sessionData: SessionData) {
+    override suspend fun addSession(sessionData: SessionData) {
         val currentList = sessionDataListFlow.value.toMutableList()
         currentList.removeAll { it.userId == sessionData.userId }
         currentList.add(sessionData)