fix: remove general use of load_block_as_user
For the config model we do not need to go as far as binding the block to the user and already get enough data out of the modulestore to satisfy the storage on the xblock case. Add a new function to get that much xblock only. For the limited cases where we are using the block more directly as a block we maintain the old function. Also includes a fix to test_views that was closing the wrong level mock and leaving an open patch into other tests.
Showing
- CHANGELOG.rst 8 additions, 0 deletionsCHANGELOG.rst
- lti_consumer/__init__.py 1 addition, 1 deletionlti_consumer/__init__.py
- lti_consumer/api.py 0 additions, 5 deletionslti_consumer/api.py
- lti_consumer/models.py 25 additions, 41 deletionslti_consumer/models.py
- lti_consumer/plugin/compat.py 14 additions, 2 deletionslti_consumer/plugin/compat.py
- lti_consumer/plugin/views.py 3 additions, 2 deletionslti_consumer/plugin/views.py
- lti_consumer/tests/unit/plugin/test_views.py 6 additions, 4 deletionslti_consumer/tests/unit/plugin/test_views.py
- lti_consumer/tests/unit/plugin/test_views_lti_ags.py 4 additions, 7 deletionslti_consumer/tests/unit/plugin/test_views_lti_ags.py
- lti_consumer/tests/unit/plugin/test_views_lti_deep_linking.py 19 additions, 16 deletions...consumer/tests/unit/plugin/test_views_lti_deep_linking.py
- lti_consumer/tests/unit/plugin/test_views_lti_nrps.py 4 additions, 7 deletionslti_consumer/tests/unit/plugin/test_views_lti_nrps.py
- lti_consumer/tests/unit/test_api.py 1 addition, 2 deletionslti_consumer/tests/unit/test_api.py
- lti_consumer/tests/unit/test_lti_xblock.py 10 additions, 9 deletionslti_consumer/tests/unit/test_lti_xblock.py
- lti_consumer/tests/unit/test_models.py 7 additions, 28 deletionslti_consumer/tests/unit/test_models.py
Loading
Please register or sign in to comment