Merge pull request #6 from jimboatarm/develop

Two batches of fixes to make productivity workloads run on more devices

wlauto/workloads/gmail/uiauto/src/com/arm/wlauto/uiauto/UiAutomation.java

@@ -17,7 +17,8 @@ public class UiAutomation extends UxPerfUiAutomation {
     public static String TAG = "uxperf_gmail";
 
     private Bundle parameters;
-    private long networkTimeout =  TimeUnit.SECONDS.toMillis(20);
+    private int networkTimeoutSecs = 20;
+    private long networkTimeout =  TimeUnit.SECONDS.toMillis(networkTimeoutSecs);
     private LinkedHashMap<String, Timer> timingResults = new LinkedHashMap<String, Timer>();
 
     public void runUiAutomation() throws Exception {
@@ -26,10 +27,10 @@ public class UiAutomation extends UxPerfUiAutomation {
         clearFirstRunDialogues();
 
         clickNewMail();
-        attachFiles();
         setToField();
         setSubjectField();
         setComposeField();
+        attachFiles();
         clickSendButton();
 
         writeResultsToFile(timingResults, parameters.getString("output_file"));
@@ -61,7 +62,7 @@ public class UiAutomation extends UxPerfUiAutomation {
 
     public void setToField() throws Exception {
         Timer result = new Timer();
-        UiObject toField = getUiObjectByDescription("To", "android.widget.TextView");
+        UiObject toField = getUiObjectByText("To", "android.widget.TextView");
         String recipient = parameters.getString("recipient").replace('_', ' ');
         result.start();
         toField.setText(recipient);
@@ -74,6 +75,8 @@ public class UiAutomation extends UxPerfUiAutomation {
         Timer result = new Timer();
         UiObject subjectField = getUiObjectByText("Subject", "android.widget.EditText");
         result.start();
+        // Click on the subject field is required on some platforms to exit the To box cleanly
+        subjectField.click();
         subjectField.setText("This is a test message");
         getUiDevice().pressEnter();
         result.end();
@@ -97,6 +100,10 @@ public class UiAutomation extends UxPerfUiAutomation {
         clickUiObject(sendButton, timeout);
         result.end();
         timingResults.put("Send", result);
+
+        UiObject drawerButton = new UiObject(new UiSelector().descriptionContains("avigat")
+                                                           .className("android.widget.ImageButton"));
+        waitObject(drawerButton, networkTimeoutSecs);
     }
 
     public void attachFiles() throws Exception {

wlauto/workloads/googlephotos/uiauto/src/com/arm/wlauto/uiauto/UiAutomation.java

@@ -84,6 +84,9 @@ public class UiAutomation extends UxPerfUiAutomation {
             getUiObjectByResourceId("com.google.android.apps.photos:id/next_button",
                                     "android.widget.ImageView");
         nextButton.clickAndWaitForNewWindow();
+
+        UiObject workingFolder = new UiObject(new UiSelector().text("wa-working"));
+        waitObject(workingFolder, viewTimeoutSecs);
     }
 
     private void gesturesTest() throws Exception {

wlauto/workloads/reader/uiauto/src/com/arm/wlauto/uiauto/UiAutomation.java

@@ -287,7 +287,7 @@ public class UiAutomation extends UxPerfUiAutomation {
         UiObject searchBox = getUiObjectByResourceId("android:id/search_src_text",
                                                      "android.widget.EditText");
         searchBox.setText(searchText);
-        searchBox.click();
+        getUiDevice().getInstance().pressSearch();
         Timer result = new Timer();
         result.start();
         getUiDevice().getInstance().pressEnter();

wlauto/workloads/skype/uiauto/src/com/arm/wlauto/uiauto/UiAutomation.java

@@ -49,6 +49,7 @@ public class UiAutomation extends UxPerfUiAutomation {
 
         // Run tests
         handleLoginScreen(loginName, loginPass);
+        confirmLocalFileAccess();
         selectContact(contactName, contactSkypeId);
         if ("video".equalsIgnoreCase(callType)) {
             videoCallTest(callDuration);
@@ -76,6 +77,15 @@ public class UiAutomation extends UxPerfUiAutomation {
         signinButton.clickAndWaitForNewWindow();
     }
 
+    private void confirmLocalFileAccess() throws Exception {
+        // First time run requires confirmation to allow access to local files
+        UiObject allowButton = new UiObject(new UiSelector().textContains("Allow")
+                                                            .className("android.widget.Button"));
+        if (allowButton.waitForExists(timeout)) {
+            allowButton.clickAndWaitForNewWindow(timeout);
+        }
+    }
+
     private void selectContact(String name, String id) throws Exception {
         Timer timer = new Timer();
         timer.start();