Exclude test/java/ in instrumentation_filter default

Googler · buchgr · commit 2c0583f3b7d7 · 2017-07-27T16:26:22.000+02:00
The computed default used in bazel coverage excludes both javatests/ and test/java. Having the stated default exclude only one of those is needlessly confusing. (Even though in practice that static default will rarely be used.)

Also updates the comment about the computed default, since that logic has moved.

PiperOrigin-RevId: 163325837
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
@@ -596,11 +596,11 @@ public static class Options extends FragmentOptions implements Cloneable {
     public boolean stampBinaries;
 
     // This default value is always overwritten in the case of "bazel coverage" by
-    // CoverageCommand.setDefaultInstrumentationFilter().
+    // a value returned by InstrumentationFilterSupport.computeInstrumentationFilter.
     @Option(
       name = "instrumentation_filter",
       converter = RegexFilter.RegexFilterConverter.class,
-      defaultValue = "-/javatests[/:]",
+      defaultValue = "-/javatests[/:],-/test/java[/:]",
       category = "semantics",
       documentationCategory = OptionDocumentationCategory.UNCATEGORIZED,
       effectTags = {OptionEffectTag.UNKNOWN},
