Codeception / Codeception/module-symfony

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

オープン 初心者向け
#245 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
PHP
スター
96
フォーク
25
平均マージ
2時間 35分
マージ済み PR(30日)
3

説明

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Start in src/Codeception/Module/Symfony/RouterAssertionsTrait.php at seeCurrentRouteIs(), and compare the behavior introduced by pull request #227. Reproduce the documented call with an integer id, then add or update a regression test if the existing test suite has coverage for this assertion. Done means prior calls continue comparing integer route parameters without converting them to strings.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
php, symfony
領域
backend, testing
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
68/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。