diff options
author | Stas Negara <snegara@google.com> | 2015-07-06 17:08:46 -0700 |
---|---|---|
committer | Stas Negara <snegara@google.com> | 2015-07-06 17:27:07 -0700 |
commit | d26d9a381824bf724f08bfd6b008759d5de1c489 (patch) | |
tree | fbabc51313dc43edbfee5bf87d3ca796adbbbc0a | |
parent | c23bb9c41c7c0372112fbfbb2865231bdb8e1aa8 (diff) | |
download | testing-d26d9a381824bf724f08bfd6b008759d5de1c489.tar.gz |
Handle additional skipped test execution states.
Also, refine messages explaining the reason a test execution
was skipped.
Change-Id: Ida15c1d74bb0bb4041c3529f804799316341ef51
-rw-r--r-- | src/com/google/gct/testing/CloudResultsLoader.java | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/com/google/gct/testing/CloudResultsLoader.java b/src/com/google/gct/testing/CloudResultsLoader.java index a15b9d4..24dd514 100644 --- a/src/com/google/gct/testing/CloudResultsLoader.java +++ b/src/com/google/gct/testing/CloudResultsLoader.java @@ -242,14 +242,15 @@ public class CloudResultsLoader { } String testExecutionState = testExecution.getState(); if (testExecutionState.equals("UNSUPPORTED_ENVIRONMENT")) { - handleTriggeringError(results, encodedConfigurationInstance, - "Skipped triggering the test execution: Incompatible API level for requested model"); + handleTriggeringError(results, encodedConfigurationInstance, "Incompatible device/OS combination"); } else if (testExecutionState.equals("INCOMPATIBLE_ENVIRONMENT")) { // It is not expected to happen for Android Studio client. - handleTriggeringError(results, encodedConfigurationInstance, "The given APK is not compatible with this configuration"); + handleTriggeringError(results, encodedConfigurationInstance, "Application does not support the specified OS version"); } else if (testExecutionState.equals("INVALID")) { // It is not expected to happen for Android Studio client. handleTriggeringError(results, encodedConfigurationInstance, "The provided APK is invalid"); + } else if (testExecutionState.equals("INCOMPATIBLE_ARCHITECTURE")) { + handleTriggeringError(results, encodedConfigurationInstance, "Application does not support the specified device architecture"); } else if (!testExecutionState.equals("PENDING")) { if (testExecutionState.equals("ERROR")) { String newProgress = INFRASTRUCTURE_FAILURE_PREFIX + " " + testExecution.getTestDetails().getErrorMessage(); @@ -275,8 +276,8 @@ public class CloudResultsLoader { } } - private void handleTriggeringError(Map<String, ConfigurationResult> results, String encodedConfigurationInstance, String newProgress) { - reportNewProgress(encodedConfigurationInstance, newProgress); + private void handleTriggeringError(Map<String, ConfigurationResult> results, String encodedConfigurationInstance, String errorMessage) { + reportNewProgress(encodedConfigurationInstance, "Skipped triggering the test execution: " + errorMessage); ConfigurationResult result = getOrCreateConfigurationResult(encodedConfigurationInstance, results); result.setTriggeringError(true); finishedConfigurationInstances.add(encodedConfigurationInstance); |