From fe91f76286222fe2c8fc0c7a968a4f6d57e8c354 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Apr 30 2024 00:38:05 +0000 Subject: Adjust unit-test for: PagureFlaskApiIssuetests.test_api_view_issues_since Signed-off-by: Pierre-Yves Chibon --- diff --git a/pagure/api/issue.py b/pagure/api/issue.py index 3773951..156e25a 100644 --- a/pagure/api/issue.py +++ b/pagure/api/issue.py @@ -454,8 +454,12 @@ def api_view_issues(repo, username=None, namespace=None): updated_after = None if since: + try: + since = float(since) + except ValueError: + pass # Validate and convert the time - if since.isdigit(): + if isinstance(since, float): # We assume its a timestamp, so convert it to datetime try: updated_after = arrow.get(int(since)).datetime diff --git a/tests/test_pagure_flask_api_issue.py b/tests/test_pagure_flask_api_issue.py index a0a838e..5ee8c46 100644 --- a/tests/test_pagure_flask_api_issue.py +++ b/tests/test_pagure_flask_api_issue.py @@ -2575,7 +2575,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest): "no_stones": None, "order": None, "priority": None, - "since": str(start), + "since": start, "status": None, "tags": [], }, @@ -2614,7 +2614,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest): "no_stones": None, "order": None, "priority": None, - "since": str(middle), + "since": middle, "status": None, "tags": [], }, @@ -2653,7 +2653,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest): "no_stones": None, "order": None, "priority": None, - "since": str(final), + "since": final, "status": None, "tags": [], }, @@ -2696,7 +2696,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest): "no_stones": None, "order": None, "priority": None, - "since": str(final), + "since": final, "status": None, "tags": [], },