Codeception / Codeception/module-symfony

Backwards compatibility break in `RouterAssertionsTrait::seeCurrentRouteIs()`

Ouverte Adaptée aux débutants
#245 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.