Change the default value for test_output to "errors".

This might be controversial, but I have many times seen users run their tests, and then select the failure log path in their terminal and then cat the log to their screen so they can search for their errors.  Every time, I've pointed out, "you can add test_output=errors to your .blazerc," they've thought it was great.  Sometimes they say, "Why isn't that just the default?"

--
MOS_MIGRATED_REVID=87734723

1 files changed, 3 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/exec/ExecutionOptions.java b/src/main/java/com/google/devtools/build/lib/exec/ExecutionOptions.java
index a12ece9f51..f6ddbe9a21 100644
--- a/src/main/java/com/google/devtools/build/lib/exec/ExecutionOptions.java
+++ b/src/main/java/com/google/devtools/build/lib/exec/ExecutionOptions.java
@@ -29,10 +29,10 @@ import java.util.Map;
 /**
  * Options affecting the execution phase of a build.
  *
- * These options are interpreted by the BuildTool to choose an Executor to
+ * <p>These options are interpreted by the BuildTool to choose an Executor to
  * be used for the build.
  *
- * Note: from the user's point of view, the characteristic function of this
+ * <p>Note: from the user's point of view, the characteristic function of this
  * set of options is indistinguishable from that of the BuildRequestOptions:
  * they are all per-request.  The difference is only apparent in the
  * implementation: these options are used only by the lib.exec machinery, which
@@ -119,7 +119,7 @@ public class ExecutionOptions extends OptionsBase {
   public PathFragment testTmpDir;
 
   @Option(name = "test_output",
-      defaultValue = "summary",
+      defaultValue = "errors",
       category = "testing",
       converter = TestStrategy.TestOutputFormat.Converter.class,
       help = "Specifies desired output mode. Valid values are 'summary' to "