darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-03-24 20:20:34 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #707 from alphagov/loosen-get-notification-key-restriction

Browse Source 
Loosen key restriction on get notification
This commit is contained in:
Chris Hill-Scott
2016-10-11 10:40:02 +01:00
committed by GitHub
parent 88362cb075 717e73a9f7
commit 2bdbc4d788
2 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/notifications/rest.py
View File

@@ -159,7 +159,7 @@ def process_firetext_response():
def get_notification_by_id(notification_id):
notification = notifications_dao.get_notification_with_personalisation(str(api_user.service_id),
notification_id,
key_type=api_user.key_type)
key_type=None)
return jsonify(data={"notification": notification_with_personalisation_schema.dump(notification).data}), 200

7
tests/app/notifications/test_rest.py
View File

@@ -93,7 +93,7 @@ def test_get_notifications_empty_result(notify_api, sample_api_key):
(KEY_TYPE_TEAM, KEY_TYPE_NORMAL),
(KEY_TYPE_TEAM, KEY_TYPE_TEST),
])
def test_get_notification_from_different_api_key_fails(
def test_get_notification_from_different_api_key_works(
notify_api,
sample_notification,
api_key_type,
@@ -110,10 +110,7 @@ def test_get_notification_from_different_api_key_fails(
response = client.get(
path='/notifications/{}'.format(sample_notification.id),
headers=_create_auth_header_from_key(api_key))
notification = json.loads(response.get_data(as_text=True))
assert response.status_code == 404
assert notification['result'] == "error"
assert notification['message'] == "No result found"
assert response.status_code == 200
@pytest.mark.parametrize('key_type', [KEY_TYPE_NORMAL, KEY_TYPE_TEAM, KEY_TYPE_TEST])