Merge pull request #892 from ostcar/issue-891

Fixed keyerror on user setting page
This commit is contained in:
Oskar Hahn 2013-09-28 12:49:17 -07:00
commit d926f4da07
2 changed files with 21 additions and 0 deletions

View File

@ -165,6 +165,7 @@ class UsersettingsForm(CssClassMixin, forms.ModelForm):
language = forms.ChoiceField(choices=settings.LANGUAGES) language = forms.ChoiceField(choices=settings.LANGUAGES)
def __init__(self, *args, **kwargs): def __init__(self, *args, **kwargs):
kwargs['initial'] = kwargs.get('initial', {})
kwargs['initial']['user_name'] = kwargs['instance'].username kwargs['initial']['user_name'] = kwargs['instance'].username
return super(UsersettingsForm, self).__init__(*args, **kwargs) return super(UsersettingsForm, self).__init__(*args, **kwargs)

View File

@ -122,3 +122,23 @@ class LockoutProtection(TestCase):
form='form', form='form',
field=None, field=None,
errors='You can not remove the permission to manage participants from the last group your are in.') errors='You can not remove the permission to manage participants from the last group your are in.')
class TestUserSettings(TestCase):
def setUp(self):
self.admin = User.objects.get(pk=1)
self.admin_client = Client()
self.admin_client.login(username='admin', password='admin')
def test_get(self):
response = self.admin_client.get('/usersettings/')
self.assertEqual(response.status_code, 200)
def test_pst(self):
response = self.admin_client.post('/usersettings/', {
'user_name': 'new_name',
'language': 'de'})
self.assertEqual(response.status_code, 200)
admin = User.objects.get(pk=1)
self.assertEqual(admin.username, 'new_name')