chore: new test workspace

.github/scripts/run-maestro.sh

@@ -5,7 +5,7 @@ PLATFORM="${1:-${PLATFORM:-android}}"
 SHARD="${2:-${SHARD:-default}}"
 FLOWS_DIR=".maestro/tests"
 MAIN_REPORT="maestro-report.xml"
-MAX_RERUN_ROUNDS="${MAX_RERUN_ROUNDS:-2}"
+MAX_RERUN_ROUNDS="${MAX_RERUN_ROUNDS:-3}"
 RERUN_REPORT_PREFIX="maestro-rerun"
 export MAESTRO_DRIVER_STARTUP_TIMEOUT="${MAESTRO_DRIVER_STARTUP_TIMEOUT:-120000}"
 
@@ -51,11 +51,12 @@ echo "Mapped flows for tag test-${SHARD}:"
 awk -F'\t' '{ printf "  %s -> %s\n", $1, $2 }' "$MAPFILE"
 
 FLOW_FILES=()
-declare -A SEEN
+SEEN_PATHS=""
+
 while IFS=$'\t' read -r name path; do
-  if [ -z "${SEEN[$path]:-}" ]; then
+  if ! printf '%s\n' "$SEEN_PATHS" | grep -Fqx "$path"; then
     FLOW_FILES+=("$path")
-    SEEN[$path]=1
+    SEEN_PATHS="${SEEN_PATHS}"$'\n'"$path"
   fi
 done < "$MAPFILE"
 

.github/workflows/maestro-android.yml

@@ -68,4 +68,4 @@ jobs:
         with:
           name: Android Maestro Logs - Shard ${{ inputs.shard }}
           path: ~/.maestro/tests/**/*.png
-          retention-days: 7
+          retention-days: 7

.github/workflows/maestro-ios.yml

@@ -101,4 +101,4 @@ jobs:
         with:
           name: iOS Maestro Logs - Shard ${{ inputs.shard }}
           path: ~/.maestro/tests/**/*.png
-          retention-days: 7
+          retention-days: 7

.maestro/README.md

@@ -39,7 +39,7 @@ Contains the scripts that are going to be executed by the flows before running t
 #### `data.js`
 
 - Contains seeds to common test data, like server url, public channels, etc
-- Currently we point to https://mobile.rocket.chat as main server
+- Currently we point to https://mobile.qa.rocket.chat as main server
   - Pointing to a local server is not recommended yet, as you would need to create a few public channels and change some permissions
   - Ideally we should point to a docker or even a mocked server, but that's tbd
 - Try not to add new data there. Use random values instead.

.maestro/scripts/data.js

@@ -1,5 +1,5 @@
 const data = {
-	server: 'https://mobile.rocket.chat',
+	server: 'https://mobile.qa.rocket.chat',
 	alternateServer: 'https://stable.rocket.chat',
 	...output.account,
     accounts: [],
@@ -15,4 +15,4 @@ const data = {
 	e2eePassword: 'Password1@abcdefghijklmnopqrst'
 };
 
-output.data = data;
+output.data = data;

.maestro/scripts/random.js

@@ -7,12 +7,37 @@ function random(length = 10) {
 	return text;
 }
 
+function generatePassword(length = 10) {
+    const lower = 'abcdefghijklmnopqrstuvwxyz'
+    const upper = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ'
+    const nums = '0123456789'
+    const all = lower + upper + nums
+
+    let password = ''
+    password += lower[Math.floor(Math.random() * lower.length)]
+    password += upper[Math.floor(Math.random() * upper.length)]
+    password += nums[Math.floor(Math.random() * nums.length)]
+
+    while (password.length < length) {
+        const char = all[Math.floor(Math.random() * all.length)]
+        const last = password[password.length - 1]
+        const secondLast = password[password.length - 2]
+        const thirdLast = password[password.length - 3]
+
+        if (char === last && char === secondLast && char === thirdLast) continue
+        password += char
+    }
+
+    return password
+}
+
+
 const randomUser = () => {
     const randomVal = random();
     return {
         username: 'user' + randomVal,
         name: 'user' + randomVal,
-        password: 'Password1@' + randomVal,
+        password: generatePassword(),
         email: 'mobile+' + randomVal + '@rocket.chat'
     };
 }

.maestro/tests/assorted/accessibility-and-appearance.yaml

@@ -87,11 +87,11 @@ tags:
 
 - extendedWaitUntil:
     visible:
-      text: '.*@all - all mention.*'
+      text: '@all - all mention'
     timeout: 60000
 - extendedWaitUntil:
     visible:
-      text: '.*@rocket.cat - user mention.*'
+      text: '@rocket.cat - user mention'
     timeout: 60000
 
 # should disable mentions with @ symbol
@@ -126,19 +126,11 @@ tags:
 
 - extendedWaitUntil:
     visible:
-      text: '.*all - all mention.*'
-    timeout: 60000
-- extendedWaitUntil:
-    notVisible:
-      text: '.*@all - all mention.*'
+      text: 'all - all mention'
     timeout: 60000
 - extendedWaitUntil:
     visible:
-      text: '.*rocket.cat - user mention.*'
-    timeout: 60000
-- extendedWaitUntil:
-    notVisible:
-      text: '.*@rocket.cat - user mention.*'
+      text: 'rocket.cat - user mention'
     timeout: 60000
 
 # should enable rooms with # symbol
@@ -173,7 +165,7 @@ tags:
 
 - extendedWaitUntil:
     visible:
-      text: '.*#general - channel mention.*'
+      text: '#general - channel mention'
     timeout: 60000
 
 # should disable mentions with # symbol
@@ -205,12 +197,7 @@ tags:
     file: '../../helpers/search-and-navigate-room.yaml'
     env:
       ROOM: 'maestro-accessibility-test'
-
 - extendedWaitUntil:
     visible:
-      text: '.*general - channel mention.*'
-    timeout: 60000
-- extendedWaitUntil:
-    notVisible:
-      text: '.*#general - channel mention.*'
+      text: 'general - channel mention'
     timeout: 60000

.maestro/tests/assorted/changeserver.yaml

@@ -180,8 +180,20 @@ tags:
     visible:
       text: '.*Delete.*'
     timeout: 60000
-- tapOn:
-    text: 'Delete'
+- runFlow:
+    when:
+      platform: iOS
+    commands:
+      - tapOn:
+          point: 65%,55%
+          retryTapIfNoChange: true
+- runFlow:
+    when:
+      platform: Android
+    commands:
+      - tapOn:
+          text: 'Delete'
+          retryTapIfNoChange: true
 
 # verify alternate server is deleted
 - waitForAnimationToEnd:

.maestro/tests/assorted/delete-server.yaml

@@ -93,8 +93,20 @@ tags:
     visible:
       text: '.*Delete.*'
     timeout: 60000
-- tapOn:
-    text: 'Delete'
+- runFlow:
+    when:
+      platform: iOS
+    commands:
+      - tapOn:
+          point: 65%,55%
+          retryTapIfNoChange: true
+- runFlow:
+    when:
+      platform: Android
+    commands:
+      - tapOn:
+          text: 'Delete'
+          retryTapIfNoChange: true
 - waitForAnimationToEnd:
     timeout: 300
 - extendedWaitUntil:

.maestro/tests/assorted/user-preferences.yaml

@@ -83,6 +83,7 @@ tags:
           visible:
             text: ':('
           timeout: 60000
+
       - extendedWaitUntil:
           notVisible:
             text: '😞'
@@ -140,7 +141,3 @@ tags:
           visible:
             text: '😞'
           timeout: 60000
-      - extendedWaitUntil:
-          notVisible:
-            text: ':('
-          timeout: 60000

.maestro/tests/onboarding/server-history.yaml

@@ -23,11 +23,11 @@ tags:
 - assertVisible:
     id: 'action-sheet'
 - assertVisible:
-    id: 'servers-history-https://mobile.rocket.chat'
+    id: 'servers-history-https://mobile.qa.rocket.chat'
 
 # should tap on a server history and navigate to login
 - tapOn:
-    id: 'servers-history-https://mobile.rocket.chat'
+    id: 'servers-history-https://mobile.qa.rocket.chat'
 - assertVisible:
     id: 'login-view-submit'
 - assertVisible: ${output.user.username}
@@ -43,17 +43,17 @@ tags:
 - tapOn:
     id: 'servers-history-button'
 - assertVisible:
-    id: 'servers-history-https://mobile.rocket.chat'
+    id: 'servers-history-https://mobile.qa.rocket.chat'
 
 # swipe left to reveal delete button and delete the server history
 - swipe:
     direction: LEFT
     from:
-      id: 'servers-history-https://mobile.rocket.chat'
+      id: 'servers-history-https://mobile.qa.rocket.chat'
 - waitForAnimationToEnd:
     timeout: 500
 - tapOn:
-    id: 'servers-history-https://mobile.rocket.chat-delete'
+    id: 'servers-history-https://mobile.qa.rocket.chat-delete'
 - extendedWaitUntil:
     notVisible:
       id: 'servers-history-button'

.maestro/tests/room/room.yaml

@@ -256,7 +256,7 @@ tags:
 - swipe:
     from:
       id: action-sheet-handle
-    direction: down
+    direction: DOWN
 - extendedWaitUntil:
     notVisible:
       id: 'reactionsList'
@@ -269,6 +269,7 @@ tags:
     timeout: 60000
 - tapOn:
     id: 'username-header-${output.user.username}'
+    retryTapIfNoChange: true
 - extendedWaitUntil:
     visible:
       id: room-info-view-username
@@ -289,10 +290,11 @@ tags:
 
 - extendedWaitUntil:
     visible:
-      text: .*${output.otherUser.username}.*
+      id: 'username-header-${output.otherUser.username}'
     timeout: 60000
 - tapOn:
-    text: .*${output.otherUser.username}.*
+    id: 'username-header-${output.otherUser.username}'
+    retryTapIfNoChange: true
 - extendedWaitUntil:
     visible:
       id: room-info-view-username

.maestro/tests/room/unread-badge.yaml

@@ -4,6 +4,7 @@ onFlowStart:
   - runFlow: '../../helpers/setup.yaml'
 tags:
   - test-13
+  - android-only
 
 ---
 - evalScript: ${output.user = output.utils.createUser()}
@@ -21,6 +22,8 @@ tags:
     visible:
       id: 'rooms-list-view-item-${output.otherUser.username}'
     timeout: 60000
+- waitForAnimationToEnd:
+    timeout: 10000
 - extendedWaitUntil:
     visible:
       id: 'unread-badge-1'
@@ -31,6 +34,9 @@ tags:
     id: 'rooms-list-view-item-${output.otherUser.username}'
 - tapOn:
     id: header-back
+    retryTapIfNoChange: true
+- waitForAnimationToEnd:
+    timeout: 5000
 - extendedWaitUntil:
     notVisible:
       id: 'unread-badge-1'
@@ -44,6 +50,8 @@ tags:
     visible:
       id: 'rooms-list-view-item-${output.otherUser.username}'
     timeout: 60000
+- waitForAnimationToEnd:
+    timeout: 10000
 - extendedWaitUntil:
     visible:
       id: 'mention-badge-1'
@@ -54,6 +62,9 @@ tags:
     id: 'rooms-list-view-item-${output.otherUser.username}'
 - tapOn:
     id: header-back
+    retryTapIfNoChange: true
+- waitForAnimationToEnd:
+    timeout: 5000
 - extendedWaitUntil:
     notVisible:
       id: 'mention-badge-1'
@@ -68,6 +79,8 @@ tags:
     visible:
       id: 'rooms-list-view-item-${output.otherUser.username}'
     timeout: 60000
+- waitForAnimationToEnd:
+    timeout: 10000
 - extendedWaitUntil:
     visible:
       id: 'mention-badge-2'
@@ -78,6 +91,9 @@ tags:
     id: 'rooms-list-view-item-${output.otherUser.username}'
 - tapOn:
     id: header-back
+    retryTapIfNoChange: true
+- waitForAnimationToEnd:
+    timeout: 5000
 - extendedWaitUntil:
     notVisible:
       id: 'mention-badge-2'
@@ -92,6 +108,8 @@ tags:
     visible:
       id: 'rooms-list-view-item-${output.otherUser.username}'
     timeout: 60000
+- waitForAnimationToEnd:
+    timeout: 10000
 - extendedWaitUntil:
     visible:
       id: 'mention-badge-2'
@@ -102,6 +120,9 @@ tags:
     id: 'rooms-list-view-item-${output.otherUser.username}'
 - tapOn:
     id: header-back
+    retryTapIfNoChange: true
+- waitForAnimationToEnd:
+    timeout: 5000
 - extendedWaitUntil:
     notVisible:
       id: 'mention-badge-2'

.maestro/tests/teams/utils/close-action-sheet.yaml

@@ -6,7 +6,7 @@ tags:
 - swipe:
     from:
       id: action-sheet-handle
-    direction: down
+    direction: DOWN
 - extendedWaitUntil:
     notVisible:
       id: action-sheet-handle

app/containers/InAppNotification/index.tsx

@@ -59,7 +59,7 @@
 				componentProps: {
 					notification
 				},
-				duration: notification.customTime || process.env.RUNNING_E2E_TESTS ? 60000 : 3000, // default 3s,
+				duration: notification.customTime || (process.env.RUNNING_E2E_TESTS ? 5000 : 3000), // default 3s,
 				hideOnPress: notification.hideOnPress ?? true,
 				swipeEnabled: notification.swipeEnabled ?? true
 			});
@@ -71,7 +71,7 @@
 		return () => {
 			EventEmitter.removeListener(INAPP_NOTIFICATION_EMITTER, listener);
 		};
 	}, [subscribedRoom, appState]);
 
 	return <NotifierRoot />;
 });