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

New field on GET /notifications to allow users to request CSV data

Browse Source
This commit is contained in:
Martyn Inglis
2016-09-15 15:59:34 +01:00
parent 9f02e94f38
commit 803820d9aa
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/schemas.py
View File

@@ -421,6 +421,7 @@ class NotificationsFilterSchema(ma.Schema):
page = fields.Int(required=False)
page_size = fields.Int(required=False)
limit_days = fields.Int(required=False)
include_jobs = fields.Boolean(required=False)
@pre_load
def handle_multidict(self, in_data):

4
tests/app/test_schemas.py
View File

@@ -1,7 +1,7 @@
def test_job_schema_doesnt_return_notifications(sample_notification):
def test_job_schema_doesnt_return_notifications(sample_notification_with_job):
from app.schemas import job_schema
job = sample_notification.job
job = sample_notification_with_job.job
assert job.notifications.count() == 1
data, errors = job_schema.dump(job)