Development #40730
python3: mise à jour des url de redirection mellon
Début:
13 mars 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Les tests ne passaient pas par là.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-python3-update-idp-login-redirection-40730.patch 0001-python3-update-idp-login-redirection-40730.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a environ 4 ans
Mis à jour par Nicolas Roche il y a environ 4 ans
- Lié à Development #40699: Tourner en python3 ajouté
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Je ack mais ça serait plus clair que les tests expliquent qu'ils mockent la présence d'un mellon configuré. Je propose :
- renommer le test test_mellon_idp_redirections.
- le résultat de get_idps mocké en [{'METADATA': '...'}] au lieu de "foo" (qui n'a pas trop de sens)
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-python3-update-idp-login-redirection-40730.patch 0001-python3-update-idp-login-redirection-40730.patch ajouté
- Statut changé de Solution validée à Solution proposée
Mis à jour par Thomas Noël il y a environ 4 ans
Et avec un coup de retard (désolé) je me dis qu'il faudrait avoir un next contenant des choses à encoder, pour vérifier que le quote() fait son job (genre "http%3A//foo/%3Fbar" à la place de "bar-url").
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-python3-update-idp-login-redirection-40730.patch 0001-python3-update-idp-login-redirection-40730.patch ajouté
oui, je n'y ai pas pensé.
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Retire l'inutile « assert unquote(resp.location) == ... » (on s'en fiche de tester la fonction unquote de Django, elle marche) et go.
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8c16da984d024504b85a90debab7c386918b78e4 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Fri Mar 13 17:18:50 2020 +0100 python3: update idp login redirection (#40730)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
python3: update idp login redirection (#40730)