kobotoolbox / kpi / 25859849859

82%

master: 76%

Build Type

push

github

Committed by web-flow

Commit Message

fix(subsequences): use correct method for getting xpath DEV-2094 (#7051)

### 📣 Summary
Fixes a bug in the previous migration that was causing it to exit with
an error.

### 💭 Notes
Also replaces all print statements with log statements for better
visibility.


### 👀 Preview steps
It's a pill to actually get old data to test this, but you can make mock
data that reasonably acts like old data. Make sure to start on the
release branch so all supplements will have the old version.

1. ℹ️ have an account and a project with an audio question inside a
group
2. ℹ️ have at least one submission with a transcript
3. Add a QA question and answer (manual)
4. Open a django shell
5. 
```
asset = Asset.objects.get(uid='<uid>')
ss = SubmissionSupplement.objects.get(submission_uuid='<uuid>', asset=asset)
ss_content_copy = copy.deepcopy(ss.content)
for key in [ k for k in ss.content.keys() if '/' in k]:
        ss_content_copy[key.replace('/','-')] = copy.deepcopy(ss_content_copy[key])
        del ss_content_copy[key]
ss.content = ss_content_copy
ss.save()
```
8. Run LRM 24
9. 🟢 [on release] Fails with an AttributeError
11. 🟢 [on PR] migration succeeds

Coverage Stats

7801 of 12051 branches covered (64.73%)

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

1 existing line in 1 file now uncovered.

29638 of 36178 relevant lines covered (81.92%)

5.77 hits per line