Correctif pour étiquette "Non disponible" incorrecte sur le flux SIRI de Boulogne #4284
Conversation
| @@ -56,6 +56,15 @@ defmodule Transport.AvailabilityCheckerTest do | |||
|
|
|||
| assert AvailabilityChecker.available?("SIRI", "url303") | |||
| end | |||
|
|
|||
| test "500 response (but with SOAP body)" do | |||
There was a problem hiding this comment.
Peux-tu ajouter un test sans du SOAP en réponse ?
There was a problem hiding this comment.
Yes, j'ai fait ça dans eb3b5c3
|
Merci pour le fix 🙏 Tu confirmes que tu vas laisser ainsi les données historiques d'indisponibilité et les scores ? Ça peut être compliqué/long à nettoyer. |
@AntoineAugusti oui tout à fait, je ne vais pas chercher à corriger ça. Merci pour le reste ! |
|
La traduction est là J'étais sur mobile avant, désolé de pas avoir partagé ce lien. |
J'ai fait ça dans ff302fc (merci pour le lien PO). |
Voir:
Le serveur SIRI est techniquement bien joignable (si on met de côté, comme beaucoup d'autres flux, l'absence de
requestor_ref#4254), cette PR modifie l'availability check pour qu'il indique aux utilisateurs que c'est bien le cas (malgré le code 500 pas très pratique).Test rapide sous iex
(contre
falseavant la PR)./cc @Brewennn pour information
Écran "avant"