fix: TT-220 apply refactoring on some code

Author: Pablo

time_tracker_api/projects/projects_model.py

@@ -125,7 +125,7 @@ def get_all(
         customers_ids_conditions = (
             customers_ids_conditions + customers_id
             if customers_id
-            else customers_id
+            else customers_ids_conditions
         )
 
         projects = self.repository.find_all(

time_tracker_api/time_entries/time_entries_dao.py

@@ -98,8 +98,7 @@ def get_all(self, conditions: dict = None, **kwargs) -> list:
             conditions=conditions,
         )
         date_range = self.handle_date_filter_args(args=conditions)
-        limit = conditions.get("limit", None)
-        conditions.pop("limit", None)
+        limit = conditions.pop("limit", None)
         azure_connection = AzureConnection()
         current_user_is_tester = azure_connection.is_test_user(
             event_ctx.user_id
@@ -143,12 +142,11 @@ def get_lastest_entries_by_project(
         for id_project in projects_ids:
             conditions.update({"project_id": id_project})
 
-            limit = 1
             latest = self.repository.find_all_entries(
                 event_ctx,
                 conditions=conditions,
                 date_range=date_range,
-                max_count=limit,
+                max_count=1,
             )
 
             if len(latest) > 0:

time_tracker_api/time_entries/time_entries_repository.py

@@ -196,13 +196,6 @@ def find_all(
             time_entries, max_count, exist_conditions
         )
 
-    def create_sql_test_users_exclusion_condition(self, test_user_ids):
-        if test_user_ids != None:
-            tuple_string = convert_list_to_tuple_string(test_user_ids)
-            return "AND c.owner_id NOT IN {list}".format(list=tuple_string)
-
-        return ""
-
     def get_last_entry(
         self,
         owner_id: str,