Merge branch 'master' into master

Author: arthurnnguyen

backend/Dockerfile

@@ -34,6 +34,7 @@ COPY ./scripts /app/scripts
 COPY ./pyproject.toml ./uv.lock ./alembic.ini /app/
 
 COPY ./app /app/app
+COPY ./tests /app/tests
 
 # Sync the project
 # Ref: https://docs.astral.sh/uv/guides/integration/docker/#intermediate-layers

backend/README.md

@@ -97,7 +97,7 @@ To test the backend run:
 $ bash ./scripts/test.sh
 ```
 
-The tests run with Pytest, modify and add tests to `./backend/app/tests/`.
+The tests run with Pytest, modify and add tests to `./backend/tests/`.
 
 If you use GitHub Actions the tests will run automatically.
 

backend/app/core/config.py

@@ -17,7 +17,7 @@
 
 def parse_cors(v: Any) -> list[str] | str:
     if isinstance(v, str) and not v.startswith("["):
-        return [i.strip() for i in v.split(",")]
+        return [i.strip() for i in v.split(",") if i.strip()]
     elif isinstance(v, list | str):
         return v
     raise ValueError(v)

backend/scripts/test.sh

@@ -3,6 +3,6 @@
 set -e
 set -x
 
-coverage run -m pytest
+coverage run -m pytest tests/
 coverage report
 coverage html --title "${@-coverage}"

backend/app/tests/__init__.py renamed to backend/tests/__init__.py

@@ -4,7 +4,7 @@
 from sqlmodel import Session
 
 from app.core.config import settings
-from app.tests.utils.item import create_random_item
+from tests.utils.item import create_random_item
 
 
 def test_create_item(

backend/app/tests/api/__init__.py renamed to backend/tests/api/__init__.py

@@ -7,9 +7,9 @@
 from app.core.security import verify_password
 from app.crud import create_user
 from app.models import UserCreate
-from app.tests.utils.user import user_authentication_headers
-from app.tests.utils.utils import random_email, random_lower_string
 from app.utils import generate_password_reset_token
+from tests.utils.user import user_authentication_headers
+from tests.utils.utils import random_email, random_lower_string
 
 
 def test_get_access_token(client: TestClient) -> None: