diff options
author | asteinb <asteinb@google.com> | 2018-05-22 08:38:40 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-05-22 08:40:04 -0700 |
commit | 06e43f8b56ca50d7b87c7963d2251d1c72c00977 (patch) | |
tree | b41965fdb1f2b6d91c8c0b2ac19e14ea7c4b18b2 /src/test/java | |
parent | 85a6f166cef1231f1e436978171870ce6b10a6d8 (diff) |
Pass AndroidDataContext through manifest processing methods
Manifest processing methods are particularly messy for this migration, since
the old ApplicationManifest class is still around. Anyway, pass around
AndroidDataContext instead of RuleContext everywhere we can.
Note that the built-in expander does not seem able to be modified to support
decoupling attributes and other information, and thus really can't be done once
we get rid of RuleContext. Instead, for Skylark rules, document that expansion
must happen outside of the Android data Skylark method calls (for example, for
manifest_values and nocompress_extensions).
RELNOTES: none
PiperOrigin-RevId: 197567541
Diffstat (limited to 'src/test/java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java index bc06435f46..6ba8585b01 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java @@ -1076,17 +1076,17 @@ public class AndroidLibraryTest extends AndroidBuildViewTestCase { " srcs = ['foo.java'])"); ConfiguredTarget target = getConfiguredTarget("//java/android:l"); - Artifact manifest = getBinArtifact("l_generated/l/AndroidManifest.xml", target); + Artifact manifest = getBinArtifact("_generated/l/AndroidManifest.xml", target); FileWriteAction action = (FileWriteAction) getGeneratingAction(manifest); assertThat(action.getFileContents()).contains("package=\"android\""); target = getConfiguredTarget("//java/android:l2"); - manifest = getBinArtifact("l2_generated/l2/AndroidManifest.xml", target); + manifest = getBinArtifact("_generated/l2/AndroidManifest.xml", target); action = (FileWriteAction) getGeneratingAction(manifest); assertThat(action.getFileContents()).contains("package=\"foo\""); target = getConfiguredTarget("//third_party/android:l"); - manifest = getBinArtifact("l_generated/l/AndroidManifest.xml", target); + manifest = getBinArtifact("_generated/l/AndroidManifest.xml", target); action = (FileWriteAction) getGeneratingAction(manifest); assertThat(action.getFileContents()).contains("package=\"third_party.android\""); } |