0cac46cb36
Turns the manual QA ask ("verify DiffView + CodeBlock horizontal-scroll
on-device with a populated diff") into two automated layers:
1. Unit (deterministic, runs in `npm test` now): extracted the shared
ScrollView props into src/lib/scroll-config.ts (WIDE_CONTENT_SCROLL_CONFIG)
so DiffView.tsx and CodeBlock.tsx spread the SAME plain object their tests
assert on — no react-native-renderer needed. Added a source-scan
regression test (wide-content-scroll.regression.test.ts) that fails if
either component loses its ScrollView wiring or reintroduces
numberOfLines truncation.
2. E2E (Maestro): .maestro/flows/diff-scroll.yaml opens a session with a
pre-seeded wide edit-diff tool call and a wide fenced code block, then
swipes each horizontal ScrollView left and asserts the off-screen marker
text becomes visible. mock-opencode-server.ts gained a --seed-diff mode
that serves this session via GET /session/:id/message (pre-existing
history), not SSE — issue #90 (a separate SSE-render bug) is being fixed
independently, and this flow must not depend on it landing first. Wired
the new flow + port 4100 into run-e2e-flows.sh and activation-e2e.yml.
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
97 lines
4.0 KiB
Bash
Executable File
97 lines
4.0 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
#
|
|
# Runs the Maestro activation E2E flows against the mock opencode servers the
|
|
# workflow started on the host (ports 4096-4100). Invoked as a single line from
|
|
# activation-e2e.yml's emulator-runner `script:` so cd/trap/loop actually work.
|
|
#
|
|
# Networking: `adb reverse` maps each mock port so the emulator's own
|
|
# localhost:PORT forwards to the host's localhost:PORT; flows connect to
|
|
# 127.0.0.1:<port>. (10.0.2.2 was unreliable headless.)
|
|
set -uo pipefail
|
|
|
|
ROOT="$(pwd)" # capture BEFORE any cd, so diag paths are absolute
|
|
APK="android/app/build/outputs/apk/release/app-release.apk"
|
|
FLOWS=(activation-positive activation-negative-401 directory-picker all-sessions variant-picker diff-scroll)
|
|
mkdir -p "$ROOT/artifacts/screenshots" "$ROOT/artifacts/diag"
|
|
|
|
echo "== installing APK =="
|
|
adb install "$APK"
|
|
|
|
echo "== forwarding mock ports into the emulator (adb reverse) =="
|
|
for p in 4096 4097 4098 4099 4100; do adb reverse "tcp:$p" "tcp:$p"; done
|
|
adb reverse --list
|
|
|
|
# Decisive probe: can the EMULATOR actually reach the host mock via 127.0.0.1?
|
|
# If any in-emulator method prints the health JSON, the network path is good
|
|
# and any flow failure is app-side; if none succeed, attribution falls back to
|
|
# a host-side check + adb reverse state so we at least know the mock is up.
|
|
# `toybox wget`/`nc` don't work reliably on the API-28 image (wget: unknown
|
|
# command; nc connects but the request/response framing is unreliable) — so
|
|
# this tries curl (if present), then mksh's /dev/tcp pseudo-device (a shell
|
|
# builtin, not an external binary, so it survives whatever toybox applets
|
|
# this image did/didn't build), then nc as a last in-emulator attempt, before
|
|
# falling back to a host-side confirmation. This never blocks the flow — the
|
|
# result is recorded and the script continues regardless.
|
|
PROBE_FILE="$ROOT/artifacts/diag/probe.txt"
|
|
probe_result="UNKNOWN"
|
|
|
|
{
|
|
echo "== emulator -> mock reachability probe (127.0.0.1:4096/global/health) =="
|
|
} > "$PROBE_FILE"
|
|
|
|
probe_via() {
|
|
# $1 = human label for the log, $2 = remote shell command string
|
|
local label="$1" cmd="$2" out
|
|
echo "[$label]" | tee -a "$PROBE_FILE"
|
|
out="$(timeout 10 adb shell "$cmd" 2>&1)"
|
|
echo "$out" | tee -a "$PROBE_FILE"
|
|
[[ "$out" == *'"healthy"'* ]]
|
|
}
|
|
|
|
if probe_via "curl" 'curl -s -m 5 http://127.0.0.1:4096/global/health'; then
|
|
probe_result="PASS (curl)"
|
|
elif probe_via "toybox-wget" 'toybox wget -qO - http://127.0.0.1:4096/global/health'; then
|
|
probe_result="PASS (wget)"
|
|
elif probe_via "mksh-devtcp" 'exec 3<>/dev/tcp/127.0.0.1/4096 && printf "GET /global/health HTTP/1.0\r\n\r\n" >&3 && cat <&3'; then
|
|
probe_result="PASS (/dev/tcp)"
|
|
elif probe_via "nc" 'printf "GET /global/health HTTP/1.0\r\n\r\n" | toybox nc -w 3 127.0.0.1 4096'; then
|
|
probe_result="PASS (nc)"
|
|
else
|
|
echo "[fallback: host-side confirmation]" | tee -a "$PROBE_FILE"
|
|
host_check="$(timeout 5 curl -s http://127.0.0.1:4096/global/health 2>&1 || true)"
|
|
reverse_list="$(adb reverse --list 2>&1 || true)"
|
|
{
|
|
echo "host curl: $host_check"
|
|
echo "adb reverse --list: $reverse_list"
|
|
} | tee -a "$PROBE_FILE"
|
|
if [[ "$host_check" == *'"healthy"'* ]]; then
|
|
probe_result="UNKNOWN (host mock is up, but no in-emulator method could confirm emulator-side reachability)"
|
|
else
|
|
probe_result="FAIL (host mock itself is not responding on 127.0.0.1:4096)"
|
|
fi
|
|
fi
|
|
|
|
echo "== probe result: $probe_result ==" | tee -a "$PROBE_FILE"
|
|
|
|
adb logcat -c || true # clear, so the captured log is just this run
|
|
|
|
dump_diag() {
|
|
echo "== capturing diagnostics =="
|
|
adb logcat -d > "$ROOT/artifacts/diag/logcat.txt" 2>&1 || true
|
|
# Maestro writes per-run debug (UI hierarchy + commands) under ~/.maestro/tests
|
|
cp -r "$HOME/.maestro/tests" "$ROOT/artifacts/diag/maestro-tests" 2>/dev/null || true
|
|
}
|
|
trap dump_diag EXIT
|
|
|
|
cd "$ROOT/artifacts/screenshots"
|
|
rc=0
|
|
for f in "${FLOWS[@]}"; do
|
|
echo "--- flow: $f ---"
|
|
if ! maestro test --debug-output "$ROOT/artifacts/diag/maestro-$f" "$ROOT/.maestro/flows/$f.yaml"; then
|
|
echo "::error::Maestro flow failed: $f"
|
|
rc=1
|
|
break
|
|
fi
|
|
done
|
|
exit $rc
|