Clean up bugpattern AutoValueSubclassLeaked (bazelbuild#647)

Author: gkdn

javatests/io/bazel/rules/closure/worker/testing/ProgramResult.java

@@ -21,7 +21,6 @@
 import com.google.common.truth.FailureMetadata;
 import com.google.common.truth.Subject;
 import com.google.common.truth.Truth;
-import java.util.Arrays;
 import java.util.Set;
 
 /** Result of program invocation. */
@@ -37,6 +36,10 @@ public abstract class ProgramResult {
   /** Returns {@code true} if program invocation failed. */
   public abstract boolean failed();
 
+  static ProgramResult create(Set<String> errors, Set<String> warnings, boolean failed) {
+    return new AutoValue_ProgramResult(errors, warnings, failed);
+  }
+
   /** Begins a Truth assertion about a program invocation result. */
   public static ResultChain assertThat(ProgramResult result) {
     return Truth.assertAbout(SUBJECT_FACTORY).that(result);

javatests/io/bazel/rules/closure/worker/testing/ProgramRunner.java

@@ -37,6 +37,6 @@ public final class ProgramRunner<T extends Program> {
   /** Runs program and returns result of invocation. */
   public ProgramResult run() throws Exception {
     delegate.run();
-    return new AutoValue_ProgramResult(reporter.getErrors(), reporter.getWarnings(), failed.get());
+    return ProgramResult.create(reporter.getErrors(), reporter.getWarnings(), failed.get());
   }
 }