diff --git a/app/templates/views/providers/providers.html b/app/templates/views/providers/providers.html index ee6e933ac..9cf706e1f 100644 --- a/app/templates/views/providers/providers.html +++ b/app/templates/views/providers/providers.html @@ -23,7 +23,7 @@ Providers field_headings_visible=True ) %} - {{ text_field(item.display_name) }} + {{ link_field(item.display_name, url_for('main.view_provider', provider_id=item.id)) }} {{ text_field(item.priority) }} diff --git a/tests/app/main/views/test_providers.py b/tests/app/main/views/test_providers.py index 9f3c886cc..2279b9050 100644 --- a/tests/app/main/views/test_providers.py +++ b/tests/app/main/views/test_providers.py @@ -172,7 +172,7 @@ def test_should_show_all_providers( domestic_sms_first_row = domestic_sms_table.tbody.find_all('tr')[0] table_data = domestic_sms_first_row.find_all('td') - assert not table_data[0].find_all("a") + assert table_data[0].find_all("a")[0]['href'] == '/provider/6005e192-4738-4962-beec-ebd982d0b03f' assert table_data[0].text.strip() == "Domestic SMS Provider" assert table_data[1].text.strip() == "1" assert table_data[2].text.strip() == "42" @@ -183,7 +183,7 @@ def test_should_show_all_providers( domestic_sms_second_row = domestic_sms_table.tbody.find_all('tr')[1] table_data = domestic_sms_second_row.find_all('td') - assert not table_data[0].find_all("a") + assert table_data[0].find_all("a")[0]['href'] == '/provider/0bd529cd-a0fd-43e5-80ee-b95ef6b0d51f' assert table_data[0].text.strip() == "Second Domestic SMS Provider" assert table_data[1].text.strip() == "2" assert table_data[2].text.strip() == "58"