kobotoolbox / kpi / 29418843892

82%

master: 76%

Build Type

push

github

Committed by web-flow

Commit Message

fix(longRunningMigrations): convert legacy googlets to automatic_google_transcription in LRM 0024 DEV-2094 (#7228)

### 📣 Summary

Fixes a background data upgrade so that older automatic-transcription
results are carried into the current format instead of stopping the
upgrade for an entire server.

### 📖 Description

The background task that upgrades legacy qualitative-analysis / NLP data
stopped with an error on some servers when a project still referenced a
pre-migration transcription result. It now converts those legacy results
into the current format and finishes the upgrade.

### 💭 Notes

Two public servers failed for two confirmed causes (read from the
`error` column via admin):

- **`KeyError: 'googlets'`** — a supplement still carried the legacy
action ID `googlets` (old automatic Google transcription). **Fixed
here:** before the qpath→xpath rewrite, any question still holding a
legacy action ID is run through the canonical
`migrate_submission_supplementals` converter, one question at a time. A
bare `googlets` (no `transcript` for the converter to rebuild from — a
valid "auto-transcribed, never manually transcribed" state) is rebuilt
into `automatic_google_transcription` using the converter's own helpers,
so the transcription is preserved, not dropped.
- **`'Asset' object has no attribute 'get_attachment_xpaths'`** —
**already fixed on `main`** by 3174cc0e0. No code change needed; it just
needs to re-run.

`0029_restart_failed_lrm_0024` resets the failed migration
`failed`→`created` on deploy so the corrected job re-runs on both
servers.

Scope: the affected records are a bounded, frozen set (11 global + 3 EU,
untouched since before 0023). Verified in-container: 3 tests (bare
`googlets`, `googlets`+`transcript`, qpath happy-path) green; existing
LRM suite green; `makemigrations --check` and `darker --check` clean.

### 👀 Preview steps

Background data migration; verified via tests, no UI.

1. ℹ️ a `SubmissionSupplement` whose content has a leg... (continued)

Coverage Stats

9411 of 12714 branches covered (74.02%)

0 of 28 new or added lines in 1 file covered. (0.0%)

324 existing lines in 20 files now uncovered.

30513 of 37348 relevant lines covered (81.7%)

5.52 hits per line