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

Fix debug statement

Browse Source 
This changeset fixes a recently added debug statement that was assuming a list was not empty, which was causing end-to-end tests to break in the admin site.

Signed-off-by: Carlo Costino <carlo.costino@gsa.gov>
This commit is contained in:
Carlo Costino
2024-05-31 12:02:50 -04:00
parent 67392e64e3
commit 7aa6f62545
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/service/rest.py
View File

@@ -472,7 +472,12 @@ def get_all_notifications_for_service(service_id):
pagination.items, many=True
)
current_app.logger.debug(f"number of notifications are {len(notifications)}")
current_app.logger.debug(f"first notification is {notifications[0]}")
if len(notifications) > 0:
current_app.logger.debug(f"first notification is {notifications[0]}")
else:
current_app.logger.debug("there are no notifications to show")
# We try and get the next page of results to work out if we need provide a pagination link to the next page
# in our response if it exists. Note, this could be done instead by changing `count_pages` in the previous
# call to be True which will enable us to use Flask-Sqlalchemy to tell if there is a next page of results but