GE comments

Author: eguezgustavo

V2/tests/api/api_fixtures.py

@@ -3,6 +3,7 @@
 import time_tracker.activities._domain as domain 
 import time_tracker.activities._infrastructure as infrastructure 
 
+# TODO: este fixture no creo que debería estar aquí porque tembién lo estamos utilizando en el test de integración. Entonces, debería estar en una carpeta externa a los dos tests
 @pytest.fixture(name='activity_factory')
 def _activity_factory() -> domain.Activity:
     def _make_activity(data: dict):

V2/tests/integration/daos/activities_sql_dao_test.py

@@ -40,6 +40,7 @@ def _clean_database():
 
 def test__create_activity__returns_a_activity_dto__when_saves_correctly_with_sql_database(create_fake_dao, activity_factory):
     dao = create_fake_dao
+    # TODO: Si pasamos el demo data al activity fixture ya no tenemos que pasarle demo data
     existent_activity = activity_factory(DEMO_DATA[0])
 
     inserted_activity = dao.create(existent_activity.__dict__)

V2/time_tracker/activities/_application/_activities/_create_activity.py

@@ -15,17 +15,18 @@ def create_activity(req: func.HttpRequest) -> func.HttpResponse:
     activity_service = _domain.ActivityService(activity_dao)
     use_case = _domain._use_cases.CreateActivityUseCase(activity_service)
 
-
+    # TODO: Aquí debería haber máximo un espacio
     activity_data = req.get_json()
 
-
+    # TODO: Aquí debería haber máximo un espacio
     validation_errors = _validate_activity(activity_data)
     if validation_errors:
         return func.HttpResponse(
             body=json.dumps(validation_errors), status_code=400, mimetype="application/json"
         )
+    # TODO: Aquí debería haber máximo un espacio
 
-
+    # TODO: Le veo redundante crear un objeto activity y luego pasarle como dict. Creo que deberíamos pasar el object y no el dict al case de create activity
     activity_to_create = _domain.Activity(
         id= None,
         name=activity_data['name'],
@@ -34,7 +35,7 @@ def create_activity(req: func.HttpRequest) -> func.HttpResponse:
         deleted=activity_data['deleted']
     )
 
-
+    # TODO: Aquí debería haber máximo un espacio
     created_activity = use_case.create_activity(activity_to_create.__dict__)
     if not create_activity:
         return func.HttpResponse(

V2/time_tracker/activities/_application/_activities/_update_activity.py

@@ -29,7 +29,8 @@ def update_activity(req: func.HttpRequest) -> func.HttpResponse:
         body=response, status_code=status_code, mimetype="application/json"
     )
 
-
+# TODO: para el update podríamos hacer algo más pythonic. Algo como:
+# def _update(activity_id: str, name: str = None, description: str = None, deleted: bool = None, status: intK None) -> str:
 def _update(activity_id: str, activity_data: dict) -> str:
     activity_use_case = _domain._use_cases.UpdateActivityUseCase(
         _create_activity_service(DATABASE)