darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-21 07:51:13 -05:00
Code Issues Packages Projects Releases Wiki Activity

Ordered the query - ambiguous orders made tests flaky

Browse Source
This commit is contained in:
Martyn Inglis
2017-04-13 12:52:30 +01:00
parent b0e5062df2
commit b8cfaf02f6
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/dao/jobs_dao.py
View File

@@ -43,7 +43,7 @@ def all_notifications_are_created_for_job(job_id):
@statsd(namespace="dao") @statsd(namespace="dao")
def dao_get_all_notifications_for_job(job_id): def dao_get_all_notifications_for_job(job_id):
return db.session.query(Notification).filter(Notification.job_id == job_id).all() return db.session.query(Notification).filter(Notification.job_id == job_id).order_by(Notification.created_at).all()
def dao_get_job_by_service_id_and_job_id(service_id, job_id): def dao_get_job_by_service_id_and_job_id(service_id, job_id):