XMLModuleStore should use HiddenDescriptor instead of RawDescriptor.

In https://github.com/edx/edx-platform/pull/25955 `HiddenDescriptor`
(which was a subclass of `RawDescriptor` with a custom `student_view()`)
was converted to an XBlock. It is used as the `default_class` by the
`CachingDescriptorSystem` classes. However `RawDescriptor` is still
being used by `XMLModuleStore`. This has been replaced by
`HiddenDescriptor` as well.

cms/djangoapps/contentstore/tests/test_courseware_index.py

@@ -101,7 +101,7 @@ class MixedWithOptionsTestCase(MixedSplitTestCase):
     DATABASE = 'test_mongo_%s' % uuid4().hex[:5]
     COLLECTION = 'modulestore'
     ASSET_COLLECTION = 'assetstore'
-    DEFAULT_CLASS = 'xmodule.raw_module.RawDescriptor'
+    DEFAULT_CLASS = 'xmodule.hidden_module.HiddenDescriptor'
     RENDER_TEMPLATE = lambda t_n, d, ctx=None, nsp='main': ''
     modulestore_options = {
         'default_class': DEFAULT_CLASS,

cms/envs/test.py

@@ -98,7 +98,7 @@ BLOCK_STRUCTURES_SETTINGS['PRUNING_ACTIVE'] = True
 update_module_store_settings(
     MODULESTORE,
     module_store_options={
-        'default_class': 'xmodule.raw_module.RawDescriptor',
+        'default_class': 'xmodule.hidden_module.HiddenDescriptor',
         'fs_root': TEST_ROOT / "data",
     },
     doc_store_settings={

common/lib/xmodule/xmodule/modulestore/tests/django_utils.py

@@ -94,7 +94,7 @@ def draft_mongo_store_config(data_dir):
     """
 
     modulestore_options = {
-        'default_class': 'xmodule.raw_module.RawDescriptor',
+        'default_class': 'xmodule.hidden_module.HiddenDescriptor',
         'fs_root': data_dir,
         'render_template': 'common.djangoapps.edxmako.shortcuts.render_to_string'
     }
@@ -121,7 +121,7 @@ def split_mongo_store_config(data_dir):
     Defines split module store.
     """
     modulestore_options = {
-        'default_class': 'xmodule.raw_module.RawDescriptor',
+        'default_class': 'xmodule.hidden_module.HiddenDescriptor',
         'fs_root': data_dir,
         'render_template': 'common.djangoapps.edxmako.shortcuts.render_to_string',
     }

common/lib/xmodule/xmodule/modulestore/tests/test_mixed_modulestore.py

@@ -79,7 +79,7 @@ class CommonMixedModuleStoreSetup(CourseComparisonTest):
     COLLECTION = 'modulestore'
     ASSET_COLLECTION = 'assetstore'
     FS_ROOT = DATA_DIR
-    DEFAULT_CLASS = 'xmodule.raw_module.RawDescriptor'
+    DEFAULT_CLASS = 'xmodule.hidden_module.HiddenDescriptor'
     RENDER_TEMPLATE = lambda t_n, d, ctx=None, nsp='main': ''
 
     MONGO_COURSEID = 'MITx/999/2013_Spring'

common/lib/xmodule/xmodule/modulestore/tests/test_mongo.py

@@ -49,7 +49,7 @@ DB = 'test_mongo_%s' % uuid4().hex[:5]
 COLLECTION = 'modulestore'
 ASSET_COLLECTION = 'assetstore'
 FS_ROOT = DATA_DIR  # TODO (vshnayder): will need a real fs_root for testing load_item
-DEFAULT_CLASS = 'xmodule.raw_module.RawDescriptor'
+DEFAULT_CLASS = 'xmodule.hidden_module.HiddenDescriptor'
 RENDER_TEMPLATE = lambda t_n, d, ctx=None, nsp='main': ''
 
 

common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py

@@ -61,7 +61,7 @@ class SplitModuleTest(unittest.TestCase):
         'collection': 'modulestore',
     }
     modulestore_options = {
-        'default_class': 'xmodule.raw_module.RawDescriptor',
+        'default_class': 'xmodule.hidden_module.HiddenDescriptor',
         'fs_root': tempdir.mkdtemp_clean(),
         'xblock_mixins': (InheritanceMixin, XModuleMixin, EditInfoMixin)
     }

common/lib/xmodule/xmodule/modulestore/tests/test_split_w_old_mongo.py

@@ -42,7 +42,7 @@ class SplitWMongoCourseBootstrapper(unittest.TestCase):
     }
 
     modulestore_options = {
-        'default_class': 'xmodule.raw_module.RawDescriptor',
+        'default_class': 'xmodule.hidden_module.HiddenDescriptor',
         'fs_root': '',
         'render_template': mock.Mock(return_value=""),
         'xblock_mixins': (InheritanceMixin, XModuleMixin)

common/lib/xmodule/xmodule/modulestore/tests/utils.py

@@ -105,7 +105,7 @@ class MixedSplitTestCase(TestCase):
     """
     RENDER_TEMPLATE = lambda t_n, d, ctx=None, nsp='main': '{}: {}, {}'.format(t_n, repr(d), repr(ctx))
     modulestore_options = {
-        'default_class': 'xmodule.raw_module.RawDescriptor',
+        'default_class': 'xmodule.hidden_module.HiddenDescriptor',
         'fs_root': DATA_DIR,
         'render_template': RENDER_TEMPLATE,
         'xblock_mixins': (EditInfoMixin, InheritanceMixin, LocationMixin, XModuleMixin),

common/lib/xmodule/xmodule/modulestore/xml_importer.py

@@ -227,7 +227,7 @@ class ImportManager:
 
     def __init__(
             self, store, user_id, data_dir, source_dirs=None,
-            default_class='xmodule.raw_module.RawDescriptor',
+            default_class='xmodule.hidden_module.HiddenDescriptor',
             load_error_modules=True, static_content_store=None,
             target_id=None, verbose=False,
             do_import_static=True, do_import_python_lib=True,
@@ -1182,7 +1182,7 @@ def validate_course_policy(module_store, course_id):
 
 def perform_xlint(  # lint-amnesty, pylint: disable=missing-function-docstring
         data_dir, source_dirs,
-        default_class='xmodule.raw_module.RawDescriptor',
+        default_class='xmodule.hidden_module.HiddenDescriptor',
         load_error_modules=True,
         xblock_mixins=(LocationMixin, XModuleMixin)):
     err_cnt = 0

lms/djangoapps/courseware/tests/test_module_render.py

@@ -2504,7 +2504,7 @@ class TestDisabledXBlockTypes(ModuleStoreTestCase):
     def test_get_item(self, default_ms):
         with self.store.default_store(default_ms):
             course = CourseFactory()
-            self._verify_descriptor('video', course, 'RawDescriptorWithMixins')
+            self._verify_descriptor('video', course, 'HiddenDescriptorWithMixins')
 
     @ddt.data(ModuleStoreEnum.Type.mongo, ModuleStoreEnum.Type.split)
     def test_dynamic_updates(self, default_ms):
@@ -2519,7 +2519,7 @@ class TestDisabledXBlockTypes(ModuleStoreTestCase):
 
             # Now simulate a new request cache.
             self.store.request_cache.data.clear()
-            self._verify_descriptor('problem', course, 'RawDescriptorWithMixins', item_usage_id)
+            self._verify_descriptor('problem', course, 'HiddenDescriptorWithMixins', item_usage_id)
 
     def _verify_descriptor(self, category, course, descriptor, item_id=None):
         """