I think that the existing `override_settings` will cause the settings to be
reset, even though we're manipulating it inside the test rather than via
the decorator.


On Thu, Jun 6, 2013 at 1:16 PM, Jay Zoldak <notifications@github.com> wrote:

> In lms/djangoapps/django_comment_client/base/tests.py:
>
> >      def setUp(self):
> > +        settings.MITX_FEATURES['ENABLE_DISCUSSION_SERVICE'] = True
> > +
>
> I'm not sure myself, but it seems that this wouldn't get set back to False
> after the testcases in this class are executed?
> I would think that this should instead be done in an @override_settingsline, no?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/92/files#r4572458>
> .
>