Project

General

Profile

Tâche #21118

Scénario #21098: Faire le point sur les problèmes Eclair/gaspacho-agent

Levée d’une exception si le serveur répond que tout ne va pas bien

Added by Benjamin Bohard over 3 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
08/18/2017
Due date:
% Done:

100%

Estimated time:
1.00 h
Spent time:
Remaining (hours):
0.0

Description

Dans agent.py, il y a un test sur la réponse du serveur gaspacho. Si tout est OK (code 200 et raison OK), le code fait un raise (pas de la bonne manière puisque l’objet n’hérite pas de BaseException ni de object).
Il semble plus logique de lever une exception si le code est différent de 200 ou la raison différente de OK.

À voir quel opérateur booléen utiliser (and ou or).

agent.py.patch View (755 Bytes) Benjamin Bohard, 08/18/2017 02:14 PM

Associated revisions

Revision 11df4ba7 (diff)
Added by Emmanuel GARETTE over 3 years ago

vérifier correctement le status renvoyé par le serveur (ref #21118 @1h)

Revision a5260d82 (diff)
Added by Emmanuel GARETTE over 3 years ago

vérifier correctement le status renvoyé par le serveur (ref #21118 @1h)

Revision 60b6ffe5 (diff)
Added by Emmanuel GARETTE over 3 years ago

Revert "vérifier correctement le status renvoyé par le serveur (ref #21118 @1h)"

This reverts commit 11df4ba7aabb1c5812ed32e92e41cae9946dc6e7.

History

#1 Updated by Scrum Master over 3 years ago

  • Tracker changed from Demande to Tâche
  • Parent task set to #21098

#2 Updated by Emmanuel GARETTE over 3 years ago

  • Subject changed from Levée d’une exception si tout le serveur répond que tout va bien. to Levée d’une exception si le serveur répond que tout ne va pas bien
  • Status changed from Nouveau to En cours
  • Assigned To set to Emmanuel GARETTE
  • % Done changed from 0 to 100
  • Estimated time set to 1.00 h
  • Remaining (hours) set to 0.25

#3 Updated by Emmanuel GARETTE over 3 years ago

  • Project changed from Gaspacho Agent to Eclair
  • Status changed from En cours to Résolu

#4 Updated by Laurent Flori over 3 years ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) changed from 0.25 to 0.0

Also available in: Atom PDF