Blaze doesn't crash if a coverage combined report was requested and no test targets were found.

--
MOS_MIGRATED_REVID=85984804

1 files changed, 2 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/test/CoverageReportActionFactory.java b/src/main/java/com/google/devtools/build/lib/rules/test/CoverageReportActionFactory.java
index 5f7571a02e..c3f5788a64 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/test/CoverageReportActionFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/test/CoverageReportActionFactory.java
@@ -20,6 +20,7 @@ import com.google.devtools.build.lib.actions.ArtifactFactory;
 import com.google.devtools.build.lib.actions.ArtifactOwner;
 import com.google.devtools.build.lib.analysis.ConfiguredTarget;
 
+import java.util.Collection;
 import java.util.Set;
 
 import javax.annotation.Nullable;
@@ -35,7 +36,7 @@ public interface CoverageReportActionFactory {
    * the factory implementation, such as command line arguments.
    */
   @Nullable
-  public Action createCoverageReportAction(Iterable<ConfiguredTarget> targetsToTest,
+  public Action createCoverageReportAction(Collection<ConfiguredTarget> targetsToTest,
       Set<Artifact> baselineCoverageArtifacts,
       ArtifactFactory artifactFactory, ArtifactOwner artifactOwner);
 }
\ No newline at end of file