darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-19 05:59:44 -04:00
Code Issues Packages Projects Releases Wiki Activity

Removed get_all_notifications method from notification_api_client as it is not longer being used.

Browse Source
This commit is contained in:
Rebecca Law
2016-11-18 14:39:23 +00:00
parent 045864f052
commit cebf8e1d65
2 changed files with 0 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
tests/app/notify_client/test_notification_client.py
View File

@@ -2,20 +2,6 @@ import pytest
from app.notify_client.notification_api_client import NotificationApiClient
def test_client_gets_notifications(mocker):
mock_get = mocker.patch('app.notify_client.notification_api_client.NotificationApiClient.get')
NotificationApiClient().get_all_notifications()
mock_get.assert_called_once_with(url='/notifications', params={})
def test_client_gets_notifications_with_page(mocker):
mock_get = mocker.patch('app.notify_client.notification_api_client.NotificationApiClient.get')
NotificationApiClient().get_all_notifications(page=99)
mock_get.assert_called_once_with(url='/notifications', params={'page': 99})
@pytest.mark.parametrize("arguments,expected_call", [
(
{},