From 38f2d4599776c8d5af70e16c7151bd40708a4860 Mon Sep 17 00:00:00 2001 From: Googler Date: Tue, 10 Feb 2015 14:53:24 +0000 Subject: Blaze doesn't crash if a coverage combined report was requested and no test targets were found. -- MOS_MIGRATED_REVID=85984804 --- .../devtools/build/lib/rules/test/CoverageReportActionFactory.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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 targetsToTest, + public Action createCoverageReportAction(Collection targetsToTest, Set baselineCoverageArtifacts, ArtifactFactory artifactFactory, ArtifactOwner artifactOwner); } \ No newline at end of file -- cgit v1.2.3