]> git.ozlabs.org Git - patchwork/commitdiff
tests/test_user: Add "profile POST" tests
authorStephen Finucane <stephen.finucane@intel.com>
Fri, 21 Aug 2015 14:32:13 +0000 (15:32 +0100)
committerDamien Lespiau <damien.lespiau@intel.com>
Thu, 17 Sep 2015 16:50:57 +0000 (17:50 +0100)
POSTing to the 'profile' view will allow configuration of some user
profiles, but this was not being tested. Resolve this.

Acked-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
patchwork/tests/test_user.py

index 61a844377bb6fd1e2bc4756bc5841d9e1882335e..9eeda7f7b8c83cf470a8bd103a0a92c31c6eb70c 100644 (file)
@@ -24,7 +24,7 @@ from django.core import mail
 from django.core.urlresolvers import reverse
 from django.conf import settings
 from django.contrib.auth.models import User
-from patchwork.models import EmailConfirmation, Person, Bundle
+from patchwork.models import EmailConfirmation, Person, Bundle, UserProfile
 from patchwork.tests.utils import defaults, error_strings
 
 def _confirmation_url(conf):
@@ -157,6 +157,27 @@ class UserProfileTest(TestCase):
         self.assertContains(response, 'You have the following bundle')
         self.assertContains(response, bundle.get_absolute_url())
 
+    def testUserProfileValidPost(self):
+        user_profile = UserProfile.objects.get(user=self.user.user.id)
+        old_ppp = user_profile.patches_per_page
+        new_ppp = old_ppp + 1
+
+        response = self.client.post('/user/', {'patches_per_page': new_ppp})
+
+        user_profile = UserProfile.objects.get(user=self.user.user.id)
+        self.assertEquals(user_profile.patches_per_page, new_ppp)
+
+    def testUserProfileInvalidPost(self):
+        user_profile = UserProfile.objects.get(user=self.user.user.id)
+        old_ppp = user_profile.patches_per_page
+        new_ppp = -1
+
+        response = self.client.post('/user/', {'patches_per_page': new_ppp})
+
+        user_profile = UserProfile.objects.get(user=self.user.user.id)
+        self.assertEquals(user_profile.patches_per_page, old_ppp)
+
+
 class UserPasswordChangeTest(TestCase):
     form_url = reverse('django.contrib.auth.views.password_change')
     done_url = reverse('django.contrib.auth.views.password_change_done')