Codeception / Codeception/module-symfony
Backwards compatibility break in `RouterAssertionsTrait::seeCurrentRouteIs()`
- Langage dominant
- PHP
- Étoiles
- 96
- Forks
- 25
- Merge moyen
- 2 h 35 min
- PR mergées (30 j)
- 3
Description
https://github.com/Codeception/module-symfony/pull/227 broke backwards compatibility with tests written for prior versions of the module.
The following test now fails:
```php
$I->seeCurrentRouteIs('some_route', [
'id' => 1,
]);
```
With:
```
- Expected | + Actual
@@ @@
Array &0 [
'_route' => 'some_route',
- 'id' => 1,
+ 'id' => '1',
]
```
This usage is even how the method is documented:
https://github.com/Codeception/module-symfony/blob/01cfa3e12917e396cf99cee1fb63a553865cad16/src/Codeception/Module/Symfony/RouterAssertionsTrait.php#L87
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez dans src/Codeception/Module/Symfony/RouterAssertionsTrait.php, au niveau de seeCurrentRouteIs(), et comparez le comportement introduit par la pull request #227. Reproduisez l’appel documenté avec un id entier, puis ajoutez ou mettez à jour un test de régression si la suite de tests existante couvre cette assertion. Le travail est terminé lorsque les appels précédents continuent de comparer des paramètres de route entiers sans les convertir en chaînes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- php, symfony
- Domaine
- backend, testing
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 68/100