Yes, because block/block/mit.eecs is legitimately course_id='block',
block='mit.eecs'


On Mon, Oct 21, 2013 at 11:08 AM, Christina Roberts <
notifications@github.com> wrote:

> In common/lib/xmodule/xmodule/modulestore/tests/test_locators.py:
>
> > @@ -77,19 +77,19 @@ def test_course_constructor_bad_course_id(self):
> >          """
> >          for bad_id in (' mit.eecs',
> >                         'mit.eecs ',
> > -                       URL_VERSION_PREFIX + 'mit.eecs',
> > -                       BLOCK_PREFIX + 'block/mit.eecs',
> > +                       VERSION_PREFIX + 'mit.eecs',
> > +                       BLOCK_PREFIX + 'black/mit.eecs',
>
> Is this intentionally "black" instead of "block"?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/1416/files#r7096334>
> .
>