darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-02-06 11:23:48 -05:00
Code Issues Packages Projects Releases Wiki Activity

Remove search_bar_visible test param

Browse Source 
It’s always `True` now, so we can just `assert True` instead.
This commit is contained in:
Chris Hill-Scott
2020-04-27 12:47:30 +01:00
parent 986f2752e0
commit b01bc17dbb
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
tests/app/main/views/test_activity.py
View File

@@ -623,10 +623,10 @@ def test_redacts_templates_that_should_be_redacted(
@pytest.mark.parametrize(
"message_type, tablist_visible, search_bar_visible", [
('email', True, True),
('sms', True, True),
('letter', False, True)
"message_type, tablist_visible", [
('email', True),
('sms', True),
('letter', False)
]
)
def test_big_numbers_dont_show_for_letters(
@@ -639,7 +639,6 @@ def test_big_numbers_dont_show_for_letters(
mock_get_no_api_keys,
message_type,
tablist_visible,
search_bar_visible
):
page = client_request.get(
'main.view_notifications',
@@ -650,7 +649,7 @@ def test_big_numbers_dont_show_for_letters(
)
assert (len(page.select("[role=tablist]")) > 0) == tablist_visible
assert (len(page.select("[type=search]")) > 0) == search_bar_visible
assert (len(page.select("[type=search]")) > 0) is True
@freeze_time("2017-09-27 16:30:00.000000")