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

Add comment explaining mocking

Browse Source 
> I'd find it useful to know mock_get_jobs mocks
> app.job_api_client.get_jobs here, perhaps through a comment. Reading
> it, I associated it with contact_list.get_jobs above.
This commit is contained in:
Chris Hill-Scott
2020-11-30 14:24:40 +00:00
parent a9783f7907
commit c898b68fa8
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/app/models/test_contact_list.py
View File

@@ -13,6 +13,8 @@ def test_created_at():
def test_get_jobs(mock_get_jobs): def test_get_jobs(mock_get_jobs):
contact_list = ContactList({'id': 'a', 'service_id': 'b'}) contact_list = ContactList({'id': 'a', 'service_id': 'b'})
assert isinstance(contact_list.get_jobs(page=123), PaginatedJobs) assert isinstance(contact_list.get_jobs(page=123), PaginatedJobs)
# mock_get_jobs mocks the underlying API client method, not
# contact_list.get_jobs
mock_get_jobs.assert_called_once_with( mock_get_jobs.assert_called_once_with(
'b', 'b',
contact_list_id='a', contact_list_id='a',