From 1c26c757f0b4e9ef54209a1f5510ff924338e4d8 Mon Sep 17 00:00:00 2001 From: sowjanyakch Date: Fri, 12 Apr 2024 12:26:47 +0200 Subject: [PATCH] Refactor ShareUtils and remove ShareUtilsTest file Signed-off-by: sowjanyakch --- .../com/nextcloud/talk/utils/ShareUtils.kt | 11 --- app/src/main/res/values/strings.xml | 2 +- .../nextcloud/talk/utils/ShareUtilsTest.kt | 67 ------------------- 3 files changed, 1 insertion(+), 79 deletions(-) delete mode 100644 app/src/test/java/com/nextcloud/talk/utils/ShareUtilsTest.kt diff --git a/app/src/main/java/com/nextcloud/talk/utils/ShareUtils.kt b/app/src/main/java/com/nextcloud/talk/utils/ShareUtils.kt index 0bf08153d..619fde434 100644 --- a/app/src/main/java/com/nextcloud/talk/utils/ShareUtils.kt +++ b/app/src/main/java/com/nextcloud/talk/utils/ShareUtils.kt @@ -7,22 +7,11 @@ package com.nextcloud.talk.utils import android.app.Activity -import android.content.Context import android.content.Intent import android.net.Uri import com.nextcloud.talk.R -import com.nextcloud.talk.data.user.model.User -import com.nextcloud.talk.models.domain.ConversationModel object ShareUtils { - fun getStringForIntent(context: Context, user: User, conversation: ConversationModel?): String { - return String.format( - context.resources.getString(R.string.nc_share_link_text), - conversation?.name, - user.baseUrl, - conversation?.token - ) - } fun shareConversationLink(context: Activity, baseUrl: String?, roomToken: String?, conversationName: String?) { if (baseUrl.isNullOrBlank() || roomToken.isNullOrBlank() || conversationName.isNullOrBlank()) { diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index c800a1722..8f94031c5 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -331,7 +331,7 @@ How to translate with transifex: Add emojis %1$s invitation \nPassword: %1$s - Join the conversation %1$s at %2$s/index.php/call/%3$s + Push-to-talk With microphone disabled, click&hold to use Push-to-talk diff --git a/app/src/test/java/com/nextcloud/talk/utils/ShareUtilsTest.kt b/app/src/test/java/com/nextcloud/talk/utils/ShareUtilsTest.kt deleted file mode 100644 index cc3ed9b7d..000000000 --- a/app/src/test/java/com/nextcloud/talk/utils/ShareUtilsTest.kt +++ /dev/null @@ -1,67 +0,0 @@ -/* - * Nextcloud Talk - Android Client - * - * SPDX-FileCopyrightText: 2017-2019 Mario Danic - * SPDX-License-Identifier: GPL-3.0-or-later - */ -package com.nextcloud.talk.utils - -import android.content.Context -import android.content.res.Resources -import com.nextcloud.talk.R -import com.nextcloud.talk.data.user.model.User -import com.nextcloud.talk.models.domain.ConversationModel -import com.nextcloud.talk.users.UserManager -import io.reactivex.Maybe -import org.junit.Assert -import org.junit.Before -import org.junit.Test -import org.mockito.Mock -import org.mockito.Mockito -import org.mockito.MockitoAnnotations - -class ShareUtilsTest { - @Mock - private val context: Context? = null - - @Mock - private val resources: Resources? = null - - @Mock - private val userManager: UserManager? = null - - @Mock - private val user: User? = null - - private val baseUrl = "https://my.nextcloud.com" - private val token = "2aotbrjr" - - private lateinit var conversation: ConversationModel - - @Before - fun setUp() { - MockitoAnnotations.openMocks(this) - Mockito.`when`(userManager!!.currentUser).thenReturn(Maybe.just(user)) - Mockito.`when`(user!!.baseUrl).thenReturn(baseUrl) - Mockito.`when`(context!!.resources).thenReturn(resources) - Mockito.`when`(resources!!.getString(R.string.nc_share_text)) - .thenReturn("Join the conversation at %1\$s/index.php/call/%2\$s") - Mockito.`when`(resources.getString(R.string.nc_share_text_pass)).thenReturn("\nPassword: %1\$s") - - conversation = ConversationModel(token = token) - } - - @Test - fun stringForIntent_noPasswordGiven_correctStringWithoutPasswordReturned() { - val expectedResult = String.format( - "Join the conversation at %s/index.php/call/%s", - baseUrl, - token - ) - Assert.assertEquals( - "Intent string was not as expected", - expectedResult, - ShareUtils.getStringForIntent(context!!, userManager!!.currentUser.blockingGet(), conversation) - ) - } -}