Skip to content

Fix Fossify Messages GUI fixture normalization#9

Open
panDing19 wants to merge 1 commit into
LiberCoders:mainfrom
panDing19:fix/fossify-messages-fixture-normalization
Open

Fix Fossify Messages GUI fixture normalization#9
panDing19 wants to merge 1 commit into
LiberCoders:mainfrom
panDing19:fix/fossify-messages-fixture-normalization

Conversation

@panDing19

@panDing19 panDing19 commented Jun 25, 2026

Copy link
Copy Markdown

Problem

TGUI46's Fossify Messages fixture mixes flat SMS rows with threaded message rows. The initializer only handled one shape at a time, so mixed fixtures could fail normalization or lose message content before SQLite injection.

Affected tasks

  • TGUI46_job_failure_triage_workmail_sms_notes

Fix

  • Normalize message ids, timestamps, text fields, and outgoing status across the observed field aliases.
  • Normalize threaded rows independently.
  • Group flat rows by thread_id while preserving mixed flat/threaded fixture order.
  • Add a regression test using the real TGUI46 Fossify Messages fixture.

Tests

  • uv run pytest tests/test_mobile_gui_init.py -q

@panDing19 panDing19 marked this pull request as ready for review June 25, 2026 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant