diff --git a/app/utils.py b/app/utils.py index 3f8a3b599..29154c7b6 100644 --- a/app/utils.py +++ b/app/utils.py @@ -629,13 +629,15 @@ LETTER_VALIDATION_MESSAGES = { def get_letter_validation_error(validation_message, invalid_pages=None, page_count=None): + if not invalid_pages: + invalid_pages = [] if validation_message not in LETTER_VALIDATION_MESSAGES: return {'title': 'Validation failed'} invalid_pages_are_or_is = 'is' if len(invalid_pages) == 1 else 'are' invalid_pages = unescaped_formatted_list( - invalid_pages or [], + invalid_pages, before_each='', after_each='', prefix='page', diff --git a/tests/app/test_utils.py b/tests/app/test_utils.py index a8dc0755b..e04a44fa8 100644 --- a/tests/app/test_utils.py +++ b/tests/app/test_utils.py @@ -458,7 +458,7 @@ def test_get_letter_validation_error_for_unknown_error(): ), ( 'letter-too-long', - [2], + None, 'Your letter is too long', ( 'Letters must be 10 pages or less. ' @@ -471,7 +471,7 @@ def test_get_letter_validation_error_for_unknown_error(): ), ( 'unable-to-read-the-file', - [2], + None, 'There’s a problem with your file', ( 'Notify cannot read this PDF.'