fix: e2e tests

Author: YogendraShelke

.github/workflows/NativePipeline.yml

@@ -105,7 +105,7 @@ env:
   # Pin the test project (mendix/Native-Mobile-Resources) to a SHA for reproducibility instead
   # of tracking a moving `main`. Single source of truth — bump deliberately. The repo publishes
   # no tags, so a commit SHA is the only stable ref. Override per run via the dispatch input.
-  NATIVE_MOBILE_RESOURCES_REF: ${{ github.event.inputs.test_project_ref || 'a915484ef2cfed403cfbdfa531638df3c46c9d00' }}
+  NATIVE_MOBILE_RESOURCES_REF: ${{ github.event.inputs.test_project_ref || '05255f5ad19de7b109f6a68088e4fb198447ed00' }}
 permissions:
   packages: write
 jobs:

maestro/helpers/helpers.sh

@@ -78,9 +78,24 @@ stop_recording() {
   REC_FILE=""
 }
 
+# Function to clean up stale XCTest processes (iOS only)
+cleanup_xctest_processes() {
+    if [ "$PLATFORM" != "ios" ]; then
+        return 0
+    fi
+    echo "🧹 Cleaning up stale XCTest processes..."
+    # Kill any leftover XCTest runner processes that might be blocking driver initialization
+    pkill -9 -f "XCTRunner" 2>/dev/null || true
+    pkill -9 -f "xctest" 2>/dev/null || true
+    pkill -9 -f "maestro.*driver" 2>/dev/null || true
+    sleep 2
+}
+
 # Function to restart the iOS simulator
 restart_simulator() {
     echo "🔄 Restarting iOS Simulator..."
+    # Clean up XCTest processes first
+    cleanup_xctest_processes
     # Shut down whatever is booted; the device is auto-selected in prepare_ios.sh,
     # so we don't depend on a hardcoded device name here.
     xcrun simctl shutdown all || true

maestro/helpers/prepare_ios.sh

@@ -64,6 +64,34 @@ elif best:
     xcrun simctl bootstatus "$DEVICE_ID" -b || echo "Simulator booted"
 }
 
+# Cut UIKit animation time, mirroring `disable-animations: true` on the Android emulator
+# (NativePipeline.yml) — iOS previously had no equivalent, so every navigation push/modal
+# played at full length on a 3-vCPU runner while Android's ran instantly.
+#
+# Scope: this shortens SYSTEM transitions (nav pushes, modal presents, alerts). React
+# Native's `Animated` does NOT read this flag automatically, so widget-internal animations
+# (carousel, animation-native) are unaffected — this is not a substitute for the per-flow
+# waits those need.
+#
+# Requires a BOOTED device (`simctl spawn`), so call after start_simulator. The value is
+# written to the device's persistent domain and survives shutdown/boot, so a
+# restart_simulator in helpers.sh keeps it.
+reduce_motion() {
+    echo "Reducing motion on iOS Simulator..."
+    if [ -z "$SIMULATOR_DEVICE_ID" ]; then
+        echo "Error: SIMULATOR_DEVICE_ID not set"
+        return 1
+    fi
+    # Best-effort: a future runtime that drops/renames these keys should not fail the shard,
+    # since tests still pass with animations on (just slower).
+    xcrun simctl spawn "$SIMULATOR_DEVICE_ID" defaults write com.apple.Accessibility ReduceMotionEnabled -bool true \
+        || echo "::warning::Could not set ReduceMotionEnabled; animations stay at full length"
+    # Replaces the sliding push/pop transition with a cross-fade — the slide is what Maestro
+    # most often races against when asserting right after a navigation tap.
+    xcrun simctl spawn "$SIMULATOR_DEVICE_ID" defaults write com.apple.Accessibility ReduceMotionReduceSlideTransitionsEnabled -bool true \
+        || echo "::warning::Could not set ReduceMotionReduceSlideTransitionsEnabled"
+}
+
 set_status_bar() {
     echo "Setting status bar on iOS Simulator..."
     if [ -z "$SIMULATOR_DEVICE_ID" ]; then
@@ -92,6 +120,7 @@ verify_installed_app() {
 }
 
 start_simulator
+reduce_motion
 set_status_bar
 install_ios_app
 verify_installed_app