fix: TT-403 comments solved

Author: mandres2015

V2/tests/api/azure/time_entry_azure_endpoints_test.py

@@ -43,13 +43,11 @@ def test__time_entry_azure_endpoint__creates_an_time_entry__when_time_entry_has_
 
 
 def test__delete_time_entries_azure_endpoint__returns_an_time_entry_with_true_deleted__when_its_id_is_found(
-    create_fake_database, time_entry_factory, insert_time_entry, insert_activity, activity_factory,
+    test_db, time_entry_factory, insert_time_entry, insert_activity, activity_factory,
 ):
-    db = create_fake_database
-    azure_time_entries._delete_time_entry.DATABASE = db
-    inserted_activity = insert_activity(activity_factory(), db).__dict__
+    inserted_activity = insert_activity(activity_factory(), test_db).__dict__
     time_entry_body = time_entry_factory(activity_id=inserted_activity["id"], technologies="[jira,sql]")
-    inserted_time_entry = insert_time_entry(time_entry_body, db)
+    inserted_time_entry = insert_time_entry(time_entry_body, test_db)
 
     req = func.HttpRequest(
         method='DELETE',
@@ -66,11 +64,8 @@ def test__delete_time_entries_azure_endpoint__returns_an_time_entry_with_true_de
 
 
 def test__delete_time_entries_azure_endpoint__returns_a_status_code_400__when_time_entry_recive_invalid_id(
-    create_fake_database,
+    test_db,
 ):
-    db = create_fake_database
-    azure_time_entries._delete_time_entry.DATABASE = db
-
     req = func.HttpRequest(
         method="DELETE",
         body=None,

V2/tests/integration/daos/time_entries_dao_test.py

@@ -1,5 +1,5 @@
 import pytest
-
+from faker import Faker
 
 import time_tracker.time_entries._domain as domain
 import time_tracker.time_entries._infrastructure as infrastructure
@@ -48,11 +48,10 @@ def test__time_entry__returns_None__when_not_saves_correctly(
     assert inserted_time_entry is None
 
 
-
 def test_delete__returns_an_time_entry_with_true_deleted__when_an_time_entry_matching_its_id_is_found(
-    create_fake_dao, time_entry_factory, insert_activity, activity_factory
+    create_fake_dao, test_db, time_entry_factory, insert_activity, activity_factory
 ):
-    dao = create_fake_dao
+    dao = create_fake_dao(test_db)
     inserted_activity = insert_activity(activity_factory(), dao.db)
     existent_time_entry = time_entry_factory(activity_id=inserted_activity.id, technologies="[jira,sql]")
     inserted_time_entry = dao.create(existent_time_entry)
@@ -63,9 +62,9 @@ def test_delete__returns_an_time_entry_with_true_deleted__when_an_time_entry_mat
 
 
 def test_delete__returns_none__when_no_time_entry_matching_its_id_is_found(
-    create_fake_dao,
+    create_fake_dao, test_db
 ):
-    dao = create_fake_dao
+    dao = create_fake_dao(test_db)
 
     result = dao.delete(Faker().pyint())
 

V2/tests/unit/services/time_entry_service_test.py

@@ -1,3 +1,5 @@
+from faker import Faker
+
 from time_tracker.time_entries._domain import TimeEntryService
 
 

V2/time_tracker/time_entries/_application/__init__.py

@@ -1,2 +1,2 @@
 # flake8: noqa
-from ._time_entries import delete_time_entry
+from ._time_entries import create_time_entry, delete_time_entry

V2/time_tracker/time_entries/_application/_time_entries/__init__.py

@@ -1,2 +1,3 @@
 # flake8: noqa
+from ._create_time_entry import create_time_entry
 from ._delete_time_entry import delete_time_entry

V2/time_tracker/time_entries/_application/_time_entries/_create_time_entry.py

@@ -6,15 +6,12 @@
 
 from ... import _domain
 from ... import _infrastructure
-
-_JSON_PATH = (
-    'time_tracker/time_entries/_infrastructure/_data_persistence/time_entries_data.json'
-)
+from time_tracker._infrastructure import DB
 
 
 def create_time_entry(req: func.HttpRequest) -> func.HttpResponse:
-
-    time_entry_dao = _infrastructure.TimeEntriesJsonDao(_JSON_PATH)
+    database = DB()
+    time_entry_dao = _infrastructure.TimeEntriesSQLDao(database)
     time_entry_service = _domain.TimeEntryService(time_entry_dao)
     use_case = _domain._use_cases.CreateTimeEntryUseCase(time_entry_service)
 

V2/time_tracker/time_entries/_application/_time_entries/_delete_time_entry.py

@@ -6,11 +6,9 @@
 from ... import _infrastructure
 from time_tracker._infrastructure import DB
 
-DATABASE = DB()
-
 
 def delete_time_entry(req: func.HttpRequest) -> func.HttpResponse:
-    time_entry_dao = _infrastructure.TimeEntriesSQLDao(DATABASE)
+    time_entry_dao = _infrastructure.TimeEntriesSQLDao(DB())
     time_entry_service = _domain.TimeEntryService(time_entry_dao)
     use_case = _domain._use_cases.DeleteTimeEntryUseCase(time_entry_service)
 

V2/time_tracker/time_entries/_domain/__init__.py

@@ -3,5 +3,6 @@
 from ._persistence_contracts import TimeEntriesDao
 from ._services import TimeEntryService
 from ._use_cases import (
+    CreateTimeEntryUseCase,
     DeleteTimeEntryUseCase
 )

V2/time_tracker/time_entries/_domain/_use_cases/__init__.py

@@ -1,2 +1,3 @@
 # flake8: noqa
+from ._create_time_entry_use_case import CreateTimeEntryUseCase
 from ._delete_time_entry_use_case import DeleteTimeEntryUseCase

V2/time_tracker/time_entries/_domain/_use_cases/_create_time_entry_use_case.py

@@ -7,4 +7,4 @@ def __init__(self, time_entry_service: TimeEntryService):
         self.time_entry_service = time_entry_service
 
     def create_time_entry(self, time_entry_data: TimeEntry) -> TimeEntry:
-        return self.time_entry_service.create(time_entry_data.__dict__)
+        return self.time_entry_service.create(time_entry_data)