diff options
| author |  carmi <carmi@google.com> | 2017-05-17 22:56:19 +0200 |
|---|---|---|
| committer |  Dmitry Lomov <dslomov@google.com> | 2017-05-19 15:08:02 +0200 |
| commit | 31c6a6188b46008308f6dc2d8af965a5f6c352fe (patch) | |
| tree | 260bb2016d97dd2cb4df4ff9c6cae7bb65c41b1a /src/test/java/com/google/devtools/build/lib/testutil | |
| parent | 8dd69b609304bf3f89048dd9446076698c092e1b (diff) | |
Expose the target map from Package.
This doesn't really expose new information, because one can iterate over the existing getTargets() and recreate the map.
Moreover, the map is immutable, so there's no risk in returning it to the user.
RELNOTES: None
PiperOrigin-RevId: 156349797
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/testutil')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/testutil/BazelPackageBuilderHelperForTesting.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/testutil/BazelPackageBuilderHelperForTesting.java b/src/test/java/com/google/devtools/build/lib/testutil/BazelPackageBuilderHelperForTesting.java index 21f20cfb16..946ece8cee 100644 --- a/src/test/java/com/google/devtools/build/lib/testutil/BazelPackageBuilderHelperForTesting.java +++ b/src/test/java/com/google/devtools/build/lib/testutil/BazelPackageBuilderHelperForTesting.java @@ -88,9 +88,10 @@ public class BazelPackageBuilderHelperForTesting implements Package.Builder.Help throw new IllegalStateException(e); } ImmutableSet<Label> targetsInPkg = - ImmutableSet.copyOf(Iterables.transform(pkg.getTargets(), TARGET_TO_LABEL)); + ImmutableSet.copyOf(Iterables.transform(pkg.getTargets().values(), TARGET_TO_LABEL)); ImmutableSet<Label> targetsInNewlyLoadedPkg = - ImmutableSet.copyOf(Iterables.transform(newlyLoadedPkg.getTargets(), TARGET_TO_LABEL)); + ImmutableSet.copyOf( + Iterables.transform(newlyLoadedPkg.getTargets().values(), TARGET_TO_LABEL)); if (!targetsInPkg.equals(targetsInNewlyLoadedPkg)) { throw new IllegalStateException(String.format( "The Package for %s had a different set of targets (<targetsInPkg> - " |