diff options
| author |  Michajlo Matijkiw <michajlo@google.com> | 2015-02-26 16:39:01 +0000 |
|---|---|---|
| committer |  Han-Wen Nienhuys <hanwen@google.com> | 2015-02-26 16:39:01 +0000 |
| commit | 8f7358a32428a83324195c8783664281ee2fc3b0 (patch) | |
| tree | e99f4ab2876191b63f8d9feb358ef26fd9282aec /src/test/java/com/google/devtools/build/lib/analysis/BuildViewTestCase.java | |
| parent | 6682a318a75c4d19e4202753265af64eba5f340b (diff) | |
fix bug in test-specific top-level artifact helper behavior
A recent change resulted in TopLevelArtifactHelper.getTopLevelArtifacts()
returning an empty result when passed TopLevelArtifactContext.DEFAULT.
This only happens in tests and wasn't caught because tests were never
fully checking the results. Update MultiGroupForgeResourceAccountantTest
to better consider/expose this failure and fixed.
--
MOS_MIGRATED_REVID=87254923
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/analysis/BuildViewTestCase.java')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/analysis/BuildViewTestCase.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTestCase.java index 99c5740ab4..846f80c218 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTestCase.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTestCase.java @@ -1139,7 +1139,7 @@ public abstract class BuildViewTestCase extends FoundationTestCase { return null; } return view.update(loadingResult, masterConfig, viewOptions, - TopLevelArtifactContext.DEFAULT, reporter, eventBus); + AnalysisTestUtil.TOP_LEVEL_ARTIFACT_CONTEXT, reporter, eventBus); } protected static Predicate<Artifact> artifactNamed(final String name) { |