diff options
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/test')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/test/BaselineCoverageAction.java | 7 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/test/CoverageReportActionFactory.java | 3 |
2 files changed, 6 insertions, 4 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/test/BaselineCoverageAction.java b/src/main/java/com/google/devtools/build/lib/rules/test/BaselineCoverageAction.java index 6a19f92def..b341b32856 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/test/BaselineCoverageAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/test/BaselineCoverageAction.java @@ -24,6 +24,9 @@ import com.google.devtools.build.lib.actions.NotifyOnActionCacheHit; import com.google.devtools.build.lib.analysis.RuleContext; import com.google.devtools.build.lib.analysis.Util; import com.google.devtools.build.lib.analysis.actions.AbstractFileWriteAction; +import com.google.devtools.build.lib.collect.nestedset.NestedSet; +import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder; +import com.google.devtools.build.lib.collect.nestedset.Order; import com.google.devtools.build.lib.events.EventHandler; import com.google.devtools.build.lib.syntax.Label; import com.google.devtools.build.lib.util.Fingerprint; @@ -117,7 +120,7 @@ public class BaselineCoverageAction extends AbstractFileWriteAction * Returns collection of baseline coverage artifacts associated with the given target. * Will always return 0 or 1 elements. */ - public static ImmutableList<Artifact> getBaselineCoverageArtifacts(RuleContext ruleContext, + public static NestedSet<Artifact> getBaselineCoverageArtifacts(RuleContext ruleContext, Iterable<Artifact> instrumentedFiles) { // Baseline coverage artifacts will still go into "testlogs" directory. Artifact coverageData = ruleContext.getAnalysisEnvironment().getDerivedArtifact( @@ -126,7 +129,7 @@ public class BaselineCoverageAction extends AbstractFileWriteAction ruleContext.registerAction(new BaselineCoverageAction( ruleContext.getActionOwner(), instrumentedFiles, coverageData)); - return ImmutableList.of(coverageData); + return NestedSetBuilder.create(Order.STABLE_ORDER, coverageData); } } 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 9354d4340b..3a1a6137d8 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 @@ -23,7 +23,6 @@ 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; @@ -70,6 +69,6 @@ public interface CoverageReportActionFactory { @Nullable public CoverageReportActionsWrapper createCoverageReportActionsWrapper( Collection<ConfiguredTarget> targetsToTest, - Set<Artifact> baselineCoverageArtifacts, + Iterable<Artifact> baselineCoverageArtifacts, ArtifactFactory artifactFactory, ArtifactOwner artifactOwner); } |