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

Fix test broken by HTML changes on API page

Browse Source 
I moved the meta for each message into the
`<h3>` in each `<summary>` to make the heading
unique. This broke the test.
This commit is contained in:
Tom Byers
2019-11-29 17:40:45 +00:00
parent d13db305c1
commit 146d5cc07a
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/app/main/views/test_api_integration.py
View File

@@ -37,7 +37,7 @@ def test_should_show_api_page(
rows = page.find_all('details') rows = page.find_all('details')
assert len(rows) == 5 assert len(rows) == 5
for row in rows: for row in rows:
assert row.find('h3').string.strip() == '07123456789' assert row.select('h3 .govuk-details__summary-text')[0].string.strip() == '07123456789'
def test_should_show_api_page_with_lots_of_notifications( def test_should_show_api_page_with_lots_of_notifications(