Clean up: use constants where we should be using them.
Xavier Ducrohet [Fri, 15 Feb 2013 19:01:01 +0000 (11:01 -0800)]
Change-Id: I51ae6bbd97b0365595ddc84653e4a0ce18cbd6eb

builder/src/main/java/com/android/builder/BuilderConstants.java
builder/src/main/java/com/android/builder/ProductFlavor.java
builder/src/main/java/com/android/builder/VariantConfiguration.java
gradle/src/fromGradle/groovy/com/android/build/gradle/internal/test/report/CompositeTestResults.java
gradle/src/main/groovy/com/android/build/gradle/AppPlugin.groovy
gradle/src/main/groovy/com/android/build/gradle/BaseExtension.groovy
gradle/src/main/groovy/com/android/build/gradle/BasePlugin.groovy
gradle/src/main/groovy/com/android/build/gradle/internal/ApplicationVariant.groovy

index 90dc77e..5c1500b 100644 (file)
@@ -26,7 +26,13 @@ public class BuilderConstants {
      */
     public final static String EXT_LIB_ARCHIVE = "aar";
 
+    /**
+     * The name of the default config.
+     */
+    public static final String MAIN = "main";
+
     public final static String DEBUG = "debug";
     public final static String RELEASE = "release";
 
+    public final static String TEST = "test";
 }
index cdc91f2..8d3450e 100644 (file)
@@ -29,11 +29,6 @@ import com.google.common.base.Objects;
 public class ProductFlavor extends BuildConfig {
     private static final long serialVersionUID = 1L;
 
-    /**
-     * The name of the default config.
-     */
-    public static final String MAIN = "main";
-
     private final String mName;
     private int mMinSdkVersion = -1;
     private int mTargetSdkVersion = -1;
@@ -51,7 +46,7 @@ public class ProductFlavor extends BuildConfig {
      * Names can be important when dealing with flavor groups.
      * @param name the name of the flavor.
      *
-     * @see #MAIN
+     * @see BuilderConstants#MAIN
      */
     public ProductFlavor(@NonNull String name) {
         mName = name;
index 4cfebca..84bbaa5 100644 (file)
@@ -544,7 +544,7 @@ public class VariantConfiguration {
 
         Set<File> mainResDirs = mDefaultSourceProvider.getResourcesDirectories();
 
-        ResourceSet resourceSet = new ResourceSet(ProductFlavor.MAIN);
+        ResourceSet resourceSet = new ResourceSet(BuilderConstants.MAIN);
         resourceSet.addSources(mainResDirs);
         if (generatedResFolder != null) {
             resourceSet.addSource(generatedResFolder);
@@ -599,7 +599,7 @@ public class VariantConfiguration {
 
         Set<File> mainResDirs = mDefaultSourceProvider.getAssetsDirectories();
 
-        AssetSet assetSet = new AssetSet(ProductFlavor.MAIN);
+        AssetSet assetSet = new AssetSet(BuilderConstants.MAIN);
         assetSet.addSources(mainResDirs);
         assetSets.add(assetSet);
 
index 4d5d10d..aab9cc6 100644 (file)
@@ -16,6 +16,7 @@
 
 package com.android.build.gradle.internal.test.report;
 
+import com.android.builder.BuilderConstants;
 import org.gradle.api.internal.tasks.testing.junit.report.TestResultModel;
 
 import java.math.BigDecimal;
@@ -151,7 +152,7 @@ public abstract class CompositeTestResults extends TestResultModel {
     }
 
     private static String getVariantKey(String projectName, String flavorName) {
-        if (flavorName.equals("Main")) {
+        if (BuilderConstants.MAIN.equalsIgnoreCase(flavorName)) {
             return projectName;
         }
 
index 7e70b0e..97b74bf 100644 (file)
@@ -117,7 +117,6 @@ class AppPlugin extends com.android.build.gradle.BasePlugin implements org.gradl
         productFlavorContainer.whenObjectRemoved {
             throw new UnsupportedOperationException("Removing product flavors is not supported.")
         }
-
     }
 
     /**
@@ -127,7 +126,7 @@ class AppPlugin extends com.android.build.gradle.BasePlugin implements org.gradl
      */
     private void addBuildType(BuildType buildType) {
         String name = buildType.name
-        if (name.startsWith("test")) {
+        if (name.startsWith(BuilderConstants.TEST)) {
             throw new RuntimeException("BuildType names cannot start with 'test'")
         }
         if (productFlavors.containsKey(name)) {
@@ -149,7 +148,7 @@ class AppPlugin extends com.android.build.gradle.BasePlugin implements org.gradl
      * @param productFlavor the product flavor
      */
     private void addProductFlavor(GroupableProductFlavor productFlavor) {
-        if (productFlavor.name.startsWith("test")) {
+        if (productFlavor.name.startsWith(BuilderConstants.TEST)) {
             throw new RuntimeException("ProductFlavor names cannot start with 'test'")
         }
         if (buildTypes.containsKey(productFlavor.name)) {
@@ -157,7 +156,7 @@ class AppPlugin extends com.android.build.gradle.BasePlugin implements org.gradl
         }
 
         def mainSourceSet = extension.sourceSetsContainer.create(productFlavor.name)
-        String testName = "test${productFlavor.name.capitalize()}"
+        String testName = "${BuilderConstants.TEST}${productFlavor.name.capitalize()}"
         def testSourceSet = extension.sourceSetsContainer.create(testName)
 
         ProductFlavorData<GroupableProductFlavor> productFlavorData =
index 8ae1ba3..0230396 100644 (file)
  * limitations under the License.
  */
 package com.android.build.gradle
-
 import com.android.build.gradle.internal.CompileOptions
 import com.android.build.gradle.internal.dsl.AaptOptionsImpl
 import com.android.build.gradle.internal.dsl.AndroidSourceSetFactory
 import com.android.build.gradle.internal.dsl.DexOptionsImpl
-import com.android.build.gradle.internal.dsl.GroupableProductFlavor
 import com.android.build.gradle.internal.dsl.ProductFlavorDsl
 import com.android.build.gradle.internal.test.TestOptions
+import com.android.builder.BuilderConstants
 import com.android.builder.ProductFlavor
 import org.gradle.api.Action
 import org.gradle.api.NamedDomainObjectContainer
@@ -58,7 +57,7 @@ public abstract class BaseExtension {
     BaseExtension(BasePlugin plugin, ProjectInternal project, Instantiator instantiator) {
         this.plugin = plugin
 
-        defaultConfig = instantiator.newInstance(ProductFlavorDsl.class, ProductFlavor.MAIN)
+        defaultConfig = instantiator.newInstance(ProductFlavorDsl.class, BuilderConstants.MAIN)
 
         aaptOptions = instantiator.newInstance(AaptOptionsImpl.class)
         dexOptions = instantiator.newInstance(DexOptionsImpl.class)
@@ -115,7 +114,7 @@ public abstract class BaseExtension {
         action.execute(sourceSetsContainer)
     }
 
-    void defaultConfig(Action<GroupableProductFlavor> action) {
+    void defaultConfig(Action<ProductFlavor> action) {
         action.execute(defaultConfig)
     }
 
index 968e655..e489559 100644 (file)
@@ -180,7 +180,7 @@ public abstract class BasePlugin {
     protected setDefaultConfig(ProductFlavor defaultConfig,
                                NamedDomainObjectContainer<AndroidSourceSet> sourceSets) {
         mainSourceSet = sourceSets.create(defaultConfig.name)
-        testSourceSet = sourceSets.create("test")
+        testSourceSet = sourceSets.create(BuilderConstants.TEST)
 
         defaultConfigData = new ProductFlavorData<ProductFlavor>(defaultConfig, mainSourceSet,
                 testSourceSet, project, ConfigurationDependencies.ConfigType.DEFAULT)
index a54f917..fa750d8 100644 (file)
@@ -29,6 +29,7 @@ import com.android.build.gradle.tasks.ProcessManifest
 import com.android.build.gradle.tasks.RenderscriptCompile
 import com.android.build.gradle.tasks.ZipAlign
 import com.android.builder.AndroidBuilder
+import com.android.builder.BuilderConstants
 import com.android.builder.ProductFlavor
 import com.android.builder.VariantConfiguration
 import org.gradle.api.Task
@@ -97,7 +98,7 @@ public abstract class ApplicationVariant {
         if (config.hasFlavors()) {
             return "${getFlavoredName(true)}"
         } else {
-            return "Main"
+            return BuilderConstants.MAIN.capitalize()
         }
     }