Skip to content

Commit d68f458

Browse files
committed
fix(ci): exclude iosOnly flows from Android pre-filter
The Android pre-filter was only excluding "flakyAndroid" on top of the user-supplied EXCLUDE_TAGS — it never excluded the platform-scope tag "iosOnly", so any flow tagged iosOnly was being run on the Android emulator anyway. Caught by the new google-sso-from-forgot-password flow, which is correctly tagged iosOnly but failed on Android because the system OAuth presentation strings ("Wants to Use" / "accounts.google.com" / "Continue") never appear there. iOS already had the symmetric guard ("$EXCLUDE_TAGS,androidOnly").
1 parent b8b9742 commit d68f458

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/mobile-e2e.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -561,7 +561,7 @@ jobs:
561561
script: >-
562562
adb install -r /tmp/cached-app-release.apk &&
563563
cd integration/mobile &&
564-
excluded="$EXCLUDE_TAGS,flakyAndroid" &&
564+
excluded="$EXCLUDE_TAGS,iosOnly,flakyAndroid" &&
565565
pattern="$(echo "$excluded" | sed 's/,/|/g')" &&
566566
find flows -type f -name '*.yaml' ! -path '*/common/*' ${FLOWS_FILTER:+-path "*$FLOWS_FILTER*"}
567567
| while read f; do grep -qE "^[[:space:]]*-[[:space:]]*(${pattern})[[:space:]]*$" "$f" || printf '%s\n' "$f"; done

0 commit comments

Comments
 (0)