Update utils to version 43.8.

Invalid characters for the first line of a postal address now include < >
This commit is contained in:
Rebecca Law
2021-02-09 14:07:01 +00:00
parent 6972b4b6b0
commit f0ce2c6f5b
8 changed files with 20 additions and 19 deletions
+1 -1
View File
@@ -1362,7 +1362,7 @@ class LetterAddressForm(StripWhitespaceForm):
if address.has_invalid_characters: if address.has_invalid_characters:
raise ValidationError( raise ValidationError(
'Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / ,' 'Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / , < >'
) )
+1 -1
View File
@@ -85,7 +85,7 @@
Last line of the address must be a real UK postcode Last line of the address must be a real UK postcode
{% endif %} {% endif %}
{% elif item.as_postal_address.has_invalid_characters %} {% elif item.as_postal_address.has_invalid_characters %}
Address lines must not start with any of the following characters: @ ( ) = [ ] ” \ / , Address lines must not start with any of the following characters: @ ( ) = [ ] ” \ / , < >
{% endif %} {% endif %}
</span> </span>
{% endcall %} {% endcall %}
+2 -2
View File
@@ -557,11 +557,11 @@ LETTER_VALIDATION_MESSAGES = {
'invalid-char-in-address': { 'invalid-char-in-address': {
'title': 'Theres a problem with the address for this letter', 'title': 'Theres a problem with the address for this letter',
'detail': ( 'detail': (
"Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / ," "Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / , < >"
), ),
'summary': ( 'summary': (
"Validation failed because address lines must not start with any of the " "Validation failed because address lines must not start with any of the "
"following characters: @ ( ) = [ ] ” \\ / ," "following characters: @ ( ) = [ ] ” \\ / , < >"
), ),
}, },
'notify-tag-found-in-content': { 'notify-tag-found-in-content': {
+1 -1
View File
@@ -24,7 +24,7 @@ Shapely==1.7.1
awscli-cwlogs>=1.4,<1.5 awscli-cwlogs>=1.4,<1.5
itsdangerous==1.1.0 itsdangerous==1.1.0
git+https://github.com/alphagov/notifications-utils.git@43.7.0#egg=notifications-utils==43.7.0 git+https://github.com/alphagov/notifications-utils.git@43.8.3#egg=notifications-utils==43.8.3
git+https://github.com/alphagov/govuk-frontend-jinja.git@v0.5.8-alpha#egg=govuk-frontend-jinja==0.5.8-alpha git+https://github.com/alphagov/govuk-frontend-jinja.git@v0.5.8-alpha#egg=govuk-frontend-jinja==0.5.8-alpha
# gds-metrics requires prometheseus 0.2.0, override that requirement as later versions bring significant performance gains # gds-metrics requires prometheseus 0.2.0, override that requirement as later versions bring significant performance gains
+6 -6
View File
@@ -12,7 +12,7 @@ awscli==1.18.211
# via # via
# awscli-cwlogs # awscli-cwlogs
# notifications-utils # notifications-utils
bleach==3.2.1 bleach==3.3.0
# via notifications-utils # via notifications-utils
blinker==1.4 blinker==1.4
# via # via
@@ -111,12 +111,10 @@ markupsafe==1.1.1
mistune==0.8.4 mistune==0.8.4
# via notifications-utils # via notifications-utils
monotonic==1.5 monotonic==1.5
# via # via notifications-python-client
# notifications-python-client
# notifications-utils
notifications-python-client==5.7.1 notifications-python-client==5.7.1
# via -r requirements.in # via -r requirements.in
git+https://github.com/alphagov/notifications-utils.git@43.7.0#egg=notifications-utils==43.7.0 git+https://github.com/alphagov/notifications-utils.git@43.8.3#egg=notifications-utils==43.8.3
# via -r requirements.in # via -r requirements.in
openpyxl==3.0.5 openpyxl==3.0.5
# via pyexcel-xlsx # via pyexcel-xlsx
@@ -184,7 +182,9 @@ s3transfer==0.3.3
# awscli # awscli
# boto3 # boto3
shapely==1.7.1 shapely==1.7.1
# via -r requirements.in # via
# -r requirements.in
# notifications-utils
six==1.15.0 six==1.15.0
# via # via
# awscli-cwlogs # awscli-cwlogs
+5 -4
View File
@@ -21,7 +21,7 @@ awscli==1.18.211
# notifications-utils # notifications-utils
beautifulsoup4==4.8.1 beautifulsoup4==4.8.1
# via -r requirements_for_test.in # via -r requirements_for_test.in
bleach==3.2.1 bleach==3.3.0
# via # via
# -r requirements.txt # -r requirements.txt
# notifications-utils # notifications-utils
@@ -191,12 +191,11 @@ monotonic==1.5
# via # via
# -r requirements.txt # -r requirements.txt
# notifications-python-client # notifications-python-client
# notifications-utils
more-itertools==8.6.0 more-itertools==8.6.0
# via pytest # via pytest
notifications-python-client==5.7.1 notifications-python-client==5.7.1
# via -r requirements.txt # via -r requirements.txt
git+https://github.com/alphagov/notifications-utils.git@43.7.0#egg=notifications-utils==43.7.0 git+https://github.com/alphagov/notifications-utils.git@43.8.3#egg=notifications-utils==43.8.3
# via -r requirements.txt # via -r requirements.txt
openpyxl==3.0.5 openpyxl==3.0.5
# via # via
@@ -325,7 +324,9 @@ s3transfer==0.3.3
# awscli # awscli
# boto3 # boto3
shapely==1.7.1 shapely==1.7.1
# via -r requirements.txt # via
# -r requirements.txt
# notifications-utils
six==1.15.0 six==1.15.0
# via # via
# -r requirements.txt # -r requirements.txt
+2 -2
View File
@@ -688,7 +688,7 @@ def test_upload_csv_file_with_bad_postal_address_shows_check_page_with_errors(
'6 Address must be no more than 7 lines long', '6 Address must be no more than 7 lines long',
'1 2 3 4 5 6 7 8', '1 2 3 4 5 6 7 8',
'7 Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / ,', '7 Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / , < >',
'=Firstname Lastname 123 Example St. SW1A 1AA', '=Firstname Lastname 123 Example St. SW1A 1AA',
] ]
@@ -2324,7 +2324,7 @@ def test_send_one_off_letter_address_populates_address_fields_in_session(
( (
'a\n(b\nSW1A 1AA', 'a\n(b\nSW1A 1AA',
[], [],
'Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / ,', 'Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / , < >',
), ),
]) ])
def test_send_one_off_letter_address_rejects_bad_addresses( def test_send_one_off_letter_address_rejects_bad_addresses(
+2 -2
View File
@@ -562,11 +562,11 @@ def test_get_letter_validation_error_for_unknown_error():
None, None,
'Theres a problem with the address for this letter', 'Theres a problem with the address for this letter',
( (
'Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / ,' 'Address lines must not start with any of the following characters: @ ( ) = [ ] ” \\ / , < >'
), ),
( (
'Validation failed because address lines must not start with any of the following ' 'Validation failed because address lines must not start with any of the following '
'characters: @ ( ) = [ ] ” \\ / ,' 'characters: @ ( ) = [ ] ” \\ / , < >'
), ),
), ),
]) ])