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

Strip commas from estimated volumes

Browse Source 
We suggest people format their numbers with commas when telling us how
many things they’re going to send.

This causes problems when we paste these values into a spreadsheet,
because the commas get interpreted as column separators.
This commit is contained in:
Chris Hill-Scott
2018-12-11 10:47:28 +00:00
parent 77c61238e9
commit 3cb0b164e8
2 changed files with 13 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
tests/app/main/views/test_service_settings.py
View File

@@ -816,9 +816,9 @@ def test_should_redirect_after_request_to_go_live(
'main.submit_request_to_go_live',
service_id=SERVICE_ONE_ID,
_data={
'volume_email': '111',
'volume_sms': '222',
'volume_letter': '333',
'volume_email': '111,111',
'volume_sms': '222,222',
'volume_letter': '333,333',
'research_consent': 'yes',
},
_follow_redirects=True
@@ -845,13 +845,13 @@ def test_should_redirect_after_request_to_go_live(
'Organisation type: Central\n'
'Agreement signed: Can’t tell (domain is user.gov.uk)\n'
'Checklist completed: No\n'
'Emails in next year: 111\n'
'Text messages in next year: 222\n'
'Letters in next year: 333\n'
'Emails in next year: 111,111\n'
'Text messages in next year: 222,222\n'
'Letters in next year: 333,333\n'
'Consent to research: Yes\n'
'\n'
'---\n'
'{}, None, service one, Test User, test@user.gov.uk, -, 21/12/2012, 222, 111, 333'
'{}, None, service one, Test User, test@user.gov.uk, -, 21/12/2012, 222222, 111111, 333333'
).format(SERVICE_ONE_ID, SERVICE_ONE_ID)
assert normalize_spaces(page.select_one('.banner-default').text) == (