mat/python-comwatt-client
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

✅ Fix CI: switch_capacity tests expect lowercase enable #9

Merged
mat merged 1 commit from tests/fix-switch-capacity-lowercase into main 2026-07-04 07:52:52 +00:00
Conversation 0 Commits 1 Files changed 1 +2 -2
mat commented 2026-07-04 07:50:17 +00:00
Owner
Copy link

Fix CI: aligner les tests switch_capacity sur la valeur enable minuscule

Après le merge, la CI échoue sur main (et donc sur le miroir GitHub). Deux tests seulement sont en cause.

Cause

La PR #7 (« Send lowercase boolean in switch_capacity URL ») a changé le code pour envoyer enable=true/enable=false en minuscules (str(enable).lower()). Mais la suite de tests (mergée séparément via #2, écrite avant #7) asserte encore les anciennes valeurs capitalisées True/False :

E   AssertionError: assert ['true'] == ['True']
E   AssertionError: assert ['false'] == ['False']

Les autres changements mergés entre-temps ne cassent rien : les tests d'erreur n'assertaient que le code de statut (pas le libellé), donc le fix des messages (#5) passe ; le timeout (#8) et les docstrings (#6) n'affectent aucune assertion.

Correctif

tests/test_devices.py — les deux assertions switch_capacity attendent désormais ["true"] / ["false"], conformes au comportement actuel du client.

Vérification

  • python -m pytest en venv neuf → 24 passed, sortie propre.
  • Échec GitHub reproduit puis résolu (mêmes 2 tests, mêmes assertions).
## Fix CI: aligner les tests `switch_capacity` sur la valeur `enable` minuscule Après le merge, la CI échoue sur `main` (et donc sur le miroir GitHub). Deux tests seulement sont en cause. ### Cause La PR #7 (« Send lowercase boolean in switch_capacity URL ») a changé le code pour envoyer `enable=true`/`enable=false` en minuscules (`str(enable).lower()`). Mais la suite de tests (mergée séparément via #2, écrite avant #7) asserte encore les anciennes valeurs capitalisées `True`/`False` : ``` E AssertionError: assert ['true'] == ['True'] E AssertionError: assert ['false'] == ['False'] ``` Les autres changements mergés entre-temps ne cassent rien : les tests d'erreur n'assertaient que le code de statut (pas le libellé), donc le fix des messages (#5) passe ; le timeout (#8) et les docstrings (#6) n'affectent aucune assertion. ### Correctif `tests/test_devices.py` — les deux assertions `switch_capacity` attendent désormais `["true"]` / `["false"]`, conformes au comportement actuel du client. ### Vérification - `python -m pytest` en venv neuf → **24 passed**, sortie propre. - Échec GitHub reproduit puis résolu (mêmes 2 tests, mêmes assertions).
✅ Align switch_capacity tests with lowercase enable value
All checks were successful
Tests / forgejo-pytest (push) Successful in 36s
Details
Tests / forgejo-pytest (pull_request) Successful in 38s
Details
f056c39344 
PR #7 changed switch_capacity to send enable=true/false (lowercase) via
str(enable).lower(). The tests still asserted the old capitalized True/False,
which broke CI on the merged main (and on the GitHub mirror). Update the two
assertions to expect the lowercase values.
mat merged commit 2287ee3315 into main 2026-07-04 07:52:52 +00:00
mat deleted branch tests/fix-switch-capacity-lowercase 2026-07-04 07:52:53 +00:00
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
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!9
No description provided.