Repository navigation
fix(backoffice): la connexion attend son issue, pas une durée fixe - #117
Merged
Merged
Conversation
Après le clic, login() attendait 10 s le rechargement, puis isLoggedIn() 5 s le lien de déconnexion. Le premier tableau de bord après un vidage de cache dépasse parfois ces 15 s sur un runner CI chargé : la session était ouverte, la page encore en chargement, et la connexion était déclarée refusée (check « Visual 9.2.0 classic » de la #114, reproduit sur une boutique bridée). - Login\Page::waitForLoginOutcome() : attend le lien de déconnexion ou une alerte d'erreur avec du texte (plafond 60 s), appelée par login(). - VisualTestsSuite : le message d'erreur lit .alert-text (sans le « close » du bouton en 9.2) sur une ligne ; sans alerte, il donne la page courante (chemin et contrôleur, sans jeton). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problème
Le check obligatoire « Visual 9.2.0 classic » a échoué une fois sur la #114 : « Connexion au back-office impossible : identifiants refusés ou page inattendue ». Il est passé à la relance, sans changement de code.
Cause.
login()attendait 10 s le rechargement, puisisLoggedIn()5 s le lien de déconnexion.install-french.sh) dépasse parfois ces 15 s sur un runner chargé.Reproduction. Sur une boutique 9.2 Flashlight jetable, bridée à 0,25 CPU, avec le cache vidé et la page de connexion chauffée comme en CI :
AdminDashboard(readyState: loading) ;dev, la suiteBackOfficeéchoue avec le même message que le CI.Fréquence : 1 échec sur environ 135 exécutions BO dans les 45 derniers runs Visual.
Les scénarios utilisent le même
login()puisisLoggedIn(): ils échouent aussi à tort sur une boutique lente.Correctif
Login\Page::waitForLoginOutcome()login()quand il attend la navigation.Une boutique rapide ne perd rien : chacun des deux signaux arrête l'attente dès qu'il apparaît.
Message de
VisualTestsSuite.alert-text, sans le « close » du bouton de fermeture : en 9.2, c'était la seule ligne affichée ;(page : /admin-dev/?controller=AdminDashboard), sans le jeton.Vérification
vendor/bin/phpunit: 550 tests OK. Nouveaux tests :BackOfficeLoginOutcomeTest, plus deux cas dansVisualTestsSuiteBackOfficeTest.Boutique bridée, suite
BackOffice(login,dashboard) :dev🤖 Generated with Claude Code