urls: Fix django-1.6 change in password change views

The expected name for the password change views no longer has the auth_
prefix. This change fixes this and adds a testcase.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

diff --git a/apps/patchwork/tests/test_user.py b/apps/patchwork/tests/test_user.py
index 08b9fa5791e948f81bd9464ffe0d75d3d9873d51..0faa97026a0c63811a01a970b60e5a94e6c1a178 100644 (file)
--- a/apps/patchwork/tests/test_user.py
+++ b/apps/patchwork/tests/test_user.py
@@ -155,3 +155,41 @@ class UserProfileTest(TestCase):
 
         self.assertContains(response, 'You have the following bundle')
         self.assertContains(response, bundle.get_absolute_url())
+
+class UserPasswordChangeTest(TestCase):
+    form_url = reverse('django.contrib.auth.views.password_change')
+    done_url = reverse('django.contrib.auth.views.password_change_done')
+
+    def testPasswordChangeForm(self):
+        self.user = TestUser()
+        self.client.login(username = self.user.username,
+                          password = self.user.password)
+
+        response = self.client.get(self.form_url)
+        self.assertContains(response, 'Change my password')
+
+    def testPasswordChange(self):
+        self.user = TestUser()
+        self.client.login(username = self.user.username,
+                          password = self.user.password)
+
+        old_password = self.user.password
+        new_password = User.objects.make_random_password()
+
+        data = {
+            'old_password': old_password,
+            'new_password1': new_password,
+            'new_password2': new_password,
+        }
+
+        response = self.client.post(self.form_url, data)
+        self.assertRedirects(response, self.done_url)
+
+        user = User.objects.get(id = self.user.user.id)
+
+        self.assertFalse(user.check_password(old_password))
+        self.assertTrue(user.check_password(new_password))
+
+        response = self.client.get(self.done_url)
+        self.assertContains(response,
+                "Your password has been changed sucessfully")

diff --git a/apps/patchwork/urls.py b/apps/patchwork/urls.py
index 41f01c1c803b280367c3cf9fb49a468264f4e3e0..9b681f9f3da1ed0fb05c448539be77c2acac42e6 100644 (file)
--- a/apps/patchwork/urls.py
+++ b/apps/patchwork/urls.py
@@ -45,9 +45,9 @@ urlpatterns = patterns('',
 
     # password change
     url(r'^user/password-change/$', auth_views.password_change,
-            name='auth_password_change'),
+            name='password_change'),
     url(r'^user/password-change/done/$', auth_views.password_change_done,
-            name='auth_password_change_done'),
+            name='password_change_done'),
 
     # login/logout
     url(r'^user/login/$', auth_views.login,