プルリク出したので
経緯とかまとめてみる。こんなことあったよー褒めてーくらいの記事です。
ただ、教訓となったのは、
『『『自動テストってまじで大切！！！』』』
問題となった内容
cacheのhelper関数おかしくないかな？ってことでプルリク出しました。
出したプルリク
見つけた経緯
Laravelのリファレンスを見ながら、コードを実装してたら、サンプルコード動かない。
cacheヘルパ関数の元のソースコード見る。ここおかしくね？ってなる。
他の人と一緒にコード見る。『バグじゃね？』
簡単なコードだったので、その日の内に修正してプルリクを出しました。
コード書いてるときの話。
公式のテストコードのままだと、テスト通っちゃうから修正をしてみる。で、やっぱりエラー出た。
そこから自信満々に修正のコードを書く。
テストしてみる。動かねぇ・・・ｗ
ってことでまた書き直したら通りました。
『『『自動テストってまじで大切！！！』』』
今後
とりあえず待ちます。
ぶっちゃけこの修正が正しいかわかんねぇ！
私PHPもLaravelも触ってようやく2年目だもん( ；∀；)
ただ、テストコードまで修正したし、このテストコードが間違ってなかったら、
そんなに悪くないコードだと思う！
ワクワクドキドキですね。軽微なバグなので、スルーされる可能性もありますが、
いい経験になりました。
