Ok, I also agree this file is a dumping ground, poorly structured, and
misnamed. Contentstore is really the asset repo not the xblock/course repo.


On Thu, Oct 24, 2013 at 12:18 PM, Christina Roberts <
notifications@github.com> wrote:

> In cms/djangoapps/contentstore/tests/test_contentstore.py:
>
> > @@ -1933,3 +1913,22 @@ def test_metadata_persistence(self):
> >          # TODO: create the same test as `test_metadata_not_persistence`,
> >          # but check persistence for some other module.
> >          pass
> > +
> > +
> > +def _create_course(test, course_data):
>
> Because *create*course was called from 2 different top-level test
> classes. This test file is sort of a mess-- things really need to be split
> up better and some inheritance established. But for now, making *create*course
> top-level (and because of that, *get*course_id) allowed me to call it
> from tests in both classes.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/1470/files#r7192665>
> .
>