and the test's intent (I believe) was to ensure it was order sensitive and
you couldn't specify course id after specifying the block; that is, it
wouldn't parse as block='block', course_id = 'mit.eecs'


On Mon, Oct 21, 2013 at 11:10 AM, Don-edX Mitchell <dmitchell@edx.org>wrote:

> 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>
>> .
>>
>
>