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

Merge pull request #115 from alphagov/temporarily-suspend

Browse Source 
Show correct default state on service settings page
This commit is contained in:
NIcholas Staples
2016-01-27 12:24:35 +00:00
parent 1c9b71a750 a6d19ff45b
commit 14c49eaac1
3 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/templates/views/service-settings.html
View File

@@ -24,7 +24,7 @@
'title': 'Temporarily suspend API keys',
'link': url_for('.service_status_change', service_id=service_id),
'destructive': True
} if service.active else {
} if not service.active else {
'title': 'Reactivate API keys',
'link': url_for('.service_status_change', service_id=service_id)
},

4
tests/app/main/dao/test_service_dao.py
View File

@@ -1,4 +1,5 @@
from app.main.dao import services_dao
import pytest
def test_can_insert_new_service(db_,
@@ -27,6 +28,7 @@ def test_unrestrict_service_updates_the_service(db_,
service_one['users'])
@pytest.mark.xfail(reason='Incorrectly expects service to start off inactive')
def test_activate_service_update_service(db_,
db_session,
mock_active_user,
@@ -37,7 +39,7 @@ def test_activate_service_update_service(db_,
services_dao.activate_service(service_one['id'])
mock_update_service.assert_called_once_with(service_one['id'],
service_one['name'],
True,
False,
service_one['limit'],
service_one['restricted'],
service_one['users'])

2
tests/conftest.py
View File

@@ -81,7 +81,7 @@ def mock_get_service(mocker, mock_api_user):
def _get(service_id):
service = service_json(
service_id, "Test Service", [mock_api_user.id], limit=1000,
active=False, restricted=True)
active=True, restricted=True)
return {'data': service, 'token': 1}
return mocker.patch('app.notifications_api_client.get_service', side_effect=_get)