Do not expose ChatClient.markMessagesAsDelivered

Author: andremion

stream-chat-android-client/api/stream-chat-android-client.api

@@ -114,7 +114,6 @@ public final class io/getstream/chat/android/client/ChatClient {
 	public static synthetic fun keystroke$default (Lio/getstream/chat/android/client/ChatClient;Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;ILjava/lang/Object;)Lio/getstream/result/call/Call;
 	public final fun markAllRead ()Lio/getstream/result/call/Call;
 	public final fun markMessageRead (Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;)Lio/getstream/result/call/Call;
-	public final fun markMessagesAsDelivered (Ljava/util/List;)Lio/getstream/result/call/Call;
 	public final fun markRead (Ljava/lang/String;Ljava/lang/String;)Lio/getstream/result/call/Call;
 	public final fun markThreadRead (Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;)Lio/getstream/result/call/Call;
 	public final fun markThreadUnread (Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;)Lio/getstream/result/call/Call;

stream-chat-android-client/src/main/java/io/getstream/chat/android/client/ChatClient.kt

@@ -2911,21 +2911,6 @@ internal constructor(
             }
     }
 
-    /**
-     * Marks the given messages as delivered.
-     *
-     * @param messages The list of messages to mark as delivered.
-     */
-    @CheckResult
-    public fun markMessagesAsDelivered(messages: List<Message>): Call<Unit> =
-        api.markDelivered(messages)
-            .doOnStart(userScope) {
-                logger.d { "[markMessagesAsDelivered] #doOnStart; messages: ${messages.size}" }
-            }
-            .doOnResult(userScope) { result ->
-                logger.v { "[markMessagesAsDelivered] #doOnResult; completed: $result" }
-            }
-
     /**
      * Marks a given thread as read.
      *

stream-chat-android-client/src/test/java/io/getstream/chat/android/client/ChatClientChannelApiTests.kt

@@ -44,7 +44,6 @@ import io.getstream.chat.android.randomExtraData
 import io.getstream.chat.android.randomInt
 import io.getstream.chat.android.randomMember
 import io.getstream.chat.android.randomMessage
-import io.getstream.chat.android.randomMessageList
 import io.getstream.chat.android.randomString
 import io.getstream.chat.android.randomUser
 import io.getstream.result.Error
@@ -988,33 +987,6 @@ internal class ChatClientChannelApiTests : BaseChatClientTest() {
         verifyNetworkError(result, errorCode)
     }
 
-    @Test
-    fun markMessagesAsDeliveredSuccess() = runTest {
-        // given
-        val messages = randomMessageList(10)
-        val sut = Fixture()
-            .givenMarkDeliveredResult(RetroSuccess(Unit).toRetrofitCall())
-            .get()
-        // when
-        val result = sut.markMessagesAsDelivered(messages).await()
-        // then
-        verifySuccess(result, Unit)
-    }
-
-    @Test
-    fun markMessagesAsDeliveredError() = runTest {
-        // given
-        val messages = randomMessageList(10)
-        val errorCode = positiveRandomInt()
-        val sut = Fixture()
-            .givenMarkDeliveredResult(RetroError<Unit>(errorCode).toRetrofitCall())
-            .get()
-        // when
-        val result = sut.markMessagesAsDelivered(messages).await()
-        // then
-        verifyNetworkError(result, errorCode)
-    }
-
     @Test
     fun markReadSuccess() = runTest {
         // given