From: Stephen Finucane <stephen@that.guru>
Date: Mon, 10 Sep 2018 23:11:46 +0000 (-0600)
Subject: tests: Remove useless todo
X-Git-Tag: v2.2.0-rc1~287
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1830989c9c297453c4a27c6eacaefb54444a7074;p=thirdparty%2Fpatchwork.git

tests: Remove useless todo

I'm not sure why I included this in day one: we _want_ to use the
context manager like this.

Signed-off-by: Stephen Finucane <stephen@that.guru>
---

diff --git a/patchwork/tests/test_about.py b/patchwork/tests/test_about.py
index 956d0324..ba8fd982 100644
--- a/patchwork/tests/test_about.py
+++ b/patchwork/tests/test_about.py
@@ -41,9 +41,6 @@ class AboutViewTest(TestCase):
             self.assertTrue(response.context['enabled_apis']['xmlrpc'])
 
     def test_rest(self):
-        # TODO(stephenfin): There appears to be a bug in Django 1.10.x under
-        # Python 3.5, meaning we can't use 'override_settings' here or we cause
-        # the REST API tests to fail. We should investigate this.
         with self.settings(ENABLE_REST_API=False):
             response = self.client.get(reverse('about'))
             self.assertFalse(response.context['enabled_apis']['rest'])