From 5c4179208045341e50acf279b929a68e4d9a4fae Mon Sep 17 00:00:00 2001 From: Serghei Mihai Date: Thu, 22 Mar 2018 10:48:07 +0100 Subject: [PATCH] lingo: fix url signing for new remote invoice retrieval (#22750) --- combo/apps/lingo/models.py | 2 +- tests/test_notification.py | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/combo/apps/lingo/models.py b/combo/apps/lingo/models.py index 9b53de7..ecba500 100644 --- a/combo/apps/lingo/models.py +++ b/combo/apps/lingo/models.py @@ -214,7 +214,7 @@ class Regie(models.Model): return {} url = self.webservice_url + '/users/with-pending-invoices/' response = requests.get(url, remote_service='auto', cache_duration=0, - log_errors=False) + log_errors=False, without_user=True) if not response.ok: return {} return response.json()['data'] diff --git a/tests/test_notification.py b/tests/test_notification.py index 26c9ec9..4f674bf 100644 --- a/tests/test_notification.py +++ b/tests/test_notification.py @@ -275,7 +275,7 @@ def test_notification_id_and_origin(user): assert result['err'] == 0 -@mock.patch('combo.apps.lingo.models.requests.get') +@mock.patch('combo.utils.requests_wrapper.RequestsSession.request') def test_notify_remote_items(mock_get, app, user, user2, regie): datetime_format = '%Y-%m-%dT%H:%M:%S' @@ -345,6 +345,8 @@ def test_notify_remote_items(mock_get, app, user, user2, regie): regie.webservice_url = 'http://example.org/regie' # is_remote regie.save() regie.notify_new_remote_invoices() + assert 'NameID=' not in mock_get.call_args[0][1] + assert 'email=' not in mock_get.call_args[0][1] assert Notification.objects.filter(external_id__startswith='invoice-%s:' % regie.slug).visible().new().count() == 2 assert Notification.objects.filter(external_id__startswith='invoice-%s:reminder-' % regie.slug).count() == 0 -- 2.16.2