@@ -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
@@ -158,6 +158,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):
user = None
POSTing to the 'profile' view will allow configuration of some user profiles, but this was not being tested. Resolve this. Signed-off-by: Stephen Finucane <stephen.finucane@intel.com> --- patchwork/tests/test_user.py | 23 ++++++++++++++++++++++- 1 file changed, 22 insertions(+), 1 deletion(-)