🧪 Ajout d'une suite de tests unitaires + CI #2

Merged

mat merged 15 commits from tests/add-test-suite into main 2026-07-04 07:46:22 +00:00

Conversation 0 Commits 15 Files changed 10 +558

mat commented 2026-07-04 07:05:57 +00:00

Owner

Copy link

Ajout d'une suite de tests unitaires + CI

Le dépôt n'avait aucun test. Cette PR ajoute une suite de tests unitaires couvrant les 11 méthodes publiques de ComwattClient, plus une CI qui les exécute sur Forgejo et sur GitHub.

Tests

• Harnais : pytest.ini (testpaths = tests, filterwarnings = error), requirements-dev.txt (deps de test épinglées : pytest, responses), tests/conftest.py (constante BASE_URL + fixture client).
• 24 tests en 4 modules, chaque méthode ayant un cas succès et un cas erreur. HTTP entièrement bouchonné avec responses — aucun appel réseau.
  • test_authenticate.py — auth + hash SHA-256 exact du mot de passe, retour None.
  • test_user_and_sites.py — get_authenticated_user, get_sites.
  • test_aggregations.py — get_site_networks_ts_time_ago (les deux branches de aggregation_type), get_site_consumption_breakdown_time_ago, get_device_ts_time_ago.
  • test_devices.py — get_devices, get_device, put_device (corps JSON vérifié), switch_capacity (True/False).
• Les tests vérifient le comportement réel (méthode, URL/chemin, params parsés, corps JSON, valeur de retour), pas juste « un mock a été appelé ».
• Aucune modification du code de production. Un bug de message d'erreur existant (« Error retrieving sites » copié-collé sur des méthodes device) est documenté sans être figé (assertion sur le code de statut seul).

CI — deux fichiers (le modèle utilisé aussi dans agent-fleet.nvim)

Forgejo Actions ne lit que .forgejo/workflows/ dès que ce dossier existe (il ignore alors .github/workflows/), et il ne résout pas les refs d'actions courtes (actions/checkout@v4). D'où deux fichiers :

• .forgejo/workflows/tests.yml — pour git.greil.fr. Utilise https://data.forgejo.org/actions/checkout@v4. L'image ubuntu-latest du runner fournit python3 mais pas pip, donc le job crée un virtualenv (python3-venv) pour amorcer pip (ce qui évite aussi le blocage PEP 668 de Debian), puis installe les deps et lance pytest.
• .github/workflows/tests.yml — pour le miroir GitHub. Refs standard (actions/checkout@v4, actions/setup-python@v5), matrice Python 3.9→3.12, gardé par if: github.server_url == 'https://github.com'.

Vérification

• Local : python -m pytest → 24 passed, sortie propre (validé aussi en conteneurs python:3.12 et debian:bookworm/py3.11).
• CI Forgejo : verte sur push et pull_request.

## Ajout d'une suite de tests unitaires + CI Le dépôt n'avait **aucun test**. Cette PR ajoute une suite de tests unitaires couvrant les 11 méthodes publiques de `ComwattClient`, plus une CI qui les exécute sur Forgejo et sur GitHub. ### Tests - **Harnais** : `pytest.ini` (`testpaths = tests`, `filterwarnings = error`), `requirements-dev.txt` (deps de test épinglées : `pytest`, `responses`), `tests/conftest.py` (constante `BASE_URL` + fixture `client`). - **24 tests** en 4 modules, chaque méthode ayant un cas *succès* **et** un cas *erreur*. HTTP entièrement bouchonné avec `responses` — aucun appel réseau. - `test_authenticate.py` — auth + hash SHA-256 exact du mot de passe, retour `None`. - `test_user_and_sites.py` — `get_authenticated_user`, `get_sites`. - `test_aggregations.py` — `get_site_networks_ts_time_ago` (les deux branches de `aggregation_type`), `get_site_consumption_breakdown_time_ago`, `get_device_ts_time_ago`. - `test_devices.py` — `get_devices`, `get_device`, `put_device` (corps JSON vérifié), `switch_capacity` (`True`/`False`). - Les tests vérifient le **comportement réel** (méthode, URL/chemin, params parsés, corps JSON, valeur de retour), pas juste « un mock a été appelé ». - **Aucune modification du code de production.** Un bug de message d'erreur existant (« Error retrieving sites » copié-collé sur des méthodes device) est documenté sans être figé (assertion sur le code de statut seul). ### CI — deux fichiers (le modèle utilisé aussi dans `agent-fleet.nvim`) Forgejo Actions ne lit **que** `.forgejo/workflows/` dès que ce dossier existe (il ignore alors `.github/workflows/`), et il ne résout pas les refs d'actions courtes (`actions/checkout@v4`). D'où deux fichiers : - **`.forgejo/workflows/tests.yml`** — pour `git.greil.fr`. Utilise `https://data.forgejo.org/actions/checkout@v4`. L'image `ubuntu-latest` du runner fournit `python3` mais **pas `pip`**, donc le job crée un virtualenv (`python3-venv`) pour amorcer pip (ce qui évite aussi le blocage PEP 668 de Debian), puis installe les deps et lance `pytest`. - **`.github/workflows/tests.yml`** — pour le miroir GitHub. Refs standard (`actions/checkout@v4`, `actions/setup-python@v5`), matrice Python 3.9→3.12, gardé par `if: github.server_url == 'https://github.com'`. ### Vérification - Local : `python -m pytest` → **24 passed**, sortie propre (validé aussi en conteneurs `python:3.12` et `debian:bookworm`/py3.11). - CI Forgejo : **verte** sur `push` et `pull_request`.

mat added 5 commits 2026-07-04 07:05:58 +00:00

✅ Add test harness and authenticate tests ... 3979ab3e3e

- pytest.ini: testpaths=tests, filterwarnings=error for pristine output
- requirements-dev.txt: pytest==9.1.1, responses==0.26.2
- tests/conftest.py: client fixture + BASE_URL constant (shared convention)
- tests/test_authenticate.py: success and error tests for authenticate(),
  verifying request method/URL/body and return value against mocked HTTP

✅ Add tests for get_authenticated_user and get_sites 4c7d25b6b8

✅ Add tests for aggregation endpoints 4e3ed385a3

✅ Add device mutation endpoint tests 5a39985c3d

👷 Add CI workflow to run test suite on push/PR 795cbaa3cf

mat added 1 commit 2026-07-04 07:11:33 +00:00

👷 Fix CI: Forgejo needs full-URL action refs ... bf63b8fa31

Forgejo does not resolve bare 'actions/checkout@v4' refs on this instance,
so the job failed at setup. Add .forgejo/workflows/tests.yml using
https://data.forgejo.org/actions/* refs, and guard the .github one with
'if: github.server_url == ...' so Forgejo (which also reads .github/workflows)
skips it and only GitHub runs it.

mat added 1 commit 2026-07-04 07:19:25 +00:00

🔧 CI diagnostic: distinct job names, simplify Forgejo workflow ... 1bf39a24f0

Rename jobs (forgejo-pytest / github-pytest) to identify which workflow
Forgejo actually executes, and reduce the Forgejo job to a single Python
version to isolate the planning failure.

mat added 1 commit 2026-07-04 07:22:06 +00:00

🔧 CI diagnostic: drop setup-python, use image python3 1fb2b234da

mat added 1 commit 2026-07-04 07:24:02 +00:00

🔧 CI: pull setup-python from github.com (mirror unusable) 96df5e5521

mat added 1 commit 2026-07-04 07:27:59 +00:00

🔧 CI: run Forgejo job in python:3.12 container 08c079f3fa

mat added 1 commit 2026-07-04 07:34:16 +00:00

🔧 CI bisect: trivial container step only 38fd68d4a5

mat added 1 commit 2026-07-04 07:35:25 +00:00

🔧 CI bisect: container + checkout 85cc9da328

mat added 1 commit 2026-07-04 07:37:18 +00:00

🔧 CI bisect: default image + checkout + setup-python (sanity only) e3a5d36f51

mat added 1 commit 2026-07-04 07:39:12 +00:00

🔧 CI probe: enumerate default runner image capabilities 98170e5a17

mat added 1 commit 2026-07-04 07:42:08 +00:00

🔧 CI: bootstrap pip via venv (runner image ships python3 without pip) a20882f554

mat merged commit f8d126408f into main 2026-07-04 07:46:22 +00:00

mat deleted branch tests/add-test-suite 2026-07-04 07:46:22 +00:00

mat referenced this pull request from a commit 2026-07-04 07:46:23 +00:00

🧪 Ajout d'une suite de tests unitaires + CI (#2)

mat referenced this pull request 2026-07-04 07:50:17 +00:00

✅ Fix CI: switch_capacity tests expect lowercase enable #9

mat referenced this pull request from a commit 2026-07-04 07:52:53 +00:00

✅ Fix CI: switch_capacity tests expect lowercase enable (#9)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

mat/python-comwatt-client!2

No description provided.