diff options
author | cparsons <cparsons@google.com> | 2018-01-09 11:16:48 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-01-09 11:19:18 -0800 |
commit | 48c21586a6776c91172f2ed5f9a63d0c22e08d16 (patch) | |
tree | f84da10f91ab222bb8c911b11eb7a4d102ae7ffb /src/test | |
parent | 7253a8bfe43b5c661f04b336017cda589f892855 (diff) |
Refactor ObjcProvider to abide by Native Declared Provider best practices.
This has the effect of documenting exposed struct fields on the provider.
RELNOTES: None.
PiperOrigin-RevId: 181346052
Diffstat (limited to 'src/test')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/objc/ObjcProviderTest.java | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcProviderTest.java b/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcProviderTest.java index 2b7f3fe6a3..0996bb4318 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcProviderTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcProviderTest.java @@ -20,7 +20,6 @@ import static com.google.common.truth.Truth.assertWithMessage; import com.google.common.collect.ImmutableList; import com.google.common.collect.ImmutableSet; import com.google.devtools.build.lib.rules.objc.ObjcProvider.Key; -import com.google.devtools.build.lib.syntax.SkylarkNestedSet; import com.google.devtools.build.lib.vfs.PathFragment; import java.lang.reflect.Field; import java.lang.reflect.Modifier; @@ -127,13 +126,9 @@ public class ObjcProviderTest { .build(); ObjcProvider depender = new ObjcProvider.Builder().addTransitiveAndPropagate(provider).build(); - assertThat( - ((SkylarkNestedSet) provider.getValue(ObjcProvider.INCLUDE.getSkylarkKeyName())) - .toCollection()) + assertThat(provider.include().toCollection()) .containsExactly(strictInclude.toString(), propagatedInclude.toString()); - assertThat( - ((SkylarkNestedSet) depender.getValue(ObjcProvider.INCLUDE.getSkylarkKeyName())) - .toCollection()) + assertThat(depender.include().toCollection()) .containsExactly(propagatedInclude.toString()); } |