fix: resolve unit test configuration issues in conftest.py

- Fix duplicate @pytest.fixture decorators on aws_mocks and repository_type
- Fix incorrect import: template_service -> template_persistence_service
- Update mock_template_service to use TemplatePersistenceService spec
- Tests can now be collected and run without ImportError

Fixes:
- ValueError: @pytest.fixture is being applied more than once
- ImportError: No module named 'src.infrastructure.template.template_service'
- Unit tests should now pass in CI pipeline

Author: fgogolli

src/infrastructure/persistence/components/sql_query_builder.py

@@ -146,8 +146,7 @@ def build_select_by_id(self, id_column: str) -> Tuple[str, str]:
         # Validate identifier
         self._validate_identifier(id_column)
 
-        # nosec B608
-        query = f"SELECT * FROM {self.table_name} WHERE {id_column} = :{id_column}"
+        query = f"SELECT * FROM {self.table_name} WHERE {id_column} = :{id_column}"  # nosec B608
 
         self.logger.debug(f"Built SELECT by ID query for {self.table_name}")
         return query, id_column
@@ -286,8 +285,7 @@ def build_select_by_criteria(self, criteria: Dict[str, Any]) -> Tuple[str, Dict[
                 where_clauses.append(f"{column} = :{param_name}")
                 parameters[param_name] = value
 
-        # nosec B608
-        query = f"SELECT * FROM {self.table_name} WHERE {' AND '.join(where_clauses)}"
+        query = f"SELECT * FROM {self.table_name} WHERE {' AND '.join(where_clauses)}"  # nosec B608
 
         self.logger.debug(f"Built SELECT with criteria query for {self.table_name}")
         return query, parameters

tests/conftest.py

@@ -53,7 +53,9 @@ def decorator(func):
     from src.domain.template.aggregate import Template
     from src.infrastructure.di.buses import CommandBus, QueryBus
     from src.infrastructure.di.container import DIContainer
-    from src.infrastructure.template.template_service import TemplateService
+    from src.infrastructure.template.services.template_persistence_service import (
+        TemplatePersistenceService,
+    )
     from src.providers.aws.configuration.config import AWSConfig
 
     IMPORTS_AVAILABLE = True
@@ -257,7 +259,6 @@ def aws_config() -> AWSConfig:
         return AWSConfig()
 
 
-@pytest.fixture
 @pytest.fixture
 def aws_mocks():
     """Set up comprehensive AWS service mocks."""
@@ -387,7 +388,7 @@ def sample_machine() -> Machine:
 @pytest.fixture
 def mock_template_service() -> Mock:
     """Create a mock template service."""
-    service = Mock(spec=TemplateService)
+    service = Mock(spec=TemplatePersistenceService)
     service.get_available_templates.return_value = []
     service.get_template_by_id.return_value = None
     service.get_templates_by_provider.return_value = []
@@ -464,8 +465,6 @@ def di_container() -> DIContainer:
     return container
 
 
-@pytest.fixture(autouse=True)
-@pytest.fixture
 @pytest.fixture(params=["json", "sql", "memory"])
 def repository_type(request):
     """Parametrized fixture for different repository types."""