Print coverage.dat location always.

--
MOS_MIGRATED_REVID=121960458

1 files changed, 13 insertions, 17 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/TestSummaryPrinter.java b/src/main/java/com/google/devtools/build/lib/runtime/TestSummaryPrinter.java
index ee24048b07..ae6e3934f9 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/TestSummaryPrinter.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/TestSummaryPrinter.java
@@ -127,24 +127,20 @@ public class TestSummaryPrinter {
       }
     }
 
-    if (printFailedTestCases) {
-      // In this mode, test output and coverage files would just clutter up
-      // the output.
-      return;
-    }
-
-    for (String warning : summary.getWarnings()) {
-      terminalPrinter.print("  " + AnsiTerminalPrinter.Mode.WARNING + "WARNING: "
-          + AnsiTerminalPrinter.Mode.DEFAULT + warning + "\n");
-    }
+    if (!printFailedTestCases) {
+      for (String warning : summary.getWarnings()) {
+        terminalPrinter.print("  " + AnsiTerminalPrinter.Mode.WARNING + "WARNING: "
+            + AnsiTerminalPrinter.Mode.DEFAULT + warning + "\n");
+      }
 
-    for (Path path : summary.getFailedLogs()) {
-      if (path.exists()) {
-        // Don't use getPrettyPath() here - we want to print the absolute path,
-        // so that it cut and paste into a different terminal, and we don't
-        // want to use the blaze-bin etc. symbolic links because they could be changed
-        // by a subsequent build with different options.
-        terminalPrinter.print("  " + path.getPathString() + "\n");
+      for (Path path : summary.getFailedLogs()) {
+        if (path.exists()) {
+          // Don't use getPrettyPath() here - we want to print the absolute path,
+          // so that it cut and paste into a different terminal, and we don't
+          // want to use the blaze-bin etc. symbolic links because they could be changed
+          // by a subsequent build with different options.
+          terminalPrinter.print("  " + path.getPathString() + "\n");
+        }
       }
     }
     for (Path path : summary.getCoverageFiles()) {