Suggestion: Make `amOnPage()` wait for JavaScript to finish by default
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez au point d’entrée amOnPage() et comparez son comportement actuel de navigation avec l’exemple waitForJS() de l’issue. Avant de modifier le comportement par défaut, résolvez la rupture de compatibilité ascendante proposée et définissez le comportement attendu du paramètre optionnel ; le travail est terminé lorsque le comportement convenu est implémenté et vérifié pour la navigation avec et sans attente.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Starting point: Everybody always wants JavaScript to finish, before performing any action/assertion on the page, right?
At least my usual pattern is:
$I->amOnPage('/');
$I->waitForJS('return document.readyState === "complete";', 5); // https://gist.github.com/pastuhov/43674b195dc293ffd847
So I thought about ways to simplify this. My first idea was to introduce a new method, something like waitForJSToComplete().
But now, I think it's even better to include this in amOnPage(), by introducing a second parameter:
public function amOnPage($page, bool $waitTillJSIsComplete = true): void
This would be a BC break.
- Langage dominant
- PHP
- Étoiles
- 39
- Forks
- 25
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Codeception/module-webdriver
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
Codeception/module-webdriver#135 ·
-
Example of `executeAsyncJs()` Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 38/100
Codeception/module-webdriver#130 ·
-
Bug: `MalformedLocatorException` thrown for `id`s that don't start with a character (HTML 5) Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
Codeception/module-webdriver#128 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
Codeception/module-webdriver#126 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
Codeception/module-webdriver#122 ·
Toutes les issues de Codeception/module-webdriver
Issues similaires
-
tooling
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
UX
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
bug customer-reported
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
MagnaCapax/PMSS#949 ·
-
responsive-media kses allow-list strips clipPathUnits, collapsing objectBoundingBox clip paths Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Automattic/static-site-importer#1829 ·