Change the _test_no_locations logic to
re.findall(r"(?<!jump_to/)i4x://", content)
and remove the 2 jump_to searches.


On Fri, Nov 22, 2013 at 3:50 PM, Christina Roberts <notifications@github.com
> wrote:

> In cms/djangoapps/contentstore/tests/test_contentstore.py:
>
> > @@ -1355,6 +1346,19 @@ def test_export_course_without_content_store(self):
> >          items = module_store.get_items(stub_location)
> >          self.assertEqual(len(items), 1)
> >
> > +    def _check_verticals(self, items, course_id):
> > +        """ Test getting the editing HTML for each vertical. """
> > +        # Assert is here to make sure that the course being tested actually has verticals (units) to check.
> > +        self.assertGreater(len(items), 0)
> > +        for descriptor in items:
> > +            unit_locator = loc_mapper().translate_location(course_id, descriptor.location, False, True)
> > +            print "Checking {0}....".format(unicode(unit_locator))
>
> I am fine with deleting them-- don't know when/if they are useful. No
> logging in this file, so I'm not going to try to add that. But deletion
> sounds good.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/1728/files#r7868648>
> .
>