commit 06e5720041e4aa26cfaaddf2544f011f2addb25a
author Stas Negara <snegara@google.com> Mon Mar 02 13:43:10 2015 -0800
committer Stas Negara <snegara@google.com> Wed Mar 04 16:39:59 2015 -0800
tree 42a25181cb7e596f7a0642f172ecca541558f989
parent 8d57c376fa697381763c3e0878d0f20b67ed94a5

Use app package id instead of module name for history name.

Change-Id: I05fb41e9324064b3513cb4c8fdc9bff1b4d180fa

src/com/google/gct/testing/CloudTestConfigurationProviderImpl.java

diff --git a/src/com/google/gct/testing/CloudTestConfigurationProviderImpl.java b/src/com/google/gct/testing/CloudTestConfigurationProviderImpl.java
index 3531388..434d493 100644
--- a/src/com/google/gct/testing/CloudTestConfigurationProviderImpl.java
+++ b/src/com/google/gct/testing/CloudTestConfigurationProviderImpl.java
@@ -293,7 +293,7 @@
           String testSpecification = GoogleCloudTestingUtils.prepareTestSpecification(testRunConfiguration);
 
           Map<String, TestExecution> testExecutions = CloudTestsLauncher
-            .triggerTestApi(cloudProjectId, moduleName, getBucketGcsPath(bucketName), getApkGcsPath(bucketName, appApkName),
+            .triggerTestApi(cloudProjectId, getBucketGcsPath(bucketName), getApkGcsPath(bucketName, appApkName),
                             getApkGcsPath(bucketName, testApkName), testSpecification, testRunConfiguration.INSTRUMENTATION_RUNNER_CLASS,
                             encodedMatrixInstances, appPackage, testPackage);
 

src/com/google/gct/testing/launcher/CloudTestsLauncher.java

diff --git a/src/com/google/gct/testing/launcher/CloudTestsLauncher.java b/src/com/google/gct/testing/launcher/CloudTestsLauncher.java
index cfae4f6..26e5955 100644
--- a/src/com/google/gct/testing/launcher/CloudTestsLauncher.java
+++ b/src/com/google/gct/testing/launcher/CloudTestsLauncher.java
@@ -101,7 +101,7 @@
   }
 
   public static Map<String, TestExecution> triggerTestApi(
-    String cloudProjectId, String applicationName, String bucketGcsPath, String appApkGcsPath, String testApkGcsPath,
+    String cloudProjectId, String bucketGcsPath, String appApkGcsPath, String testApkGcsPath,
     String testSpecification, String instrumentationTestRunner, List<String> matrixInstances, String appPackage, String testPackage) {
 
     if (instrumentationTestRunner.isEmpty()) {
@@ -117,7 +117,7 @@
         .setTestApk(new FileReference().setGcsPath(testApkGcsPath)).setAppPackageId(appPackage).setTestPackageId(testPackage)
         .setTestRunnerClass(instrumentationTestRunner).setTestTargets(Lists.newArrayList(testSpecification))));
 
-    String historyId = getHistoryId(cloudProjectId, applicationName);
+    String historyId = getHistoryId(cloudProjectId, appPackage);
 
     for (String matrixInstance : matrixInstances) {
       TestExecution currentTestExecution = testExecution.clone();