Patch plugin to use local execution by default for test debugging

Author: Brian Dong

base/src/com/google/idea/blaze/base/command/BlazeFlags.java

@@ -40,6 +40,9 @@ public final class BlazeFlags {
   // It expands to: --test_arg=--wrapper_script_flag=--debug --test_output=streamed
   //   --test_strategy=exclusive --test_timeout=9999 --nocache_test_results
   public static final String JAVA_TEST_DEBUG = "--java_debug";
+  // Debugging tests should only run locally, so make sure no remote executor is defined
+  // while debugging tests
+  public static final String RUN_LOCALLY = "--remote_executor=";
   // Streams stdout/stderr output from each test in real-time.
   // Implies --test_strategy=exclusive and --test_sharding_strategy=disabled
   public static final String TEST_OUTPUT_STREAMED = "--test_output=streamed";

java/src/com/google/idea/blaze/java/run/BlazeJavaRunProfileState.java

@@ -286,6 +286,7 @@ static BlazeCommand.Builder getBlazeCommandBuilder(
       }
       else {
         command.addBlazeFlags(BlazeFlags.JAVA_TEST_DEBUG);
+        command.addBlazeFlags(BlazeFlags.RUN_LOCALLY);
         command.addBlazeFlags(debugPortFlag(true, debugPort));
       }
       if (kotlinxCoroutinesJavaAgent != null) {

java/tests/unittests/com/google/idea/blaze/java/run/BlazeJavaRunProfileStateTest.java

@@ -236,6 +236,7 @@ public void debugFlagShouldBeIncludedForJavaTest() {
                 "command",
                 BlazeFlags.getToolTagFlag(),
                 "--java_debug",
+                "--remote_executor=",
                 "--test_arg=--wrapper_script_flag=--debug=127.0.0.1:5005",
                 "--",
                 "//label:rule"));
@@ -263,6 +264,7 @@ public void debugFlagShouldBeIncludedForJavaTestSuite() {
               "test",
               BlazeFlags.getToolTagFlag(),
               "--java_debug",
+              "--remote_executor=",
               "--test_arg=--wrapper_script_flag=--debug=127.0.0.1:5005",
               "--",
               "//label:java_test_suite_rule"));