diff options
author | 2017-06-16 21:53:55 +0200 | |
---|---|---|
committer | 2017-06-19 18:24:48 +0200 | |
commit | cce0ec603e57929bc1de5f2c22f3ed4e903cc431 (patch) | |
tree | b223f4a67c3b9fb7c59481f2a250f61053b72835 /src/main/java/com/google | |
parent | e44da5a00f1094335fe143deac47684215f45b7e (diff) |
Roll forward R generation for Robolectric behind a flag to enable testing
RELNOTES: none
PiperOrigin-RevId: 159263527
Diffstat (limited to 'src/main/java/com/google')
3 files changed, 144 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java index a22c6a1655..77f047876a 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java +++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java @@ -526,6 +526,16 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { ) public boolean exportsManifestDefault; + @Option( + name = "experimental_android_generate_robolectric_r_class", + defaultValue = "false", + optionUsageRestrictions = OptionUsageRestrictions.UNDOCUMENTED, + help = + "If passed, R classes will be generated for Robolectric tests. Otherwise, only inherited" + + " R classes will be used." + ) + public boolean generateRobolectricRClass; + @Override public void addAllLabels(Multimap<String, Label> labelMap) { if (androidCrosstoolTop != null) { @@ -611,6 +621,7 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { private final boolean includeLibraryResourceJars; private final boolean useNocompressExtensionsOnApk; private final boolean exportsManifestDefault; + private final boolean generateRobolectricRClass; AndroidConfiguration(Options options, Label androidSdk) throws InvalidConfigurationException { this.sdk = androidSdk; @@ -643,6 +654,7 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { this.includeLibraryResourceJars = options.includeLibraryResourceJars; this.useNocompressExtensionsOnApk = options.useNocompressExtensionsOnApk; this.exportsManifestDefault = options.exportsManifestDefault; + this.generateRobolectricRClass = options.generateRobolectricRClass; if (!dexoptsSupportedInIncrementalDexing.contains("--no-locals")) { // TODO(bazel-team): Still needed? See DexArchiveAspect @@ -759,6 +771,10 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { return exportsManifestDefault; } + boolean generateRobolectricRClass() { + return generateRobolectricRClass; + } + @Override public void addGlobalMakeVariables(ImmutableMap.Builder<String, String> globalMakeEnvBuilder) { globalMakeEnvBuilder.put("ANDROID_CPU", cpu); diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLocalTestBase.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLocalTestBase.java index 09180a6ff0..5bc42bd331 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLocalTestBase.java +++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLocalTestBase.java @@ -82,6 +82,20 @@ public abstract class AndroidLocalTestBase implements RuleConfiguredTargetFactor // since they run on a JVM, not an android device. JavaTargetAttributes.Builder attributesBuilder = javaCommon.initCommon(); + if (ruleContext.getFragment(AndroidConfiguration.class).generateRobolectricRClass()) { + // Add reconciled R classes for all dependencies with resources to the classpath before the + // dependency jars. Must use a NestedSet to have it appear in the correct place on the + // classpath. + attributesBuilder.addRuntimeClassPathEntries( + RobolectricResourceSymbolsActionBuilder.create( + ResourceDependencies.fromRuleDeps(ruleContext, false)) + .setSdk(AndroidSdkProvider.fromRuleContext(ruleContext)) + .setJarOut( + ruleContext.getImplicitOutputArtifact( + AndroidRuleClasses.ANDROID_RESOURCES_CLASS_JAR)) + .buildAsClassPathEntry(ruleContext)); + } + String testClass = getAndCheckTestClass(ruleContext, ImmutableList.copyOf(attributesBuilder.getSourceFiles())); getAndCheckTestSupport(ruleContext); diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/RobolectricResourceSymbolsActionBuilder.java b/src/main/java/com/google/devtools/build/lib/rules/android/RobolectricResourceSymbolsActionBuilder.java new file mode 100644 index 0000000000..20fe4104fd --- /dev/null +++ b/src/main/java/com/google/devtools/build/lib/rules/android/RobolectricResourceSymbolsActionBuilder.java @@ -0,0 +1,114 @@ +// Copyright 2017 The Bazel Authors. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. +package com.google.devtools.build.lib.rules.android; + +import com.google.common.collect.FluentIterable; +import com.google.common.collect.Iterables; +import com.google.devtools.build.lib.actions.Artifact; +import com.google.devtools.build.lib.actions.ParameterFile.ParameterFileType; +import com.google.devtools.build.lib.analysis.RuleConfiguredTarget.Mode; +import com.google.devtools.build.lib.analysis.RuleContext; +import com.google.devtools.build.lib.analysis.actions.CustomCommandLine; +import com.google.devtools.build.lib.analysis.actions.SpawnAction; +import com.google.devtools.build.lib.collect.nestedset.NestedSet; +import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder; +import com.google.devtools.build.lib.rules.android.ResourceContainerConverter.Builder.SeparatorType; +import java.util.ArrayList; +import java.util.List; + +/** + * Builder for generating R classes for robolectric action. + * + * <p>This builder will create an action that generates r classes with internally consistent + * resource ids for the transitive closure of dependencies that provide resources. + */ +public class RobolectricResourceSymbolsActionBuilder { + + private static final ResourceContainerConverter.ToArtifacts RESOURCE_CONTAINER_TO_ARTIFACTS = + ResourceContainerConverter.builder() + .includeResourceRoots() + .includeManifest() + .includeRTxt() + .includeSymbolsBin() + .toArtifactConverter(); + private static final ResourceContainerConverter.ToArg RESOURCE_CONTAINER_TO_ARG = + ResourceContainerConverter.builder() + .includeResourceRoots() + .includeManifest() + .includeRTxt() + .includeSymbolsBin() + .withSeparator(SeparatorType.COLON_COMMA) + .toArgConverter(); + + private Artifact classJarOut; + private final ResourceDependencies dependencies; + private AndroidSdkProvider sdk; + + private RobolectricResourceSymbolsActionBuilder(ResourceDependencies dependencies) { + this.dependencies = dependencies; + } + + public static RobolectricResourceSymbolsActionBuilder create(ResourceDependencies dependencies) { + return new RobolectricResourceSymbolsActionBuilder(dependencies); + } + + public RobolectricResourceSymbolsActionBuilder setJarOut(Artifact classJar) { + this.classJarOut = classJar; + return this; + } + + public RobolectricResourceSymbolsActionBuilder setSdk(AndroidSdkProvider sdk) { + this.sdk = sdk; + return this; + } + + public NestedSet<Artifact> buildAsClassPathEntry(RuleContext ruleContext) { + CustomCommandLine.Builder builder = new CustomCommandLine.Builder(); + // Set the busybox tool. + builder.add("--tool").add("GENERATE_ROBOLECTRIC_R").add("--"); + + List<Artifact> inputs = new ArrayList<>(); + + builder.addExecPath("--androidJar", sdk.getAndroidJar()); + inputs.add(sdk.getAndroidJar()); + + if (!Iterables.isEmpty(dependencies.getResources())) { + builder.addJoinValues( + "--data", + RESOURCE_CONTAINER_TO_ARG.listSeparator(), + dependencies.getResources(), + RESOURCE_CONTAINER_TO_ARG); + } + + // This flattens the nested set. + Iterables.addAll(inputs, FluentIterable.from(dependencies.getResources()) + .transformAndConcat(RESOURCE_CONTAINER_TO_ARTIFACTS)); + + builder.addExecPath("--classJarOutput", classJarOut); + SpawnAction.Builder spawnActionBuilder = new SpawnAction.Builder(); + ruleContext.registerAction( + spawnActionBuilder + .useParameterFile(ParameterFileType.UNQUOTED) + .addInputs(inputs) + .addOutput(classJarOut) + .setCommandLine(builder.build()) + .setExecutable( + ruleContext.getExecutablePrerequisite("$android_resources_busybox", Mode.HOST)) + .setProgressMessage("Generating R classes for " + ruleContext.getLabel()) + .setMnemonic("GenerateRobolectricRClasses") + .build(ruleContext)); + + return NestedSetBuilder.<Artifact>naiveLinkOrder().add(classJarOut).build(); + } +} |