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 <[email protected]> --- patchwork/tests/test_user.py | 23 ++++++++++++++++++++++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/patchwork/tests/test_user.py b/patchwork/tests/test_user.py index e791095..b35593a 100644 --- a/patchwork/tests/test_user.py +++ b/patchwork/tests/test_user.py @@ -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 -- 2.0.0 _______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
