Larastan に自作 OSS ライブラリのテストをぶっ壊された話

PHPerKaigi 2023 Day1 アンカンファレンス

Transcript

1. Larastan に自作 OSS ライブラリ のテストをぶっ壊された話 PHPerKaigi 2023 Day1 アンカンファレンス @mpyw
2. ある日，突然自作ライブラリのテストが壊れました
3. 発端 PDO のエミュレーションを切り替えるテスト
4. 発端 PDO のエミュレーションを切り替えるテスト RefreshDatabase はコネクションを解決してしまうが 使っていない
5. CI だけ何故か落ちるようになってしまった…
6. ローカルと CI の微妙な違い • ローカルでは…？ ◦ vendor/bin/phpunit • CI では…？ ◦ vendor/bin/phpstan ◦ vendor/bin/phpunit
7. ローカルと CI の微妙な違い • ローカルでは…？ ◦ vendor/bin/phpunit • CI では…？ ◦ vendor/bin/phpstan ◦ vendor/bin/phpunit PHPStan を走らせると その後のテストは何回実行しても落ちる
8. PHPStan の実行で何が起こったのか？
9. PHPStan の実行で何が起こったのか？ orchestra/testbench はライブラリの テスト用に laravel/laravel をいい感じに 用意してくれるテストフレームワーク
10. Larastan くん勝手に vendor 配下触るの！？
11. 差分を探す
12. 差分を探す 新規作成された vendor 自体は怪しくなかった ↓ 既存ファイルが書き換えられているのでは…？
13. 差分を探す
14. 差分を探す パッケージディスカバリが走ってそう
15. Larastan がどこかのバージョンから パッケージディスカバリを orchestra/testbench 内 の laravel/laravel に対して行うようになった…
16. 差分を探す orchestra/testbench が spatie/laravel-ray に依存している
17. サービスプロバイダから呼ばれる処理で… 立ち上げ時に DB::connection() を解決している
20. というか Larastan がお節介すぎるやろ…
21. nunomaduro/larastan/issues/1588 に立てました
22. nunomaduro/larastan/issues/1588 に立てました
23. laravel-ray 側で直すんかい！ｗ
24. 遅延評価されるように修正された（1日前）
25. まとめ • Larastan は勝手に orchestra/testbench 上の依存関係で パッケージディスカバリを実行してくる • DB::connection() が解決される副作用に限っては spatie/laravel-ray 側で 修正されたが，本質的な問題に対する違和感は拭えない