During test runs, ensure buck uses plain logging.

Occasionally, system properties would mean that Buck running
as part of a test run would attempt to use an ANSI console
for logging. This makes parsing the output unstable and
leads to spurious failures.

Author: shs96c

java/client/test/org/openqa/selenium/BuckBuild.java

@@ -67,8 +67,7 @@ public Path go() throws IOException {
 
     ImmutableList.Builder<String> builder = ImmutableList.builder();
     findBuck(projectRoot, builder);
-    builder.add("build");
-    builder.add(target);
+    builder.add("build", "--config", "color.ui=never", target);
 
     ImmutableList<String> command = builder.build();
     CommandLine commandLine = new CommandLine(command.toArray(new String[command.size()]));
@@ -85,7 +84,7 @@ public Path go() throws IOException {
   private Path findOutput(Path projectRoot) throws IOException {
     ImmutableList.Builder<String> builder = ImmutableList.builder();
     findBuck(projectRoot, builder);
-    builder.add("targets", "--show-output", target);
+    builder.add("targets", "--show-full-output", "--config", "color.ui=never", target);
 
     ImmutableList<String> command = builder.build();
     CommandLine commandLine = new CommandLine(command.toArray(new String[command.size()]));
@@ -96,15 +95,16 @@ private Path findOutput(Path projectRoot) throws IOException {
       throw new WebDriverException("Unable to find output! " + target);
     }
 
-    String[] allLines = commandLine.getStdOut().split(LINE_SEPARATOR.value());
+    String stdOut = commandLine.getStdOut();
+    String[] allLines = stdOut.split(LINE_SEPARATOR.value());
     String lastLine = null;
     for (String line : allLines) {
       if (line.startsWith(target)) {
         lastLine = line;
         break;
       }
     }
-    Preconditions.checkNotNull(lastLine);
+    Preconditions.checkNotNull(lastLine, "Value read: %s", stdOut);
 
     List<String> outputs = Splitter.on(' ').limit(2).splitToList(lastLine);
     if (outputs.size() != 2) {