ReviewOS

also looking at this

craft-native/craft

test(ios): verify app termination before local notification delivery

#308
Merged glennmichael123 wants to merge codex/ios-notification-deadline into main
5 files +96 -21
scripts/mobile-e2e/protocol.tsmodified+5-1
Changes to scripts/mobile-e2e/protocol.ts
@@ -59,6 +59,11 @@ export type DriverEvent = PlanEvent | CaseEvent | DoneEvent | FatalEvent | Obser
5959
6060export type MobilePlatform = 'ios' | 'android'
6161
62// The local notification must survive UI discovery and synchronous app
63// termination before it fires. CI has spent 32 seconds in terminate() alone.
64export const LOCAL_NOTIFICATION_DELAY_MS = 120_000
65export const NOTIFICATION_TEST_TIMEOUT_MS = 300_000 + LOCAL_NOTIFICATION_DELAY_MS
66
6267/**
6368 * The cases each platform must run, named here and nowhere else.
6469 *
formatting only · 4 hidden lines
Changes to scripts/mobile-e2e/protocol.ts
@@ -867,4 +872,3 @@ export function androidDeclines(text: string): string[] { formatting only
867872 const pattern = new RegExp(`craft: (\\w+) (${phrases}) \\(([^)]*)\\)`, 'g')
868873 return [...plain.matchAll(pattern)].map(match => `${match[1]}: ${match[2]} (${match[3]})`)
869874}
870