From a09f62d036c55fb7963d4f545bbcd00efb567a5a Mon Sep 17 00:00:00 2001 From: Andrey Epin Date: Tue, 13 Jun 2023 10:45:20 -0700 Subject: Provide copy action only when a text is sent The copy button will be present to the user only when a text without images is sent i.e. intent action is ACTION_SEND, EXTRA_STREAM does not have any URI and EXTRA_TEXT is not null. Fix: 287061873 Fix: 275382122 Fix: 288425116 Test: manual testing Test: atest ChooserActionFactoryTest (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:a1c9dc6af9fb0ff4b50ed1b8e7f59bdfccbfa682) Merged-In: I26c6eef24e7f909842eb066a73286a7525124d21 Change-Id: I26c6eef24e7f909842eb066a73286a7525124d21 --- .../intentresolver/ChooserActionFactoryTest.kt | 152 +++++++++++++++------ 1 file changed, 113 insertions(+), 39 deletions(-) (limited to 'java/tests/src') diff --git a/java/tests/src/com/android/intentresolver/ChooserActionFactoryTest.kt b/java/tests/src/com/android/intentresolver/ChooserActionFactoryTest.kt index d72c9aa6..8d994f08 100644 --- a/java/tests/src/com/android/intentresolver/ChooserActionFactoryTest.kt +++ b/java/tests/src/com/android/intentresolver/ChooserActionFactoryTest.kt @@ -25,48 +25,45 @@ import android.content.IntentFilter import android.content.res.Resources import android.graphics.drawable.Icon import android.service.chooser.ChooserAction -import android.view.View import androidx.test.ext.junit.runners.AndroidJUnit4 import androidx.test.platform.app.InstrumentationRegistry -import com.android.intentresolver.flags.FeatureFlagRepository import com.google.common.collect.ImmutableList import com.google.common.truth.Truth.assertThat +import java.util.concurrent.CountDownLatch +import java.util.concurrent.TimeUnit +import java.util.function.Consumer import org.junit.After import org.junit.Assert.assertEquals import org.junit.Before import org.junit.Test import org.junit.runner.RunWith import org.mockito.Mockito -import java.util.concurrent.Callable -import java.util.concurrent.CountDownLatch -import java.util.concurrent.TimeUnit -import java.util.function.Consumer @RunWith(AndroidJUnit4::class) class ChooserActionFactoryTest { private val context = InstrumentationRegistry.getInstrumentation().getContext() private val logger = mock() - private val flags = mock() private val actionLabel = "Action label" private val modifyShareLabel = "Modify share" private val testAction = "com.android.intentresolver.testaction" private val countdown = CountDownLatch(1) - private val testReceiver: BroadcastReceiver = object : BroadcastReceiver() { - override fun onReceive(context: Context, intent: Intent) { - // Just doing at most a single countdown per test. - countdown.countDown() + private val testReceiver: BroadcastReceiver = + object : BroadcastReceiver() { + override fun onReceive(context: Context, intent: Intent) { + // Just doing at most a single countdown per test. + countdown.countDown() + } } - } - private object resultConsumer : Consumer { - var latestReturn = Integer.MIN_VALUE + private val resultConsumer = + object : Consumer { + var latestReturn = Integer.MIN_VALUE - override fun accept(resultCode: Int) { - latestReturn = resultCode + override fun accept(resultCode: Int) { + latestReturn = resultCode + } } - } - @Before fun setup() { context.registerReceiver(testReceiver, IntentFilter(testAction)) @@ -91,7 +88,7 @@ class ChooserActionFactoryTest { Mockito.verify(logger).logCustomActionSelected(eq(0)) assertEquals(Activity.RESULT_OK, resultConsumer.latestReturn) - // Verify the pendingintent has been called + // Verify the pending intent has been called countdown.await(500, TimeUnit.MILLISECONDS) } @@ -109,42 +106,119 @@ class ChooserActionFactoryTest { val action = factory.modifyShareAction ?: error("Modify share action should not be null") action.onClicked.run() - Mockito.verify(logger).logActionSelected( - eq(ChooserActivityLogger.SELECTION_TYPE_MODIFY_SHARE)) + Mockito.verify(logger) + .logActionSelected(eq(ChooserActivityLogger.SELECTION_TYPE_MODIFY_SHARE)) assertEquals(Activity.RESULT_OK, resultConsumer.latestReturn) - // Verify the pendingintent has been called + // Verify the pending intent has been called countdown.await(500, TimeUnit.MILLISECONDS) } + @Test + fun nonSendAction_noCopyRunnable() { + val targetIntent = + Intent(Intent.ACTION_SEND_MULTIPLE).apply { + putExtra(Intent.EXTRA_TEXT, "Text to show") + } + + val chooserRequest = + mock { + whenever(this.targetIntent).thenReturn(targetIntent) + whenever(chooserActions).thenReturn(ImmutableList.of()) + } + val testSubject = + ChooserActionFactory( + context, + chooserRequest, + mock(), + logger, + {}, + { null }, + mock(), + {}, + ) + assertThat(testSubject.copyButtonRunnable).isNull() + } + + @Test + fun sendActionNoText_noCopyRunnable() { + val targetIntent = Intent(Intent.ACTION_SEND) + + val chooserRequest = + mock { + whenever(this.targetIntent).thenReturn(targetIntent) + whenever(chooserActions).thenReturn(ImmutableList.of()) + } + val testSubject = + ChooserActionFactory( + context, + chooserRequest, + mock(), + logger, + {}, + { null }, + mock(), + {}, + ) + assertThat(testSubject.copyButtonRunnable).isNull() + } + + @Test + fun sendActionWithText_nonNullCopyRunnable() { + val targetIntent = Intent(Intent.ACTION_SEND).apply { putExtra(Intent.EXTRA_TEXT, "Text") } + + val chooserRequest = + mock { + whenever(this.targetIntent).thenReturn(targetIntent) + whenever(chooserActions).thenReturn(ImmutableList.of()) + } + val testSubject = + ChooserActionFactory( + context, + chooserRequest, + mock(), + logger, + {}, + { null }, + mock(), + {}, + ) + assertThat(testSubject.copyButtonRunnable).isNotNull() + } + private fun createFactory(includeModifyShare: Boolean = false): ChooserActionFactory { - val testPendingIntent = PendingIntent.getActivity(context, 0, Intent(testAction),0) + val testPendingIntent = PendingIntent.getActivity(context, 0, Intent(testAction), 0) val targetIntent = Intent() - val action = ChooserAction.Builder( - Icon.createWithResource("", Resources.ID_NULL), - actionLabel, - testPendingIntent - ).build() + val action = + ChooserAction.Builder( + Icon.createWithResource("", Resources.ID_NULL), + actionLabel, + testPendingIntent + ) + .build() val chooserRequest = mock() whenever(chooserRequest.targetIntent).thenReturn(targetIntent) whenever(chooserRequest.chooserActions).thenReturn(ImmutableList.of(action)) if (includeModifyShare) { - val modifyShare = ChooserAction.Builder( - Icon.createWithResource("", Resources.ID_NULL), - modifyShareLabel, - testPendingIntent - ).build() + val modifyShare = + ChooserAction.Builder( + Icon.createWithResource("", Resources.ID_NULL), + modifyShareLabel, + testPendingIntent + ) + .build() whenever(chooserRequest.modifyShareAction).thenReturn(modifyShare) } return ChooserActionFactory( context, chooserRequest, - mock(), + mock(), logger, - Consumer{}, - Callable{null}, - mock(), - resultConsumer) + {}, + { null }, + mock(), + resultConsumer + ) } -} \ No newline at end of file +} -- cgit v1.2.3-59-g8ed1b