Bug #35740
correction d'une typo dans les tests
Début:
02 septembre 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
def test_api_users_get_or_create(settings, app, admin): ... paylaod = {}
Fichiers
Historique
Mis à jour par Nicolas Roche il y a plus de 4 ans
La présence de payload={}
laissait sous-entendre que l'on testait l'API avec un payload vide.
Du coup j'ai rajouté ce cas.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-tests-correct-a-type-into-api-tests-35740.patch 0001-tests-correct-a-type-into-api-tests-35740.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 4 ans
Le test :
with pytest.raises(TypeError, match="email.* missing"): resp = app.post_json('/api/users/?update_or_create=email', params=payload, status=201)
me semble plutôt être le signe d'un bogue, non ? Pourquoi (par qui) une exception TypeError se retrouve levée ?
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0002-.-ne-pas-retourner-une-erreur-500-via-l-API.patch 0002-.-ne-pas-retourner-une-erreur-500-via-l-API.patch ajouté
un petit trou dans la raquette, que j'essaye de corriger au passage.
Mis à jour par Thomas Noël il y a plus de 4 ans
Nicolas Roche a écrit :
un petit trou dans la raquette, que j'essaye de corriger au passage.
Oui mais autre problème, autre ticket, stp...
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-tests-add-a-test-for-wrong-user-get_or_create-api-us.patch 0001-tests-add-a-test-for-wrong-user-get_or_create-api-us.patch ajouté
oui, d'autant plus que l'exception va disparaître avec #35710.
Donc, nouveau patch après rebasage.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-tests-add-a-test-for-wrong-user-get_or_create-api-us.patch 0001-tests-add-a-test-for-wrong-user-get_or_create-api-us.patch ajouté
aïe, patch tout pourri désolé.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution proposée à Fermé
sur avis de mes collègues parisiens, je ferme ce ticket.
(le patch n'a plus rien à voir avec le titre)