BB-4230 Fix usage of yaml.safe_load in migration
This fixes the migration introduced in #806
I think the bug was missed in tests because EDXAPP_FEATURES
is never used, so the code path in question is never exercised...
Testing
- Apply all previous migrations
./manage.py migrate instance 0140
; - Create an
OpenEdXInstance
withconfiguration_extra_settings = 'EDXAPP_FEATURES: { TEST_FLAG: true }'
; Alternatively, use any environment that contains instances whereconfiguration_extra_settings
containsEDXAPP_FEATURES
; - Apply migration:
./manage.py migrate instance
; - Verify that:
- before the fix it fails with
AttributeError: module 'yaml' has no attribute 'yaml'
- after the fix it succeeds
- before the fix it fails with
Related tickets: