Because it's in test code, I think it's less of an issue.


On Tue, May 6, 2014 at 11:33 AM, Julia Hansbrough
<notifications@github.com>wrote:

> In common/djangoapps/embargo/tests/test_middleware.py:
>
> > @@ -32,8 +32,8 @@ def setUp(self):
> >          self.embargo_course.save()
> >          self.regular_course = CourseFactory.create(org="Regular")
> >          self.regular_course.save()
> > -        self.embargoed_page = '/courses/' + self.embargo_course.id + '/info'
> > -        self.regular_page = '/courses/' + self.regular_course.id + '/info'
> > +        self.embargoed_page = '/courses/' + self.embargo_course.id.to_deprecated_string() + '/info'
> > +        self.regular_page = '/courses/' + self.regular_course.id.to_deprecated_string() + '/info'
>
> I think this is the issue of, common doesn't necessarily load urls.py, so
> we don't have a great reverse function to handle this case :(
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/3538/files#r12330720>
> .
>