diff options
author | Cal Peyser <cpeyser@google.com> | 2016-06-23 16:17:43 +0000 |
---|---|---|
committer | Lukacs Berki <lberki@google.com> | 2016-06-24 08:12:05 +0000 |
commit | 88cdce3ec99fa7b2a2e2ee764fc57a83b298975e (patch) | |
tree | ca904b843836ca66df5f71a792ab8510c367d19e | |
parent | f9c4102eefe81591dc74c94d245c6ba52618d6e5 (diff) |
Sheepishly add a warning about the problem in which getConfiguredTarget() returns null for failed SkyKey evaluations.
--
MOS_MIGRATED_REVID=125684386
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java index 956da5c8b4..ad3758b6f5 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java @@ -675,6 +675,12 @@ public abstract class BuildViewTestCase extends FoundationTestCase { /** * Returns the ConfiguredTarget for the specified label, using the * given build configuration. + * + * <p>If the evaluation of the SkyKey corresponding to the configured target fails, this + * method may return null. In that case, use a debugger to inspect the {@link ErrorInfo} + * for the evaluation, which is produced by the + * {@link MemoizingEvaluator#getExistingValueForTesting} call in + * {@link SkyframeExecutor#getConfiguredTargetForTesting}. See also b/26382502. */ protected ConfiguredTarget getConfiguredTarget(Label label, BuildConfiguration config) throws NoSuchPackageException, NoSuchTargetException, InterruptedException { |