docs: update CHANGELOG for fixes and improvements in Django/pytest integration

- Added a new entry in `CHANGELOG.md` detailing the fix for the apparent infinite hang during pytest/CI due to incorrect settings import.
- Updated `pytest.ini` to set `log_cli` to false by default to prevent logging issues during CI.
- Enhanced logging in `api/models.py` to provide clearer visibility during model imports.
- Removed the unnecessary appending of `coverage` to `INSTALLED_APPS` in `api/settings.py`.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: Andreas-Garcia

CHANGELOG.md

@@ -64,6 +64,10 @@ All contributors (including maintainers) should update `CHANGELOG.md` when creat
 
 ## [Unreleased]
 
+### Fixed
+
+- **Pytest / CI apparent infinite hang after settings**: [`api/test/tests/conftest.py`](api/test/tests/conftest.py) imports **`from django.conf import settings`** instead of **`from api import settings`**, so the full **`api.settings`** module is not run during conftest import (avoids bad interaction with pytest-django startup and **`log_cli`**). [`api/settings.py`](api/settings.py) no longer appends **`coverage`** to **`INSTALLED_APPS`** (the **`coverage`** PyPI package is not a Django application). [`pytest.ini`](pytest.ini) defaults **`log_cli=false`**; use **`pytest -o log_cli=true`** when you want live logs.
+
 ### CI
 
 - **Static files workflow**: [`.github/workflows/static-files.yml`](.github/workflows/static-files.yml) sets **`ENV=collect_static`** in the job environment (Django collect-static path in **`api/settings.py`**); it is no longer read from the GitHub Variable **`ENV`**.
@@ -80,7 +84,7 @@ All contributors (including maintainers) should update `CHANGELOG.md` when creat
 
 ### Improved
 
-- **Django / pytest startup visibility**: [`api/utils/utils.py`](api/utils/utils.py) **`print_django`** uses **`flush=True`**. [`api/settings.py`](api/settings.py) logs that **`django.setup()`** follows settings. [`api/urls.py`](api/urls.py) logs start/end of URLconf import; [`api/apps.py`](api/apps.py) **`ApiConfig.ready()`** logs when the app registry finishes the **`api`** app. [`api/test/tests/conftest.py`](api/test/tests/conftest.py) keeps **`[pytest]`** session/collection progress so long phases after settings do not look like a hang.
+- **Django / pytest startup visibility**: [`api/utils/utils.py`](api/utils/utils.py) **`print_django`** uses **`flush=True`**. [`api/settings.py`](api/settings.py) logs that **`django.setup()`** follows settings. [`api/models.py`](api/models.py) logs before/after the **`api`** model import chain (often the long gap before **`api.urls`**). [`api/urls.py`](api/urls.py) logs start/end of URLconf import; [`api/apps.py`](api/apps.py) **`ApiConfig.ready()`** logs when the **`api`** app **`ready()`** runs. [`api/test/tests/conftest.py`](api/test/tests/conftest.py) keeps **`[pytest]`** session/collection progress.
 
 ### Changed
 

api/models.py

@@ -1,5 +1,17 @@
+import os
+import sys
+
+if "pytest" in sys.argv[0] or os.environ.get("ENV") == "ci_test":
+    print(
+        "[Django] api.models: importing SpotifyArtist / SpotifyLibTrack / User (deep trees; can be slow)...",
+        flush=True,
+    )
+
 from api.model.spotify_resource.children.artist.SpotifyArtist import SpotifyArtist
 from api.model.spotify_resource.children.track.SpotifyLibTrack import SpotifyLibTrack
 from api.model.user.User import User
 
+if "pytest" in sys.argv[0] or os.environ.get("ENV") == "ci_test":
+    print("[Django] api.models: core model imports finished.", flush=True)
+
 __all__ = ["SpotifyArtist", "SpotifyLibTrack", "User"]

api/settings.py

@@ -1,6 +1,5 @@
 # Standard library imports
 import datetime
-import importlib.util
 import os
 import sys
 from pathlib import Path
@@ -539,9 +538,6 @@ def setup_installed_apps_and_caches():
         "api",
     ]
 
-    if "pytest" in sys.argv[0] and importlib.util.find_spec("coverage") is not None:
-        INSTALLED_APPS.append("coverage")
-
     if APP_IS_EXPOSED:
         INSTALLED_APPS.append("rest_framework_simplejwt")
 

api/test/tests/conftest.py

@@ -8,10 +8,9 @@
 
 import pytest
 from _pytest.main import Session
+from django.conf import settings
 from django.test import override_settings
 
-from api import settings
-
 E2E_REACHABILITY_TIMEOUT_SEC = 5
 
 

pyrightconfig.json

@@ -2,6 +2,7 @@
   "pythonVersion": "3.14",
   "typeCheckingMode": "basic",
   "reportMissingImports": "warning",
+  "reportMissingModuleSource": false,
   "reportMissingTypeStubs": false,
   "pythonPlatform": "Darwin",
   "executionEnvironments": [

pytest.ini

@@ -2,7 +2,9 @@
 DJANGO_SETTINGS_MODULE=api.settings
 # Live logs at DEBUG swamp the terminal and look “hung”; use INFO by default.
 # For deep debugging: pytest -o log_cli_level=DEBUG
-log_cli = true
+# Live CLI logging can interact badly with Django/pytest startup ordering in CI (apparent "hang" with no output).
+# Enable locally when needed: pytest -o log_cli=true -o log_cli_level=INFO
+log_cli = false
 log_cli_level = INFO
 filterwarnings =
     ignore:unclosed file:ResourceWarning