Stop posting unnecessary work to threads in espresso.

Currently Espresso will post no-op empty runnables to a background thread pool for every interaction
when running Espresso in a normal, non-remote environment.

This is wasteful and causes some noise for leaked-thread detector facilities.

This commit addresses this by adding an extra API to RemoteInteraction, so it only posts the remote
commumnication runnable if its running in remote mode.

PiperOrigin-RevId: 600526081

Author: brettchabot

espresso/CHANGELOG.md

@@ -21,6 +21,7 @@ The following artifacts were released:
 * Reference doc cleanup - document previously missing parameters, fix links, etc
 * Remove Kotlin StringKt calls from Java code
 * Remove all support for Android SDKs < 19. Minimum is API 19 (Android Kit Kat 4.4)
+* Stop posting empty tasks to background threads when running in non-remote mode
 
 **New Features**
 

espresso/core/java/androidx/test/espresso/ViewInteraction.java

@@ -198,7 +198,7 @@ public Void call() {
 
     List<ListenableFuture<Void>> interactions = new ArrayList<>();
     interactions.add(postAsynchronouslyOnUiThread(performInteraction));
-    if (!remoteInteraction.isRemoteProcess()) {
+    if (!remoteInteraction.isRemoteProcess() && remoteInteraction.isRemoteClient()) {
       // Only the original process should submit remote interactionsList;
       interactions.add(
           remoteExecutor.submit(
@@ -352,7 +352,7 @@ public Void call() {
 
     List<ListenableFuture<Void>> interactions = new ArrayList<>();
     interactions.add(postAsynchronouslyOnUiThread(checkInteraction));
-    if (!remoteInteraction.isRemoteProcess()) {
+    if (!remoteInteraction.isRemoteProcess() && remoteInteraction.isRemoteClient()) {
       // Only the original process should submit remote interactionsList;
       interactions.add(
           remoteExecutor.submit(

espresso/core/java/androidx/test/espresso/api/current_public.txt

@@ -788,6 +788,7 @@ package androidx.test.espresso.remote {
   public interface RemoteInteraction {
     method public java.util.concurrent.Callable<java.lang.Void!>! createRemoteCheckCallable(org.hamcrest.Matcher<androidx.test.espresso.Root!>!, org.hamcrest.Matcher<android.view.View!>!, java.util.Map<java.lang.String!,android.os.IBinder!>!, androidx.test.espresso.ViewAssertion!);
     method public java.util.concurrent.Callable<java.lang.Void!>! createRemotePerformCallable(org.hamcrest.Matcher<androidx.test.espresso.Root!>!, org.hamcrest.Matcher<android.view.View!>!, java.util.Map<java.lang.String!,android.os.IBinder!>!, androidx.test.espresso.ViewAction!...);
+    method public default boolean isRemoteClient();
     method public boolean isRemoteProcess();
     field public static final String BUNDLE_EXECUTION_STATUS = "executionStatus";
   }

espresso/core/java/androidx/test/espresso/remote/RemoteInteraction.java

@@ -33,9 +33,17 @@
 public interface RemoteInteraction {
   static final String BUNDLE_EXECUTION_STATUS = "executionStatus";
 
-  /** @return {@code true} if the current Espresso instance running in a remote process. */
+  /** Returns {@code true} if the current Espresso instance running in a remote process. */
   boolean isRemoteProcess();
 
+  /**
+   * Returns {code true} if the current Espresso instance is communicating with another Espresso
+   * instance in a remote process.
+   */
+  default boolean isRemoteClient() {
+    return false;
+  }
+
   /**
    * Creates a callable to run Espresso check interaction on remote processes
    *

espresso/core/javatests/androidx/test/espresso/ViewInteractionTest.java

@@ -436,6 +436,7 @@ public Void call() throws Exception {
     when(mockViewFinder.getView()).thenThrow(noActivityResumed);
     // enable remote interaction
     when(mockRemoteInteraction.isRemoteProcess()).thenReturn(false);
+    when(mockRemoteInteraction.isRemoteClient()).thenReturn(true);
     // noinspection unchecked
     when(mockRemoteInteraction.createRemoteCheckCallable(
             ArgumentMatchers.any(),
@@ -624,6 +625,7 @@ private void initWithRemoteInteraction() {
 
     // enable remote interaction
     when(mockRemoteInteraction.isRemoteProcess()).thenReturn(false);
+    when(mockRemoteInteraction.isRemoteClient()).thenReturn(true);
   }
 
   private static final class BindableViewAction implements ViewAction, Bindable {

espresso/remote/java/androidx/test/espresso/remote/EspressoRemote.java

@@ -173,6 +173,11 @@ public synchronized boolean isRemoteProcess() {
     return isRemoteProcess;
   }
 
+  @Override
+  public boolean isRemoteClient() {
+    return !isRemoteProcess();
+  }
+
   /**
    * Creates a callable to run Espresso check interaction on remote processes
    *