Add debug logging to RunBefores and RunAfters.

This change adds Log.d statements to the evaluate methods of RunBefores and RunAfters to indicate when the execution of @before and @after methods starts and ends.

This allows us to know when setup and teardown sections start and end, and to isolate these in the logcat, to help error understanding.

PiperOrigin-RevId: 871896270

Author: copybara-androidxtest

runner/android_junit_runner/CHANGELOG.md

@@ -10,6 +10,8 @@
 
 * Make perfetto trace sections for tests more identifiable by prefixing with "test:" and using fully qualified class name. (b/204992764)
 
+* Add logs at the start and end of RunBefore and RunAfters sections to help bug understanding. (b/445754263)
+
 **Breaking Changes**
 
 **API Changes**

runner/android_junit_runner/java/androidx/test/internal/runner/junit4/statement/RunAfters.java

@@ -18,6 +18,7 @@
 
 package androidx.test.internal.runner.junit4.statement;
 
+import android.util.Log;
 import java.util.List;
 import java.util.concurrent.CopyOnWriteArrayList;
 import org.junit.runners.model.FrameworkMethod;
@@ -32,6 +33,9 @@ public class RunAfters extends UiThreadStatement {
 
   private final List<FrameworkMethod> afters;
 
+  /** Tag for logging. Set to TestRunner to match LogRunListener.TAG for easier debugging. */
+  private static final String TAG = "TestRunner";
+
   /**
    * Run all non-overridden {@code @After} methods on this class and superclasses before running
    * {@code next}; all After methods are always executed: exceptions thrown by previous steps are
@@ -62,6 +66,7 @@ public void evaluate() throws Throwable {
     } catch (Throwable e) {
       errors.add(e);
     } finally {
+      Log.d(TAG, "starting @After execution");
       for (final FrameworkMethod each : afters) {
         if (shouldRunOnUiThread(each)) {
           runOnUiThread(
@@ -83,6 +88,8 @@ public void run() {
           }
         }
       }
+
+      Log.d(TAG, "finished @After execution");
     }
     MultipleFailureException.assertEmpty(errors);
   }

runner/android_junit_runner/java/androidx/test/internal/runner/junit4/statement/RunBefores.java

@@ -18,6 +18,7 @@
 
 package androidx.test.internal.runner.junit4.statement;
 
+import android.util.Log;
 import java.util.List;
 import java.util.concurrent.atomic.AtomicReference;
 import org.junit.runners.model.FrameworkMethod;
@@ -31,6 +32,9 @@ public class RunBefores extends UiThreadStatement {
 
   private final List<FrameworkMethod> befores;
 
+  /** Tag for logging. Set to TestRunner to match LogRunListener.TAG for easier debugging. */
+  private static final String TAG = "TestRunner";
+
   /**
    * Run all non-overridden {@code @Before} methods on this class and superclasses before running
    * {@code next}; if any throws an Exception, stop execution and pass the exception on.
@@ -52,6 +56,7 @@ public RunBefores(
 
   @Override
   public void evaluate() throws Throwable {
+    Log.d(TAG, "starting @Before execution");
     final AtomicReference<Throwable> exceptionRef = new AtomicReference<>();
     for (final FrameworkMethod before : befores) {
       if (shouldRunOnUiThread(before)) {
@@ -77,6 +82,7 @@ public void run() {
       }
     }
 
+    Log.d(TAG, "finished @Before execution");
     next.evaluate();
   }
 }