Issue/api calls (#254)

* add read api calls + tests

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* update ruff

* update doc string

* add start and end datetime filter

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix test

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

nowcasting_datamodel/read/read_user.py

@@ -1,6 +1,7 @@
 """ Read user"""
 import logging
-from typing import List
+from datetime import datetime
+from typing import List, Optional
 
 from sqlalchemy.orm.session import Session
 
@@ -62,21 +63,29 @@ def get_all_last_api_request(session: Session) -> List[APIRequestSQL]:
     return last_requests_sql
 
 
-def get_api_requests_for_one_user(session: Session, email: str) -> List[APIRequestSQL]:
+def get_api_requests_for_one_user(
+    session: Session,
+    email: str,
+    start_datetime: Optional[datetime] = None,
+    end_datetime: Optional[datetime] = None,
+) -> List[APIRequestSQL]:
     """
     Get all api requests for one user
 
     :param session: database session
     :param email: user email
-    :return:
+    :param start_datetime: only get api requests after start datetime
+    :param end_datetime: only get api requests before end datetime
     """
 
-    api_requests = (
-        session.query(APIRequestSQL)
-        .join(UserSQL)
-        .filter(UserSQL.email == email)
-        .order_by(APIRequestSQL.created_utc.desc())
-        .all()
-    )
+    query = session.query(APIRequestSQL).join(UserSQL).filter(UserSQL.email == email)
+
+    if start_datetime is not None:
+        query = query.filter(APIRequestSQL.created_utc >= start_datetime)
+
+    if end_datetime is not None:
+        query = query.filter(APIRequestSQL.created_utc <= end_datetime)
+
+    api_requests = query.order_by(APIRequestSQL.created_utc.desc()).all()
 
     return api_requests

tests/read/test_read_user.py

@@ -5,6 +5,8 @@
     get_api_requests_for_one_user,
 )
 
+from datetime import datetime, timedelta
+
 
 def test_get_user(db_session):
     db_session.add(UserSQL(email="[email protected]"))
@@ -41,3 +43,23 @@ def test_get_api_requests_for_one_user(db_session):
     requests_sql = get_api_requests_for_one_user(session=db_session, email=user.email)
     assert len(requests_sql) == 1
     assert requests_sql[0].url == "test"
+
+
+def test_get_api_requests_for_one_user_start_datetime(db_session):
+    user = get_user(session=db_session, email="[email protected]")
+    db_session.add(APIRequestSQL(user_uuid=user.uuid, url="test"))
+
+    requests_sql = get_api_requests_for_one_user(
+        session=db_session, email=user.email, start_datetime=datetime.now() + timedelta(hours=1)
+    )
+    assert len(requests_sql) == 0
+
+
+def test_get_api_requests_for_one_user_end_datetime(db_session):
+    user = get_user(session=db_session, email="[email protected]")
+    db_session.add(APIRequestSQL(user_uuid=user.uuid, url="test"))
+
+    requests_sql = get_api_requests_for_one_user(
+        session=db_session, email=user.email, end_datetime=datetime.now() - timedelta(hours=1)
+    )
+    assert len(requests_sql) == 0