Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
PHPカンファレンス小田原2024
Search
Kanon
April 12, 2024
Technology
5
720
PHPカンファレンス小田原2024
Kanon
April 12, 2024
Tweet
Share
More Decks by Kanon
See All by Kanon
ソフトウェアエンジニア観に影響を与えたアニメ・漫画の名言
ysknsid25
0
22
PHP"オレ"カンファレンスの告知
ysknsid25
0
420
なぜ人は組織から去っていくのか?
ysknsid25
0
44
Laravel Sail9から導入された Mailhogの後継Fake SMTP/mailpit を使ってみた
ysknsid25
0
41
GASとChatGPTを組み合わせてZennとQiitaの急上昇記事を紹介するTwitter botを作った
ysknsid25
0
14
PHPカンファレンス関西2024
ysknsid25
0
710
アジャイル勉強法〜自分という製品を開発する〜
ysknsid25
0
140
PHPカンファレンス関西2024は神戸で過ごそう
ysknsid25
2
610
要件を理解するために、非エンジニアと一緒に業務をこなした話
ysknsid25
1
89
Other Decks in Technology
See All in Technology
MLOpsの「壁」を乗り越える、LINEヤフーの Data Quality as Code
lycorptech_jp
PRO
7
570
AWSに詳しくない人でも始められるコスト最適化ガイド
yuhta28
2
260
本当のAWS基礎
toru_kubota
1
550
私が trocco を推す理由
__allllllllez__
2
270
Android Target SDK 35 (Android 15) 対応の概要
akkie76
0
120
コードや知識を組み込む / Incorporate Code and knowledge
ks91
PRO
0
110
AWS学習者向けにAzureの解説スライドを作成した話
handy
3
110
アクセシビリティを考慮したUI/CSSフレームワーク・ライブラリ選定
yajihum
2
1k
ゼロから始めるVue.jsコミュニティ貢献 / first-vuejs-community-contribution-link-and-motivation
lmi
1
140
今年のRubyKaigiはProfiler Year🤘
osyoyu
0
210
Gitlab本から学んだこと - そーだいなるプレイバック / gitlab-book
soudai
6
1.2k
推しは推せるときに推せ! プロダクトにフィードバックしていこう
nakasho
0
420
Featured
See All Featured
XXLCSS - How to scale CSS and keep your sanity
sugarenia
242
1.2M
Code Reviewing Like a Champion
maltzj
515
39k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
11
8.3k
The Success of Rails: Ensuring Growth for the Next 100 Years
eileencodes
31
6k
Mobile First: as difficult as doing things right
swwweet
217
8.6k
No one is an island. Learnings from fostering a developers community.
thoeni
16
2.1k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
3
3.4k
Cheating the UX When There Is Nothing More to Optimize - PixelPioneers
stephaniewalter
275
13k
Building Applications with DynamoDB
mza
88
5.6k
Put a Button on it: Removing Barriers to Going Fast.
kastner
58
3.1k
From Idea to $5000 a Month in 5 Months
shpigford
378
45k
Art, The Web, and Tiny UX
lynnandtonic
290
19k
Transcript
Copyright © 2023 blessing software. All Rights Reserved. Illustrated by
@amon_mikio Kanon (@samurai_se) テスト品質を向上させよう! 〜アンチパターン回避メソッド〜
1. 自己紹介 2 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio Kanon 株式会社 虎の穴ラボ samurai_se ↓詳しくは↓ • 3次元に嫁が1人います。2次元にはたくさんいます。 • 本業はKtor(Kotlin), Next.js(TypeScript)で副業がLaravel(PHP), Next.js(TypeScript) • アニメと漫画と声優ラジオが好きです • 推しは水瀬いのりさんと早見沙織さんです 個人事業 blessing software
3 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio すこしだけ宣伝
どこからきたー? 4 Copyright © 2023 blessing software. All Rights Reserved.
Illustrated by @amon_mikio
5 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio Re: すこしだけ宣伝
アジェンダ 6 • 明日からすぐにでもできること • 継続的に取り組む必要があること • さらにその先へ・・・ Copyright ©
2023 blessing software. All Rights Reserved. Illustrated by @amon_mikio
❗ CAUTION ❗ 7 Copyright © 2023 blessing software. All
Rights Reserved. Illustrated by @amon_mikio
❗ CAUTION ❗ 8 • 本業と副業の事例が入り混じります ◦ 必ずしも、とらラボの事例ではないです ◦ どっちでもやっていることもあります
• 普段からいろんな言語のテストを書いてます ◦ PHPUnitに限った話ではないです ◦ が、基本PHP前提でお話します ◦ 本業ではJest/Cypress/JUnit ◦ 副業ではJest/Cypress/PHPUnit ◦ サンプルコードに言語混ざることをお赦しください・・・ Copyright © 2023 blessing software. All Rights Reserved. Illustrated by @amon_mikio
🚀 明日からすぐにでもできること Copyright © 2023 blessing software. All Rights Reserved.
Illustrated by @amon_mikio
💡 テスト名には3つの要点を含め、明確にする 10 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio X は Y のとき Z 何をテストす るか? どのような状 況とシナリオ か? 期待される結 果は何か?
💡 テスト名には3つの要点を含め、明確にする 11 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio • isExistPrefecture()をチェックする アンチパターン テストコードがド キュメントの役割 を果たさない
💡 テスト名には3つの要点を含め、明確にする 12 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio • isExistPrefecture()をチェックする アンチパターン エラー時、何が悪 いのかテスト名か ら判別できない
💡 テスト名には3つの要点を含め、明確にする 13 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio • isExistPrefecture()をチェックする アンチパターン 具体的に何を検証 していて、期待し ていることはな に? チェックって、な にを?
💡 テスト名には3つの要点を含め、明確にする 14 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio • isExistPrefecture()は存在しない都道府県を指定したとき エラーとなる • validateStringValue()はrequiredがfalseのとき空文字でも エラーとならない 日本人しかいないプロジェクトならテスト名は日本語でいい派です 例
💡 AAAパターンでテストを構成する 15 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio • Arrange (準備) • Act (実行) • Assert (検証) 音楽グループではない・・・
💡 AAAパターンでテストを構成する 16 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio アンチパターン どこまでが準備でどこからが実行かわかりづらい
💡 AAAパターンでテストを構成する 17 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio 同じテストコードだけどスッキリ 一番わかりやすいサンプル→
💡 パラメタライズド テストで多様な入力値を検証 18 Copyright © 2023 blessing software. All
Rights Reserved. Illustrated by @amon_mikio プロダクションコード DRYとは・・・? なコードになっている
💡 パラメタライズド テストで多様な入力値を検証 19 Copyright © 2023 blessing software. All
Rights Reserved. Illustrated by @amon_mikio アンチパターン DRYとは・・・? なコードになっている
💡 パラメタライズド テストで多様な入力値を検証 20 Copyright © 2023 blessing software. All
Rights Reserved. Illustrated by @amon_mikio 入力に対して複数の結果を得られるパターンは、網羅的 かつ一つのテストケースで検証する
💡 エラーはキャッチせず、エラーを期待する 21 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio アンチパターン 何してるのかわかりにくスギィィィィ ただしテスト名がまともだと まだかろうじて把握はできる
💡 エラーはキャッチせず、エラーを期待する 22 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio 冗長さが消え、テストメソッド名と実装が合っているのでとても直感的
🕰 継続的に取り組む必要があること Copyright © 2023 blessing software. All Rights Reserved.
Illustrated by @amon_mikio
💡 適切なコードカバレッジを保つ 💡 24 Copyright © 2023 blessing software. All
Rights Reserved. Illustrated by @amon_mikio
💡 カバレッジとは? 25 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio • テストコードによって到達されるコード行の尺度で基準としては以下 • 分岐(Branch) ◦ 条件分岐文でどれだけの分岐条件が実行されたか • 行(Lines) ◦ コードの各行がテストスイートで実行されたか。空行や{}だけの行も含む • ステートメント(Stmt) ◦ プログラム内の命令単位。なので空行や{}だけの行は含まない • 関数(Funcs) ◦ コード内の関数が実行されたかどうか
💡 適切なコードカバレッジを保つ 26 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio • コードカバレッジは50~80%近辺を目安とするのがよさそう • 10~40% ◦ 低すぎる • 90%以上 ◦ 高すぎる ◦ プロダクトのクリティカルな実装部分を超え、重箱の隅にまで視点をおくこ とは好ましくない
💡カバレッジ閾値に満たない場合、CIを中止する 27 Copyright © 2023 blessing software. All Rights Reserved.
Illustrated by @amon_mikio • Laravel 9から --coverage --min=n オプションが追加 • 閾値に満たない場合、テストが失敗する • 最初は、カバレッジレポートからわかった全体カバレッジの-5%あたりか ら始める • 徐々に閾値を上げていく とにかく初期リリースまでの開発速度を重視するなら必ずしも必要な設 定ではない ただ主観だが、最初からカバレッジを高く保っていた方が あとあとリファクタやリグレッションテストの工数を減らせて デグレのリスクを下げられるので、結果的に早い
💡カバレッジ閾値に満たない場合、CIを中止する(85) 28 Copyright © 2023 blessing software. All Rights Reserved.
Illustrated by @amon_mikio
💡カバレッジ閾値に満たない場合、CIを中止する(90) 29 Copyright © 2023 blessing software. All Rights Reserved.
Illustrated by @amon_mikio
🔍 カバレッジレポートを精査する🔍 30 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio
💡 例えばここだけ見ているとする 31 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio CIごとではなく定期実行でカバレッジを把握しているシチュ
💡 業務上、重要な処理がここにあったら? 32 Copyright © 2023 blessing software. All Rights
Reserved. Illustrated by @amon_mikio
33 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio 💡カバレッジレポートを精査し、テストされていない領域 やそのほかの異常を検知する • 閾値はトータルカバレッジを基準にしている • それだけを監視しても、結局プロダクトの重要な部分が処理できているかは わからない • 実際にカバレッジレポートを見て、ファイルごとのカバレッジを確認するの が重要(→定期的にSlack通知したりHTMLレポートを限定URLでホスティング するのが効果的) • また、100%だとしてもそれが効果的なテストかはわからない(次で話します)
🚪 さらにその先へ・・・ Copyright © 2023 blessing software. All Rights Reserved.
Illustrated by @amon_mikio
🧪 Mutation Testによるテストコード自体の検証 🧪 35 Copyright © 2023 blessing software.
All Rights Reserved. Illustrated by @amon_mikio
36 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio Mutation Testとは コードに意図的なバグを植え付けることで、 テストコードの検証が適切に行われているか? を測定する手法
37 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio カバレッジレポートの罠 • 従来のカバレッジメトリクスは嘘をつく • 例えば以下の(極端な)テストケースを見てください
38 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio Pest(PHPUnitも?)はassertionがなければwarningを吐く
39 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio 以下の(これまた極端な)例の場合は検知できない 適切な表示がされているか? を確認できていないが、テスト が通っている しかしプロダクションコード自 体は実行されているため、カバ レッジの%は上がる
40 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio そこでMutation Testing • ここではInfectionを使う前提で話します • Mutation = 突然変異 ◦ コードを意図的に変更し、バグを植え付ける ◦ ex) a===0 を a!==0と変異させる ◦ その後テストを実行し、正しいテストが書かれて いればエラーとなるはず ◦ エラーとならなかった箇所がきちんと検証されて いないと判断できる 変異の内容 Infectionの詳細については ぺちこん関西の資料をみてね
41 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio Mutation Testingに関わる指標 • Killed ◦ 変異後、失敗すべきテストが失敗したことにより 検知された変異の数 • Survived ◦ 変異後、失敗すべきテストが成功したことにより 検知された変異の数 つまり、Survivedの数が多ければ多いほどテストコードの品質が低い
42 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio Infectionの導入 • 定期実行で全体のMutation Scoreを継続的に比較 ◦ 全体実行はかなり遅い ◦ ので週一程度でGHAを動かし、レポートをSlackに投げる • CIで差分発生箇所に対してMutation Test ◦ Mutation Scoreが一定を下回るとCIがコケる設定 ◦ 新たに追加されたコードや修正分に関しては一定以上のMutation Scoreを担保できる
🤫 実戦導入した経験談はまた別のカンファレンスにて… 🤫 43 Copyright © 2023 blessing software. All
Rights Reserved. Illustrated by @amon_mikio
44 Copyright © 2023 blessing software. All Rights Reserved. Illustrated
by @amon_mikio ご清聴、あざざました