darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-30 02:50:03 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1685 from alphagov/comma-error

Browse Source 
Update error message about commas in placeholders
This commit is contained in:
Chris Hill-Scott
2017-12-11 11:56:45 +00:00
committed by GitHub
parent 72dd0b369f eef66b70de
commit f156373e9a
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/main/validators.py
View File

@@ -1,5 +1,5 @@
from wtforms import ValidationError
from notifications_utils.template import Template
from notifications_utils.field import Field
from notifications_utils.gsm import get_non_gsm_compatible_characters
from app import formatted_list
@@ -45,11 +45,11 @@ class ValidGovEmail:
class NoCommasInPlaceHolders:
def __init__(self, message='You can’t have commas in your fields'):
def __init__(self, message='You can’t put commas between double brackets'):
self.message = message
def __call__(self, form, field):
if ',' in ''.join(Template({'content': field.data}).placeholders):
if ',' in ''.join(Field(field.data).placeholders):
raise ValidationError(self.message)

2
tests/app/main/test_validators.py
View File

@@ -149,7 +149,7 @@ def test_for_commas_in_placeholders(
):
with pytest.raises(ValidationError) as error:
NoCommasInPlaceHolders()(None, _gen_mock_field('Hello ((name,date))'))
assert str(error.value) == 'You can’t have commas in your fields'
assert str(error.value) == 'You can’t put commas between double brackets'
NoCommasInPlaceHolders()(None, _gen_mock_field('Hello ((name))'))