リーダブルテストコード / #vstat

「リーダブルなテストコードについて考えよう ～VeriServe Test Automation Talk No.3～」で使用したスライドです。 https://veriserve-event.connpass.com/event/243280/ 登壇動画はこちらで公開されています。 https://vimeo.com/742517199/e001ac43ac ＜参考リンク＞ Twitter https://twitter.com/jnchito Blog https://blog.jnito.com/ Qiita https://qiita.com/jnchito プロを目指す人のためのRuby入門［改訂2版］ https://gihyo.jp/book/2021/978-4-297-12437-3 Everyday Rails - RSpecによるRailsテスト入門 https://

[touka_tt]

"テストコードにおいて過度なDRYは読みやすさの敵"めっちゃわかる....

[inabajunmr]

全く構造化しないとそれぞれは読みやすいけど全体のボリュームが多くて仕様として網羅されてるかどうかがわかりづらい、見るのがしんどいのでレビューの精度が下がる、みたいなことはある気がしていてバランスに悩む

[umai_bow]

わかる。でも組み合わせで爆発するようなケースはある程度構造化しないといけないんだよな・・・

[gonta616]

よくわからんがテスト通ってるからよし！

[auto_chan]

Excelでパターンマトリックス書いてテストコードにぺたっと貼り付けるくらいがちょうどいい。

[hnishi2509]

質疑応答から、テストでは、定数などをソースコードから取得するよりベタ書きの方が好ましい、ソースコードの更新に付随してテストが通るとバグが含まれていた場合、気付けない場合があるため、独立させたほうが良い

[nilab]

「「リーダブルなテストコードについて考えよう ～VeriServe Test Automation Talk No.3～」で使用したスライドです」

[nobeans]

完全に同意。が、微妙に異なる条件での振舞いの違いをわかりやすく列挙する方が読みやすい＆メンテしやすい場合も多く、パラメタライズドテストとして書いて実行部分は割とロジック的になる、というパターンが多い。

[ch1248]

全般的に同意できる内容だった。

[erya]

dryって改修漏れ防ぐ意図もあると思うんだけど、そこはテストコードだからいいのかな？ 実装は変数で、レビューモードはインライン展開されるとかだといいよね。。

[tg30yen]

ケースバイケース。対象データだけ違う処理が同じテストを別々に10回書くのがわかりやすいとは思えない。処理を共通化すればテストの処理はすべて同じであることがひと目で分かる。

[Wafer]

話の持っていきかたが上手い。id:inabajunmr そんなときは分割して小さな単体テストの集合にすればよいのでは？一つ一つはリーダブルで、分割が適切にされていて、かつボタン一個で実行は始められるようにする

[tettekete37564]

あー、テストコードでの DRY 悩んでた時期たしかにある。今はむしろ非 DRY 寄りにする方向に固まってきた。

[motchang]

DRYなテストコードがアンチパターンというジュニアの直感に反する事実、権威やインフルエンサーの人々にもっと喧伝してほしい。フィードバックの言葉を選ぶのに消耗しているレビュアー世界に山ほどおる。

[yarumato]

“リーダブル（＝誰でも読める愚直）なテストコードとは、文書のように上から下に読み下せる（⇔ループや条件分岐）。変数を使わずに文字列や数値をべ夕書き（＝脳内メモリ消費 小）。過度なDRY（重複排除）は敵”

[Mofuyuki]

すばら

[at_yasu]

python の doctest を思い出した

[yodelx]

ベタ書きヨシ！

[arakash]

読みやすいテストコードの話。自分の実感とかなり合ってる印象なので整理に役立ちそう。

[kzm1760]

よく分からないけど、こういう風に書けるのか〜学びになったからヨシ！はやってしまいがち。気をつけよう。

[takeag]

変数入れずにベタ書きもケースによるかも。ベタ書き要素が多くなると、今度はどれとどれが対応しているかに脳内リソースを消費し始めるからバランスを見て決めていきたい。

[Phenomenon]

テストにランダムな値を使うのは普通に辛いですね

[Magicant]

結局は「コードは短ければ短い程だいたい読みやすい」法則に約される

[remonoil]

システム日付変えてるのがもやる

[kumicho24]

ためになる

[shwld]

“”

[arx0balest]

概ね同意。そしてテストは AAA pattern (Arrange-Act-Assert パターン) にしなさい。それだけだ。