Development #63119
Virer le méchanisme du fichier 'unsecure'
0%
Description
Il y a là dedans un :
with open(os.path.join(tenant_dir, 'unsecure'), 'w') as fd:
fd.write('1')
pas vraiment en rapport avec l'objet du tests, qui produit au final des assertions absurdes :
assert service['base_url'] == 'https://hobo2.example.net/' ..... assert service['base_url'] == 'http://hobo2.example.net/'
dont incohérence est à chercher dans le tests lui même. Et plutôt que de démêler ça, simplifier le test.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
Où absurde ? De la description du ticket ça montre que la possibilité de poser un fichier unsecure dans le répertoire du tenant a bien un effet sur l'url annoncée.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de En cours à Nouveau
Frédéric Péters a écrit :
De la description du ticket
Lequel ? Un git blame me donne :
commit 8371097f3b86de5ace6f36ffd5e8d7193ed74868
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jan 4 08:58:31 2017 +0100
add basic multipublik deployment tests
En fait je ne comprends pas pourquoi les url changent dans les assertions alors que le fichier 'unsecure' est posé par la fixture qui est censée s'exécuter tout au début, j'en conclus que c'est un mic mac dans le test.
Pour compenser je peux écrire des tests à part qui vérifient que le 'unsecure' fait son taf.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
Emmanuel Cazenave a écrit :
Frédéric Péters a écrit :
De la description du ticket
Lequel ?
Je me répond, ici même le ticket. Et donc description pas assez précise, je pense que "les url changent dans les assertions alors que le fichier 'unsecure' est posé par la fixture qui est censée s'exécuter tout au début" répond à ta remarque.
Mis à jour par Frédéric Péters il y a environ 2 ans
Je me répond, ici même le ticket. Et donc description pas assez précise, je pense que "les url changent dans les assertions alors que le fichier 'unsecure' est posé par la fixture qui est censée s'exécuter tout au début" répond à ta remarque.
Ok donc. Mais la proposition semble être de cacher un problème.
Si/comme ce truc unsecure n'est de toute façon plus utilisé nulle part, autant le virer, que l'avoir avec un comportement qui n'est pas suivi ou compris.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-multitenant-give-up-unsecure-file-mechanism-63119.patch 0001-multitenant-give-up-unsecure-file-mechanism-63119.patch ajouté
- Sujet changé de Petite simplification de tests_multipublik à Virer le méchanisme du fichier 'unsecure'
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Frédéric Péters a écrit :
autant le virer
Effectivement, voilà.
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e0f7576d843b4282c1e6cf33b36567f567266a1b
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Wed Mar 23 16:47:35 2022 +0100
multitenant: give up 'unsecure' file mechanism (#63119)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
multitenant: give up 'unsecure' file mechanism (#63119)