Removed some unnecessary mocking from tests

rapidpro · Mar 4, 2016 · f08893e · f08893e
1 parent deb888a
commit f08893e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/casepro/cases/tests.py b/casepro/cases/tests.py
@@ -532,10 +532,7 @@ def test_timeline(self, mock_create_outgoing, mock_fetch_contact_messages, mock_
 
 
 class HomeViewsTest(BaseCasesTest):
-    @patch('dash.orgs.models.TembaClient1.get_labels')
-    def test_inbox(self, mock_get_labels):
-        mock_get_labels.return_value = []
-
+    def test_inbox(self):
         url = reverse('cases.inbox')
 
         response = self.url_get('unicef', url)

diff --git a/casepro/profiles/tests.py b/casepro/profiles/tests.py
@@ -463,17 +463,12 @@ def test_self(self):
 
 class ForcePasswordChangeMiddlewareTest(BaseCasesTest):
     @override_settings(CELERY_ALWAYS_EAGER=True, CELERY_EAGER_PROPAGATES_EXCEPTIONS=True, BROKER_BACKEND='memory')
-    @patch('dash.orgs.models.TembaClient1.get_labels')
-    def test_process_view(self, mock_get_labels):
-        mock_get_labels.return_value = []
-
+    def test_process_view(self):
         self.user1.profile.change_password = True
         self.user1.profile.save()
 
         self.login(self.user1)
 
-        # TODO figure out why fetch_redirect_response=False became necessary after Dash update
-
         response = self.url_get('unicef', reverse('cases.inbox'))
         self.assertRedirects(response, 'http://unicef.localhost/profile/self/', fetch_redirect_response=False)