diff --git a/app/main/forms.py b/app/main/forms.py index eb82e9be6..fab516607 100644 --- a/app/main/forms.py +++ b/app/main/forms.py @@ -239,7 +239,7 @@ class ConfirmMobileNumberForm(Form): class CreateKeyForm(Form): def __init__(self, existing_key_names=[], *args, **kwargs): - self.existing_key_names = existing_key_names + self.existing_key_names = [x.lower() for x in existing_key_names] super(CreateKeyForm, self).__init__(*args, **kwargs) key_name = StringField(u'Description of key', validators=[ @@ -247,5 +247,5 @@ class CreateKeyForm(Form): ]) def validate_key_name(self, key_name): - if key_name.data in self.existing_key_names: + if key_name.data.lower() in self.existing_key_names: raise ValidationError('A key with this name already exists') diff --git a/tests/app/main/test_create_api_key_form.py b/tests/app/main/test_create_api_key_form.py index 54608da6f..c19114b58 100644 --- a/tests/app/main/test_create_api_key_form.py +++ b/tests/app/main/test_create_api_key_form.py @@ -11,6 +11,6 @@ def test_return_validation_error_when_key_name_exists(app_, with app_.test_request_context(): form = CreateKeyForm(_get_names(), - formdata=MultiDict([('key_name', 'some key')])) + formdata=MultiDict([('key_name', 'Some key')])) form.validate() assert {'key_name': ['A key with this name already exists']} == form.errors