Blaze doesn't crash if a coverage combined report was requested and no test targets were found.
--
MOS_MIGRATED_REVID=85984804
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 5f7571a..c3f5788 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.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 @@
* 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
