Feat/integration test

.github/workflows/test.yml

@@ -0,0 +1,60 @@
+name: Run Tests
+
+on:
+  pull_request:
+    types: [opened, synchronize, reopened]
+    branches: [main, develop]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+    - name: Checkout code
+      uses: actions/checkout@v4
+
+    - name: Set up Python
+      uses: actions/setup-python@v4
+      with:
+        python-version: '3.12'
+
+    - name: Cache pip dependencies
+      uses: actions/cache@v3
+      with:
+        path: ~/.cache/pip
+        key: ${{ runner.os }}-pip-${{ hashFiles('pyproject.toml') }}
+        restore-keys: |
+          ${{ runner.os }}-pip-
+
+    - name: Create and activate virtual environment
+      run: |
+        python -m venv venv
+        source venv/bin/activate
+
+    - name: Install dependencies
+      run: |
+        source venv/bin/activate
+        pip install --upgrade pip
+        pip install .
+
+    - name: Run tests
+      env:
+        ENV: test
+        SECRET_KEY: test_secret_key
+        ACCESS_TOKEN_EXPIRE_MINUTES: 60
+      run: |
+        source venv/bin/activate
+        uvicorn src.main:app --host 0.0.0.0 --port 8000 --reload &
+        pytest src/entrypoints/api/tests/profile.py \
+               src/entrypoints/api/tests/group.py \
+               src/entrypoints/api/tests/exercise.py \
+               src/entrypoints/api/tests/training.py \
+               src/entrypoints/api/tests/diet.py \
+               -v \
+               --tb=short
+
+    - name: Test Summary
+      if: always()
+      run: |
+        echo "Tests completed!"
+        echo "Check the logs above for details."

.gitignore

@@ -13,3 +13,5 @@
 # PYTHON
 __pycache__/
 
+# Info file
+tracknatrainapi.egg-info/

pyproject.toml

@@ -1,12 +1,23 @@
 [project]
 name = "tracknatrainapi"
 version = "0.4.0"
-requires-python = ">=3.13"
-dependencies = [ "annotated-types==0.7.0", "anyio==4.9.0", "bcrypt==4.3.0", "boto3==1.37.37", "botocore==1.37.37", "cffi==1.17.1", "click==8.1.8", "cryptography==44.0.2", "dnspython==2.7.0", "ecdsa==0.19.1", "email-validator==2.2.0", "exceptiongroup==1.2.2", "fastapi==0.115.12", "greenlet==3.1.1", "h11==0.14.0", "idna==3.10", "jmespath==1.0.1", "passlib[bcrypt]>=1.7.4", "psycopg2-binary==2.9.10", "pyasn1==0.4.8", "pycparser==2.22", "pydantic==2.11.3", "pydantic-core==2.33.1", "python-dateutil==2.9.0.post0", "python-dotenv==1.1.0", "python-jose==3.4.0", "python-multipart==0.0.20", "rsa==4.9", "s3transfer==0.11.5", "six==1.17.0", "sniffio==1.3.1", "sqlalchemy==2.0.40", "starlette==0.46.2", "typing-extensions==4.13.2", "typing-inspection==0.4.0", "urllib3==2.4.0", "uvicorn==0.34.1",]
+requires-python = ">=3.12"
+dependencies = [ "annotated-types==0.7.0", "anyio==4.9.0", "bcrypt==4.3.0", "boto3==1.37.37", "botocore==1.37.37", "cffi==1.17.1", "click==8.1.8", "cryptography==44.0.2", "dnspython==2.7.0", "ecdsa==0.19.1", "email-validator==2.2.0", "exceptiongroup==1.2.2", "fastapi==0.115.12", "greenlet==3.1.1", "h11==0.14.0", "idna==3.10", "jmespath==1.0.1", "passlib[bcrypt]>=1.7.4", "psycopg2-binary==2.9.10", "pyasn1==0.4.8", "pycparser==2.22", "pydantic==2.11.3", "pydantic-core==2.33.1", "python-dateutil==2.9.0.post0", "python-dotenv==1.1.0", "python-jose==3.4.0", "python-multipart==0.0.20", "rsa==4.9", "s3transfer==0.11.5", "six==1.17.0", "sniffio==1.3.1", "sqlalchemy==2.0.40", "starlette==0.46.2", "typing-extensions==4.13.2", "typing-inspection==0.4.0", "urllib3==2.4.0", "uvicorn==0.34.1", "pytest>=7.0",          
+  "pytest-asyncio>=0.20", 
+  "httpx>=0.24",          
+  "pytest-cov>=4.0",      
+  "coverage>=6.0"  ]
+
+[project.optional-dependencies]
+testing = [
+  "pytest>=7.0",          
+  "pytest-asyncio>=0.20", 
+  "httpx>=0.24",          
+  "pytest-cov>=4.0",      
+  "coverage>=6.0"         
+]
+
 
 [build-system]
 requires = [ "setuptools>=42", "wheel",]
 build-backend = "setuptools.build_meta"
-
-[tool.setuptools.packages.find]
-include = [ "app*", "alembic*",]

src/adapters/inmemory/repositories/group.py

@@ -9,10 +9,10 @@
 from src.adapters.inmemory.repositories.profile import InMemoryProfileRepository
 
 class InMemoryGroupRepository(GroupRepository):
-    def __init__(self):
+    def __init__(self, profile_repo: InMemoryProfileRepository):
         self._groups: dict[UUID, DomainGroup] = {}
         self._members: dict[UUID, List[UUID]] = {}
-        self._profile_repo = InMemoryProfileRepository()
+        self._profile_repo = profile_repo        
 
     def find_by_id(self, id: UUID) -> Optional[DomainGroup]:
         return self._groups.get(id)

src/adapters/inmemory/repositories/profile.py

@@ -7,8 +7,11 @@
 
 
 class InMemoryProfileRepository(ProfileRepository):
-    def __init__(self):
+    def __init__(self, initial: list[DomainProfile] | None = None):
         self._data: dict[UUID, DomainProfile] = {}
+        if initial:
+            for profile in initial:
+                self._data[profile.id] = profile
 
     def find_by_email(self, email: str) -> Optional[DomainProfile]:
         for profile in self._data.values():
@@ -41,3 +44,6 @@ def find_all_users(self) -> List[DomainProfile]:
 
     def find_all_coachs(self) -> List[DomainProfile]:
         return [p for p in self._data.values() if "coach" in (p.roles or [])]
+
+    def find_all(self) -> List[DomainProfile]:
+        return list(self._data.values())

src/container.py

@@ -1,4 +1,5 @@
 import os
+from uuid import uuid4, UUID
 
 from src.domain.lib.security import BcryptPasswordHasher
 from src.domain.services.profile import ProfileService
@@ -13,13 +14,30 @@ def __init__(self, env: str | None = None):
         self.hasher = BcryptPasswordHasher()
 
         if self.env in ("dev", "test"):
+            from src.domain.model.profile import Profile as DomainProfile
+            plain_pw = "123456789"
+            hashed_pw = self.hasher.hash(plain_pw)
+            admin = DomainProfile(
+                id=uuid4(),
+                email="admin@mail.fr",
+                password=hashed_pw,
+                name="Admin",
+                sex=None,
+                age=None,
+                contact=None,
+                pricing=None,
+                description=None,
+                legacy=False,
+                roles=["admin"],
+                created_at=None,
+            )
             from src.adapters.inmemory.repositories.profile import InMemoryProfileRepository
             from src.adapters.inmemory.repositories.group import InMemoryGroupRepository
             from src.adapters.inmemory.repositories.training import InMemoryTrainingRepository
             from src.adapters.inmemory.repositories.exercise import InMemoryExerciseRepository
             from src.adapters.inmemory.repositories.diet import InMemoryDietRepository
-            self.profile_repo = InMemoryProfileRepository()
-            self.group_repo = InMemoryGroupRepository()
+            self.profile_repo = InMemoryProfileRepository(initial=[admin])
+            self.group_repo = InMemoryGroupRepository(self.profile_repo)
             self.training_repo = InMemoryTrainingRepository()
             self.exercise_repo = InMemoryExerciseRepository()
             self.diet_repo = InMemoryDietRepository()

src/domain/services/diet.py

@@ -158,14 +158,16 @@ def update_meal_plan(
     ) -> DomainMealPlan:
         mp = self.get_meal_plan_by_id(plan_id)
 
-        domain_meals = [DomainMealItem(**m.model_dump()) for m in meals]
-        if not domain_meals:
+        updated_name = name if name is not None else mp.name
+        updated_meals = [DomainMealItem(**m.model_dump()) for m in meals] if meals is not None else mp.meals
+
+        if not updated_meals:
             raise ValueError("Meal Plan must have at least one meal")
 
         if name is not None:
-            mp.name = name
+            mp.name = updated_name
         if meals is not None:
-            mp.meals = domain_meals
+            mp.meals = updated_meals
         return self._repo.update_meal_plan(mp)
 
     def delete_meal_plan(self, plan_id: UUID) -> None:

src/domain/services/group.py

@@ -54,7 +54,9 @@ def remove_member(self, group_id: UUID, user_id: UUID) -> None:
         if not group:
             raise NotFoundError(f"Group with id {group_id} not found")
 
-        if user_id not in self._repo.list_members(group_id):
+        profiles = self._repo.list_members(group_id) 
+        member_ids = [p.id for p in profiles] 
+        if user_id not in member_ids:
             raise NotFoundError(f"User with id {user_id} is not a member of group {group_id}")
 
         self._repo.remove_member(group_id, user_id)

src/entrypoints/api/deps/auth.py

@@ -36,7 +36,7 @@ def require_owner_or_admin(
     if str(user_id) != str(profile_id) and "admin" not in roles:
         raise HTTPException(
             status_code=status.HTTP_403_FORBIDDEN,
-            detail="Forbidden access"
+            detail="Access forbidden"
         )
     return user
 

src/entrypoints/api/routers/exercise.py

@@ -12,7 +12,7 @@
 
 router = APIRouter(prefix="/exercises", tags=["exercises"])
 
-@router.post("/", response_model=ExerciseRead, status_code=201, dependencies=[Depends(require_roles("admin", "coach"))])
+@router.post("", response_model=ExerciseRead, status_code=201, dependencies=[Depends(require_roles("admin", "coach"))])
 def create_exercise(
     dto: ExerciseCreate,
     user=Depends(get_current_user)
@@ -30,7 +30,7 @@ def create_exercise(
 
     return ExerciseRead.model_validate(exercise)
 
-@router.get("/", response_model=List[ExerciseRead], dependencies=[Depends(get_current_user)])
+@router.get("", response_model=List[ExerciseRead], dependencies=[Depends(get_current_user)])
 def get_exercises(
     user=Depends(get_current_user)
 ):

src/entrypoints/api/routers/group.py

@@ -100,7 +100,7 @@ def list_owner_groups(owner_id: UUID):
 def leave_group(group_id: UUID, user=Depends(get_current_user)):
     service = container.get_group_service()
     try:
-        service.remove_member(group_id, user["sub"])
+        service.remove_member(group_id, UUID(user["sub"]))
     except NotFoundError as e:
         raise HTTPException(404, str(e))
 

src/entrypoints/api/schemas/profile.py

@@ -34,13 +34,13 @@ class ProfileRead(BaseModel):
 
 class CoachProfileRead(BaseModel):
     id: UUID
-    name: str
-    sex: str
-    age: int
-    contact: str
-    pricing: float
-    description: str
-    legacy: str
+    name: Optional[str] = None
+    sex: Optional[str] = None
+    age: Optional[int] = None
+    contact: Optional[str] = None
+    pricing: Optional[float] = None
+    description: Optional[str] = None
+    legacy: Optional[str] = None
 
     model_config = {
         "from_attributes": True

src/entrypoints/api/tests/conftest.py

@@ -0,0 +1,36 @@
+import pytest
+import os
+from httpx import AsyncClient, ASGITransport
+from src.main import app
+from src.container import Container
+import pytest_asyncio
+
+
+
+@pytest.fixture(scope="session", autouse=True)
+def set_test_env():
+    os.environ["ENV"] = "test" 
+    yield
+
+
+@pytest.fixture(scope="module", autouse=True)
+def container():
+    c = Container(env="test")  
+
+    assert os.getenv("ENV") == "test", "L'environnement n'est pas correctement configuré sur 'test'."
+
+    return c
+
+
+@pytest_asyncio.fixture
+async def client(container):
+
+    transport = ASGITransport(app=app)
+
+    async with AsyncClient(transport=transport, base_url="http://testserver") as ac:
+        yield ac 
+
+@pytest.fixture(scope="session")
+def test_state():
+
+    return {}