Discussion is restricted to the CCX course (there is no sharing of
discussions with the parent course or other CCX instances).

We're often grabbing the metadata of a specific thread to then be able
to perform other operations, but we never need the actual responses or
comments of a thread unless we're displaying it in the normal forum
view.

This change sets a default of with_responses=False, which instructs the
comment service to not send back the responses/comments for the given
thread.  We only ask for responses in the case of rendering a single
thread or inline discussion.

TNL-5002

2. Update COMPREHNSIVE_THEME_DIR to COMPREHENSIVE_THEME_DIRS
3. Update paver commands to support multi theme dirs
4. Updating template loaders
5. Add ENABLE_COMPREHENSIVE_THEMING flag to enable or disable theming via settings
6. Update tests
7. Add backward compatibility for COMPREHEHNSIVE_THEME_DIR

* Renames discussion_module to discussion_xblock
* Moves common/lib/xmodule/xmodule_discussion to openedx/core/lib/xblock_builtin/xblock_discussion

Reset current language after discussion tests, so that other languages don't leak into tests that don't use the LocaleMiddleware

TNL-4622

By default, disable all caching in tests, to preserve test independence.
In order to enable caching, inherit from CacheSetupMixin, and specify
which cache configuration is needed.

[EV-32]

Instead, use a class attribute to define test behavior. This allows for
easier addition of new mixins over time.

More accurately, this creates a 4th shard because the
last shard is always the default.

Revert "SUST-35 Implementation of course_key_from_string_or_404 to return course_key or raise a 404"

TEST_DATA_MIXED_MODULESTORE
Remove these test mixed modulestores:
TEST_DATA_MIXED_TOY_MODULESTORE
TEST_DATA_MIXED_CLOSED_MODULESTORE
TEST_DATA_MIXED_GRADED_MODULESTORE

More accurately, this creates a 4th shard because the
last shard is always the default.

Revert "SUST-35 Implementation of course_key_from_string_or_404 to return course_key or raise a 404"

-Adressed all comments

-Updated method "course_key_from_string_or_404" to raise message too.
-Wrote tests for "course_key_from_string_or_404" when exception message is given.
-Modified existing methods to use "ddt.data".
-Used Splunk logs to find exectly where we were getting "Invalid Key Error"
-Updated Views where we were getting "Invalid Key Error" in splunk logs.
-Wrote tests for those View End points where we were getting "Invalid Key Error"

TEST_DATA_MIXED_MODULESTORE
Remove these test mixed modulestores:
TEST_DATA_MIXED_TOY_MODULESTORE
TEST_DATA_MIXED_CLOSED_MODULESTORE
TEST_DATA_MIXED_GRADED_MODULESTORE

Prevents start date alone from being used to filter out categories in self-paced courses during construction of the category map. ECOM-4017.

Prevents start date alone from being used to filter out categories in self-paced courses during construction of the category map. ECOM-4017.

TNL-4284

This will leverage the nose attrib plugin to allow us
to break tests apart on our build system to run
in parallel on separate nodes.

Also accelerates forums UnicodeTestCases