Skip to content

test(mobile): pin Arabic rendering of the account settings screen - #6342

Open
iscekic wants to merge 1 commit into
mainfrom
kwf/explorer-account-settings-arabic-the-account-screen-is-co-fe2e1-cf99
Open

iscekic wants to merge 1 commit into
mainfrom
kwf/explorer-account-settings-arabic-the-account-screen-is-co-fe2e1-cf99

Conversation

@iscekic

@iscekic iscekic commented Sep 19, 2026

Copy link
Copy Markdown
Collaborator

Changelog for users

  • The Arabic Account settings screen keeps rendering its title, three rows, and the tab bar; no user-visible change.

Changelog for maintainers

  • The blank-screen finding did not reproduce: the screen has no language or direction branch, and the device capture shows the full Arabic UI.
  • Adds a mounted-render test that pins the Arabic header title, all three row titles, and the Language row subtitle.
  • The test keeps the Arabic Language row opening the language picker and calling the picker bridge once.
  • The test mocks React Native and screen primitives as string hosts, so assertions read component props.
  • Review hints: the test forces the language preference to ar and a fixed user id; no product code changed.

E2E proof

[e1] account-settings-arabic: the Account screen is blank — android emulator-5554, cold launch in Arabic: recorded run reports 'SCENE e1 OK' and the digest shows the Account settings screen rendered with 'الحساب', 'اللغة, العربية', 'المضيفون الموثوقون, مضيفون تسمح بفتح روابطهم دون طلب تأكيد', 'جلسات الأجهزة, مراجعة الأجهزة الأخرى وتسجيل الخروج منها' and the tab bar ('الملف الشخصي، تبويب، 3 من 3') — no blank render; screenshot e1.png captured for the visual reviewer, replay saved to e1.replay.json.

[e1] account-settings-arabic: the Account screen is blank — e2e-mobile-app/e1.png

E2E proof — log excerpts

[e1] account-settings-arabic: the Account screen is blank -> pass :: android emulator-5554, cold launch in Arabic: recorded run reports 'SCENE e1 OK' and the digest shows the Account settings screen rendered with 'الحساب', 'اللغة, العربية', 'المضيفون الموثوقون, مضيفون تسمح بفتح روابطهم دون طلب تأكيد', 'جلسات الأجهزة, مراجعة الأجهزة الأخرى وتسجيل الخروج منها' and the tab bar ('الملف الشخصي، تبويب، 3 من 3') — no blank render; screenshot e1.png captured for the visual reviewer, replay saved to e1.replay.json.
/home/igor_kilocode_ai/.local/share/kwf/sections/explorer-account-settings-arabic-the-account-screen-is-co-fe2e1-cf99/e2e-mobile-app/e1-run3.log
SCENE e1 OK
android.widget.LinearLayout com.kilocode.kiloapp:id/action_bar_root tappable [0,0][1080,2400]
android.widget.FrameLayout android:id/content tappable [0,0][1080,2400]
android.widget.Button رجوع tappable [942,149][1043,250]
android.view.View الحساب tappable [37,167][970,232]
android.widget.Button اللغة, العربية tappable [55,315][1025,459]
android.widget.TextView اللغة tappable [147,343][891,389]
android.widget.TextView العربية tappable [147,393][891,430]
android.widget.Button المضيفون الموثوقون, مضيفون تسمح بفتح روابطهم دون طلب تأكيد tappable [55,486][1025,630]
android.widget.TextView المضيفون الموثوقون tappable [147,514][891,560]
android.widget.TextView مضيفون تسمح بفتح روابطهم دون طلب تأكيد tappable [147,564][891,601]
android.widget.Button جلسات الأجهزة, مراجعة الأجهزة الأخرى وتسجيل الخروج منها tappable [55,658][1025,800]
android.widget.TextView جلسات الأجهزة tappable [147,686][891,732]
android.widget.TextView مراجعة الأجهزة الأخرى وتسجيل الخروج منها tappable [147,736][891,773]
android.view.View الرئيسية، تبويب، 1 من 3 tappable [720,2195][1080,2337]
android.widget.TextView الرئيسية tappable [733,2281][1067,2330]
android.view.View الوكلاء، تبويب، 2 من 3 tappable [360,2195][720,2337]
android.widget.TextView الوكلاء tappable [373,2281][707,2330]
android.view.View الملف الشخصي، تبويب، 3 من 3 tappable [0,2195][360,2337]
android.widget.TextView الملف الشخصي tappable [13,2281][347,2330]
Owner request

Explorer finding: account-settings-arabic: The Account screen is completely blank — no title, no rows, no tab bar — so the Arabic settings content the user navigated to is missing from the render.

The user-agent explorer found this while using the app like a user.
One finding per item; the explorer never edits product code.

Flow: account-settings-arabic
Found on revision: fd04ca1

Repro:

  1. open the app on emulator-5602
  2. reach the screen the capture names (account-settings-arabic)
  3. the capture shows the defect named below

Observed: The Account screen is completely blank — no title, no rows, no tab bar — so the Arabic settings content the user navigated to is missing from the render.
Expected: the screen renders without this defect

Evidence (from the device run):

Follow-ups (not changed here)

  • not proved live: Observed: The Account screen is completely blank — no title, no rows, no tab bar — so the Arabic settings content the user navigated to is missing from the render. Expected: the screen renders without this defect (no capture cited it)

Open findings (not fixed here)

  • not proved live: account-settings-arabic.png is no longer on the host that took it, so no publish can carry it

Explorer finding: account-settings-arabic: The Account screen is completely blank — no title, no rows, no tab bar — so the Arabic settings content the user navigated to is missing from the render.

The user-agent explorer found this while using the app like a user.
One finding per item; the explorer never edits product code.

Flow: account-settings-arabic
Found on revision: fd04ca1

Repro:
1. open the app on emulator-5602
2. reach the screen the capture names (account-settings-arabic)
3. the capture shows the defect named below

Observed: The Account screen is completely blank — no title, no rows, no tab bar — so the Arabic settings content the user navigated to is missing from the render.
Expected: the screen renders without this defect

Evidence (from the device run):
- ~/.local/share/kwf/findings/explorer-account-settings-arabic-the-account-screen-is-co-fe2e15fe/account-settings-arabic.png
@kilo-code-bot

kilo-code-bot Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Test-only change: the new mounted test correctly pins the Arabic rendering of the account settings screen, and its mocked i18n/language-preference setup matches the component contract.

Files Reviewed (1 file)
  • apps/mobile/src/components/account-settings-screen.arabic.mounted.test.tsx

Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0

Review guidance: REVIEW.md from base branch main

@iscekic

iscekic commented Sep 19, 2026

Copy link
Copy Markdown
Collaborator Author

This description names a scenario the proof did not capture:

  • not proved live: Observed: The Account screen is completely blank — no title, no rows, no tab bar — so the Arabic settings content the user navigated to is missing from the render. Expected: the screen renders without this defect (no capture cited it)

A repeated proof run rebuilds the same evidence, so no proof run is dispatched for a named gap. Merging with this gap open is your decision.

@iscekic iscekic added the human-ready The PR is ready for human review. label Sep 19, 2026
@iscekic iscekic self-assigned this Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

human-ready The PR is ready for human review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant