From 776957e97196c5b86b467abf45a9eb2adda87d02 Mon Sep 17 00:00:00 2001
From: Todd Dembrey <todd.dembrey@torchbox.com>
Date: Thu, 19 Jul 2018 08:51:40 +0100
Subject: [PATCH] Make sure we handle the test case using production settings
---
opentech/apply/users/tests/test_views.py | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/opentech/apply/users/tests/test_views.py b/opentech/apply/users/tests/test_views.py
index 443f007fa..935dcdc98 100644
--- a/opentech/apply/users/tests/test_views.py
+++ b/opentech/apply/users/tests/test_views.py
@@ -17,20 +17,21 @@ class TestProfileView(BaseTestProfielView):
def test_cant_acces_if_not_logged_in(self):
self.client.logout()
- response = self.client.get(self.url)
- self.assertRedirects(response, reverse('users:login') + '?next=' + self.url)
+ response = self.client.get(self.url, follow=True)
+ # Initial redirect will be via to https through a 301
+ self.assertRedirects(response, reverse('users:login') + '?next=' + self.url, status_code=301)
def test_includes_change_password(self):
- response = self.client.get(self.url)
+ response = self.client.get(self.url, follow=True)
self.assertContains(response, reverse('users:password_change'))
def test_doesnt_includes_change_password_for_oauth(self):
self.client.force_login(OAuthUserFactory())
- response = self.client.get(self.url)
+ response = self.client.get(self.url, follow=True)
self.assertNotContains(response, reverse('users:password_change'))
def test_cant_set_slack_name(self):
- response = self.client.get(self.url)
+ response = self.client.get(self.url, follow=True)
self.assertNotContains(response, 'Slack name')
@@ -40,5 +41,5 @@ class TestStaffProfileView(BaseTestProfielView):
self.client.force_login(self.staff)
def test_can_set_slack_name(self):
- response = self.client.get(self.url)
+ response = self.client.get(self.url, follow=True)
self.assertContains(response, 'Slack name')
--
GitLab