From 20a499baa52d05db8f52fc85990e29baaf6a6701 Mon Sep 17 00:00:00 2001 From: Jinwoo-H Date: Wed, 2 Sep 2026 02:07:21 -0400 Subject: [PATCH] test(mobile-e2e): name both landmarks when screenshot parity fails The failure printed only landmarkDelta, which cannot say which side moved. The two landmarks come from different sources, a native accessibility frame and a hosted DOM rect, so the absolute pair is what makes the next failure readable. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb --- mobile/scripts/hosted-ios-screenshot-parity.mjs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/mobile/scripts/hosted-ios-screenshot-parity.mjs b/mobile/scripts/hosted-ios-screenshot-parity.mjs index 38ca62a926b..5af7554e0e6 100644 --- a/mobile/scripts/hosted-ios-screenshot-parity.mjs +++ b/mobile/scripts/hosted-ios-screenshot-parity.mjs @@ -35,6 +35,10 @@ export async function assertHostedIosScreenshotParity({ const result = { ...measurements, landmarkDelta, + // Why: a delta alone cannot say which side moved, and the two landmarks come + // from different sources (native accessibility frame vs hosted DOM rect). + landmarks: + nativeLandmark && hostedLandmark ? { native: nativeLandmark, hosted: hostedLandmark } : null, budgets: { changedPixelRatio: MAX_CHANGED_PIXEL_RATIO, meanChannelDifference: MAX_MEAN_CHANNEL_DIFFERENCE,