TDDチュートリアル

Apr. 10th 2019 Android Tests Hands-on TDDチュートリアル @hoddy3190

鈴木穂高（Hodaka Suzuki） Twitter @hoddy3190 • 2014年DeNA新卒入社 • アプリゲーム開発・運用（2014/08 〜 2018/10）
◦ サーバー、クライアント、マスター管理ツール、インフラ整備、マネジメントなど • テスト技術チーム - SWET（2018/10 〜） ◦ 仕様品質を向上させるための技術的なアプローチ研究 ▪ https://speakerdeck.com/hoddy3190/xing-shi-shou-fa-nituitediao-betemita ◦ Androidのテスト教育のための活動

目次 • TODOリストに起こそう • テストを書こう • 演習 • テストを書こう（つづき） •
構造化 • まとめ • 演習

本資料で使うツール、言語、ライブラリ • Android Studio 3.3.2 • kotlin 1.3.21 • JUnit5
1.4.0 ◦ https://junit.org/junit5/docs/current/user-guide/

TODOリストに起こそう

お題 • 1から100の数が入力されたら、その数の文字列を出力するプログラムを書け。 • ただし3の倍数のときは数の代わりに「Fizz」、 5の倍数のときは「Buzz」を出力しなければならない。 • 3と5両方の倍数の場合には「FizzBuzz」と出力しなければならない。

文脈 • フォームに数字を入力し、ボタンを押すと、 FizzBuzzの結果を表示するプログラムを書いている • ボタンを押すと、フォームに入力したテキストをそのまま結果欄に表示させるところまではできた • これからFizzBuzz変換ロジックを書こうとしている結果欄
ボタンフォーム

class MainActivity : AppCompatActivity() { override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState) setContentView(R.layout.activity_main) fizzbuzz_button.setOnClickListener { setResultText(convert(fizzbuzz_form.text.toString())) } } private fun convert(input: String): String { // fizzbuzz変換 return input // 仮 } private fun setResultText(str: String) { result_text.text = str } } 文脈（MainActivityの現在の中身）

まずは細かなタスクに分解コツ: 先に機能（仕様）の概要を定義し、そこに枝葉をつけていくドリルダウンの思考コツ

• [ ] 数を文字列にして返す • [ ] 3の倍数のときは数のかわりにFizzと返す • [
] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す

] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す概要

] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す枝葉

] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返すこの順番で実装

] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返すこの順番で実装正常系準正常系準正常系準正常系

テストを書こう

] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す対象

テストファイルを作るクラスに対応するテストファイルを作ってくれるまだMainActivity.ktしか存在しないので、MainActivity上でたたく Cmd + Shift + T

FizzBuzzTestにするチェックをつける

テストができる internal class FizzBuzzTest { @Test fun onCreate() { }
}

落ちるテストを書く internal class FizzBuzzTest { @Test fun onCreate() { assertEquals(1,
2) } } 落ちるテストを記述

2) } } assertEqualsと入力すると候補がたくさん出るが一番上のものを選んでくれればOK（なんでも良い）

2) } } usage: assertEquals(expected, actual) expectedが先に来ることに注意

テスト個別実行あるテストを個別に実行したい場合、カーソルをそのテスト内に移動させ、上のコマンドをたたく internal class FizzBuzzTest { @Test fun onCreate()
{ assertEquals(1, 2) } } Ctrl + Shift + R このあたりにカーソルを持ってきてコマンドをたたく

もしが効かない場合 Keymapがコンフリクトしている可能性。 [ Preferences ] -> [ Keymap ]
でKeymapを変更しよう。例えば、Ctrl + Shift + Z がよい。 Ctrl + Shift + R 検索ワード

しっかり落ちることを確認意図通りに落ちるということは、問題なくテストが動いていることを意味する org.openTestj.AssertionFailedError: Expected :1 Actual :2

テストを書いていこう internal class FizzBuzzTest { @Test fun onCreate() { assertEquals(1,
2) } } アサーションを削除

テストは動く仕様書 internal class FizzBuzzTest { @Test fun 数を文字列にして返す () {
} } 日本語で書くのもあり ※ただし、instrumented testでは使えない

テストは動く仕様書 internal class FizzBuzzTest { @Test @DisplayName("数を文字列にして返す ") fun intToStringTest()
{ } } DisplayNameでもOK

テストコードをどう書けばよいのかわからない => タスクに具体性が足りていない

• [ ] 数を文字列にして返す ◦ 1を渡したら文字列"1"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す
• [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す NEW

テストケース修正 internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { }
}

テストケース修正 internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { }
} 関数名の最初の文字に数字を使えない

テストケース修正 internal class FizzBuzzTest { @Test fun `1を渡したら文字列1を返す`() { }
} バッククオートで囲むのもよい ※ただ、クラス名で適用したときにテスト結果の表示が変になった

ここからどう書いていくか internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { }
}

3A internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 // Act 実行 // Assert 検証 } } 前準備、実行、検証

アサートファースト • 一番具体的な例からどういう実装をしなければならないかを逆算して考える • 使う人目線でどんなオブジェクトにどんな関数が生えているとよいのかを考える internal class FizzBuzzTest { @Test
fun _1を渡したら文字列1を返す() { // Arrange 前準備 // Act 実行 // Assert 検証 } } 検証から書こう

fun _1を渡したら文字列1を返す() { // Arrange 前準備 // Act 実行 // Assert 検証 assertEquals(“1”, ) } } 返り値として文字列を返そう

fun _1を渡したら文字列1を返す() { // Arrange 前準備 // Act 実行 // Assert 検証 assertEquals(“1”, fizzbuzz) } } ﬁzzbuzzというオブジェクトを用意して

fun _1を渡したら文字列1を返す() { // Arrange 前準備 // Act 実行 // Assert 検証 assertEquals(“1”, fizzbuzz.convert) } } 数字を文字列に変換する関数が生えていると使いやすそうだ

fun _1を渡したら文字列1を返す() { // Arrange 前準備 // Act 実行 // Assert 検証 assertEquals(“1”, fizzbuzz.convert(1)) } } 引数として数字を渡すのはどうだろう

fun _1を渡したら文字列1を返す() { // Arrange 前準備 // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } }

fun _1を渡したら文字列1を返す() { // Arrange 前準備 // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } コンパイルエラー

ローカル変数作成エラーが出ている文字の上でたたくと、修正案を表示してくれる。今回はローカル変数を作る。 internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す()
{ // Arrange 前準備 // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Opt + Enter

{ // Arrange 前準備 // Act 実行 // Assert 検証 val fizzbuzz assertEquals("1", fizzbuzz.convert(1)) } } Opt + Enter 追加される

{ // Arrange 前準備 val fizzbuzz // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Opt + Enter 移動

コンパイルエラー直し internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } 初期化処理追加

コンパイルエラー直し internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } まだエラー

クラス作成エラーが出ている文字の上でたたくと、修正案を表示してくれる。今回はクラスを作る。 Ctrl + Shift + R internal class
FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { // Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Opt + Enter

クラス作成エラーが出ている文字の上でたたくと、修正案を表示してくれる。今回はクラスを作る。 Ctrl + Shift + R internal class
FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { // Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Opt + Enter そのままOK

プロダクトコードに空のクラスができる class FizzBuzz { }

前回見ていた場所に戻る internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Cmd + [

行き来が面倒であればsplit windowを使おう

新たなコンパイルエラー internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } 新たなコンパイルエラー

また internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Opt + Enter

メンバ関数ができる class FizzBuzz { fun convert(i: Int): Any? { }
}

メンバ関数ができる class FizzBuzz { fun convert(i: Int): String { return
"" } }

メンバ関数ができる class FizzBuzz { fun convert(i: Int): String { return
"" } } 仮引数はIntにして、返り値はStringにするとりあえず空文字を返しておく

ようやくコンパイルエラーがなくなる internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } }

考えたこと • メソッド名 • クラス名 • 引数 • 返り値一番最初サイクルで書く時は結構考えること多い
なるべく小さめの機能を選ぶのがコツ

テスト実行 internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Ctrl + Shift + R

落ちる org.openTestj.AssertionFailedError: Expected :1 Actual : RED

落ちる org.openTestj.AssertionFailedError: Expected :1 Actual : RED でもちゃんとテストがコンパイルされた

テストが通るように修正 class FizzBuzz { fun convert(i: Int): String { return
"1" } } GREEN

"1" } } テストを通すための最小限の修正 GREEN

仮実装いきなり本格的なコードから書き始めると、テストが失敗したときに、原因がテストコードにあるのかプロダクトコードにあるのかがわかりづらくなるので、まず簡易的な実装をしてテストを通過させる

テスト駆動開発においてテストコードにバグがないことをどうやって保証するか

テストコードのテストコードを書く？

きりがないテストコードのテストコードを書き、テストコードのテストコードのテストコードを書き、テストコードのテストコードのテストコードのテストコードを書き...

テストコードのテストは実装側で行う

テスト実行 internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } } Ctrl + Shift + R

通った GREEN

• [ ] 数を文字列にして返す ◦ [ ] 1を渡したら文字列"1"を返す • [
] 3の倍数のときは数のかわりにFizzと返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す

三角測量テストが偶然パスしただけなのではという不安を払拭するために、 2つ以上の入力を使用してテストを書き、そのテストが予想通りに落ちるかを確かめる

別の入力を使いながら、既存のテストのパスは保ちつつ、プロダクトコードをリファクタしていく入出力は明らかなのに内部ロジックが容易にイメージできない実装をする場合にも有用なテクニック

• [ ] 数を文字列にして返す ◦ [ ] 1を渡したら文字列"1"を返す ◦ [
] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す NEW

入力が2のときのテストを書こう internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { //
Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) assertEquals("2", fizzbuzz.convert(2)) } }

アサーションルーレットアンチパターン 1つのテストに複数のアサーション internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() {
// Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) assertEquals("2", fizzbuzz.convert(2)) } } JUnitの場合、あるアサーションでエラーが出てしまうと、以降のアサーションが実行されず、 TDDサイクルを正常に回すことができなくなる

原則 1 assertion per 1 test

internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { // Arrange
前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } @Test fun _2を渡したら文字列2を返す() { // Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("2", fizzbuzz.convert(2)) } }

internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { // Arrange
前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("1", fizzbuzz.convert(1)) } @Test fun _2を渡したら文字列2を返す() { // Arrange 前準備 val fizzbuzz = FizzBuzz() // Act 実行 // Assert 検証 assertEquals("2", fizzbuzz.convert(2)) } } テストを分割して書く

テストを回すと落ちる org.openTestj.AssertionFailedError: Expected :2 Actual :1 RED

i.toString() } } GREEN

テストは個別実行ではなく全体実行プロダクトコードの書き換えにより、既存のテストの結果が変わることもあるためクリック

通った GREEN

テストコード internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { val
fizzbuzz = FizzBuzz() assertEquals("1", fizzbuzz.convert(1)) } @Test fun _2を渡したら文字列2を返す() { val fizzbuzz = FizzBuzz() assertEquals("2", fizzbuzz.convert(2)) } } Refactoring

internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { val fizzbuzz
= FizzBuzz() assertEquals("1", fizzbuzz.convert(1)) } @Test fun _2を渡したら文字列2を返す() { val fizzbuzz = FizzBuzz() assertEquals("2", fizzbuzz.convert(2)) } } 3Aコメントの削除テストコード Refactoring 3Aコメントの削除

前回と同じテストを実行 Ctrl + R 前回実行したテスト

通った Refactoring

@Test fun _1を渡したら文字列1を返す() { val fizzbuzz = FizzBuzz() assertEquals("1", fizzbuzz.convert(1))
} @Test fun _2を渡したら文字列2を返す() { val fizzbuzz = FizzBuzz() assertEquals("2", fizzbuzz.convert(2)) } テストコード Refactoring

} @Test fun _2を渡したら文字列2を返す() { val fizzbuzz = FizzBuzz() assertEquals("2", fizzbuzz.convert(2)) } テストコード Refactoring 共通化できるところを共通化する

テスト用関数（setUp関数）作成テストで使うコードのテンプレートから関数を作れる。カーソルをそのテスト内に移動させ、上のコマンドをたたく。 internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す()
{ ... } Cmd + N このあたりにカーソルを持ってきてコマンドをたたく

テスト用関数（setUp関数）作成テストで使うコードのテンプレートから関数を作れるカーソルをそのテスト内に移動させ、上のコマンドをたたく internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す()
{ ... } Cmd + N

internal class FizzBuzzTest { @BeforeEach internal fun setUp() { TODO("not
implemented") // To change body of created functions use File | Settings | File Templates. } @Test fun _1を渡したら文字列1を返す() { val fizzbuzz = FizzBuzz() assertEquals("1", fizzbuzz.convert(1)) } ... setUp関数作成 Refactoring

internal class FizzBuzzTest { private lateinit var fizzbuzz: FizzBuzz @BeforeEach
internal fun setUp() { fizzbuzz = FizzBuzz() } @Test fun _1を渡したら文字列1を返す() { val fizzbuzz = FizzBuzz() assertEquals("1", fizzbuzz.convert(1)) } ... 共通処理をsetUp関数に集約 Refactoring 追加追加

} @Test fun _2を渡したら文字列2を返す() { val fizzbuzz = FizzBuzz() assertEquals("2", fizzbuzz.convert(2)) } 少しずつ直してテスト実行 Refactoring まずはここだけ削除

@Test fun _1を渡したら文字列1を返す() { assertEquals("1", fizzbuzz.convert(1)) } @Test fun _2を渡したら文字列2を返す()
{ val fizzbuzz = FizzBuzz() assertEquals("2", fizzbuzz.convert(2)) } 少しずつ直してテスト実行 Refactoring 次はここを削除

補足共通化は、同じ処理が2箇所に現れたら行う派と、同じ処理が3箇所に現れたら行う派がいる。今回は、FizzBuzzの簡易性や説明の流れの都合を加味してこのタイミングで共通化を行った。

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す

演習

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す対象仮実装・三角測量を使いながらやってみよう

class FizzBuzz { fun convert(i: Int): String { return i.toString()
} } internal class FizzBuzzTest { private lateinit var fizzbuzz: FizzBuzz @BeforeEach internal fun setUp() { fizzbuzz = FizzBuzz() } @Test fun _1を渡したら文字列 1を返す() { assertEquals("1", fizzbuzz.convert(1)) } @Test fun _2を渡したら文字列 2を返す() { assertEquals("2", fizzbuzz.convert(2)) } } 今段階のテストコード今段階のプロダクトコード

便利なショートカット一覧（keymap種別: Mac OS X 10.5+） Ctrl + Shift + R
Cmd + N Opt + Enter Cmd + Shift + T Ctrl + R コードを生成（テスト関数などを作れる）テストファイルを作るテストファイルとプロダクトファイルの行き来をするクイック修正テストの個別実行最後に実行したテストの再実行前見ていた場所に戻る Generate... Test Run ‘hogeTest‘ Run ‘hogeTest‘ コマンドアクション名説明 Show Intention Actions Back Cmd + [

解答例

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す ◦ [ ] 3を渡したら文字列"Fizz"を返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す NEW

internal class FizzBuzzTest { @Test fun _1を渡したら文字列1を返す() { // 省略
} @Test fun _2を渡したら文字列2を返す() { // 省略 } @Test fun _3を渡したら文字列Fizzを返す() { assertEquals("Fizz", fizzbuzz.convert(3)) } } 追加

落ちる（個別実行） RED

テストを通すための最低限の修正（仮実装） class FizzBuzz { fun convert(i: Int): String { if
(i == 3) return "Fizz" return i.toString() } }

通った GREEN

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す ◦ [ ] 3を渡したら文字列"Fizz"を返す ◦ [ ] 6を渡したら文字列"Fizz"を返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す NEW

三角測量 // 省略 @Test fun _3を渡したら文字列Fizzを返す() { assertEquals("Fizz", fizzbuzz.convert(3)) }
@Test fun _6を渡したら文字列Fizzを返す() { assertEquals("Fizz", fizzbuzz.convert(6)) } // 省略追加

落ちる RED

テストが通るように修正 class FizzBuzz { fun convert(i: Int): String { if
(i % 3 == 0) return "Fizz" return i.toString() } } GREEN

通った GREEN

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す ◦ [ ] 3を渡したら文字列"Fizz"を返す ◦ [ ] 6を渡したら文字列"Fizz"を返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す

つづき

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す ◦ [ ] 3を渡したら文字列"Fizz"を返す ◦ [ ] 6を渡したら文字列"Fizz"を返す • [ ] 5の倍数のときはBuzzと返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す対象

... @Test fun _6を渡したら文字列Fizzを返す() { assertEquals("Fizz", fizzbuzz.convert(6)) } @Test fun
_5を渡したら文字列Buzzを返す() { assertEquals("Buzz", fizzbuzz.convert(5)) } ... テストケース追加追加

落ちる RED

(i % 3 == 0) return "Fizz" if (i % 5 == 0) return "Buzz" return i.toString() } }

(i % 3 == 0) return "Fizz" if (i % 5 == 0) return "Buzz" return i.toString() } } あえてif (i == 5) とは書かなかった

明白な実装テストの書き方や実装の仕方に不安がないときはテストを書いて、見えている実装をそのまま書く

通った GREEN

構造化

FizzBuzzを一切知らない人がこのテストコードを見たら、 FizzBuzzの挙動がわかるようになっているか @Test fun _1を渡したら文字列1を返す() { assertEquals("1", fizzbuzz.convert(1)) } @Test
fun _2を渡したら文字列2を返す() { assertEquals("2", fizzbuzz.convert(2)) } @Test fun _3を渡したら文字列Fizzを返す() { assertEquals("Fizz", fizzbuzz.convert(3)) } @Test fun _6を渡したら文字列Fizzを返す() { assertEquals("Fizz", fizzbuzz.convert(6)) } @Test fun _5を渡したら文字列Buzzを返す() { assertEquals("Buzz", fizzbuzz.convert(5)) }

テストは動く仕様書日本語で仕様書を書くときに、箇条書きといった見せ方の工夫をするようにテストでも同じことをやる

テストを構造化 @Nested inner class _3の倍数の場合 { } @Nested inner class
_5の倍数の場合 { } @Nested inner class その他の場合 { }

@Nested inner class _3の倍数の場合 { @Test fun _3を渡したら文字列Fizzを返す() { assertEquals("Fizz",
fizzbuzz.convert(3)) } @Test fun _6を渡したら文字列Fizzを返す() { assertEquals("Fizz", fizzbuzz.convert(6)) } } @Nested inner class _5の倍数の場合 { @Test fun _5を渡したら文字列Buzzを返す() { assertEquals("Buzz", fizzbuzz.convert(5)) } } @Nested inner class その他の場合 { @Test fun _1を渡したら文字列1を返す() { assertEquals("1", fizzbuzz.convert(1)) } @Test fun _2を渡したら文字列2を返す() { assertEquals("2", fizzbuzz.convert(2)) } }

すっきり

テストケースの整理対称性の乱れは整理のきっかけ。 ※対称性がとれていないからといってそれが悪いわけではない。ただテストコードを読む際に気にする人もいたりする。もうひとつ 2つ 1つ 2つ

テストケース数 • 1つに統一するか • 2つに統一するか 2つ 1つ 2つ

どちらがよいのか正解はないし、統一しないといけないわけでもない

不安をトレースしたテストケースになっているか「不安かどうか」というといささか抽象的に感じられるかもしれませんが、 TDDではそういった感情をとても大切にしています。

不安がない場合不安が残らないように入力を1つに統一する。プロダクトコードと同じく、後でテストコードを減らすのはものすごい大変なので、ここで消しておくメリットもある。

不安がある場合別のテストケースを加える。入力を2つに統一するようにテストケース数を調整してもよいし、対称性にこだわらず、さらにテストケースを追加しても良い。不安が残らないようにするのが大事。

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す ◦ [ ] 3を渡したら文字列"Fizz"を返す ◦ [ ] 6を渡したら文字列"Fizz"を返す • [ ] 5の倍数のときはBuzzと返す ◦ [ ] 5を渡したら文字列"Buzz"を返す ◦ [ ] 10を渡したら文字列"Buzz"を返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す

まとめ

まとめ • 問題を小さく分割する ◦ TODOリスト形式など • 歩幅を適切に選ぶ ◦ 不安だったらテスト
-> 仮実装 -> 三角測量 -> 実装 ◦ やや不安だったらテスト -> 仮実装 -> 実装 ◦ 不安がなければテスト -> 明白な実装 • テストコードの読み手のことも考えてテストの構造化とリファクタリングも忘れずに

リファレンス • 50分でわかるテスト駆動開発 ◦ https://channel9.msdn.com/Events/de-code/2017/DO03?ocid=player ◦ 本資料は上の和田卓人（t_wada）さんの動画を参考にしています ◦ 動画内でt_wadaさんがFizzBuzzの実装のTDDライブコーディングを行っていますので、もしご興味あれば是非見てみてください
◦ CC BY 4.0に基づいて使用しています • テスト駆動開発 ◦ Kent Beck 著 / 和田卓人訳 ◦ オーム社

演習

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す ◦ [ ] 3を渡したら文字列"Fizz"を返す • [ ] 5の倍数のときはBuzzと返す ◦ [ ] 5を渡したら文字列"Buzz"を返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す対象

解答例

] 2を渡したら文字列"2"を返す • [ ] 3の倍数のときは数のかわりにFizzと返す ◦ [ ] 3を渡したら文字列"Fizz"を返す ◦ [ ] 6を渡したら文字列"Fizz"を返す • [ ] 5の倍数のときはBuzzと返す ◦ [ ] 5を渡したら文字列"Buzz"を返す ◦ [ ] 10を渡したら文字列"Buzz"を返す • [ ] 3と5両方の倍数の場合にはFizzBuzzと返す ◦ [ ] 15を渡したら文字列"FizzBuzz"を返す ◦ [ ] 30を渡したら文字列"FizzBuzz"を返す

@BeforeEach fun setup() { fizzbuzz = FizzBuzzComp() } @Nested inner
class _3の倍数かつ5の倍数の場合 { @Test fun _15を渡したら文字列FizzBuzzを返す() { assertEquals("FizzBuzz", fizzbuzz.convert(15)) } } @Nested inner class _3の倍数の場合 { // 省略 }

落ちる RED

(num % 15 == 0) return "FizzBuzz" if (num % 3 == 0) return "Fizz" if (num % 5 == 0) return "Buzz" return i.toString() } }

通った GREEN

@Nested inner class _3の倍数かつ5の倍数の場合 { @Test fun _15を渡したら文字列FizzBuzzを返す() { assertEquals("FizzBuzz",
fizzbuzz.convert(15)) } @Test fun _30を渡したら文字列FizzBuzzを返す() { assertEquals("FizzBuzz", fizzbuzz.convert(30)) } } @Nested inner class _3の倍数の場合 { // 省略 } 不安だったので追加

通った GREEN

@Nested inner class _3の倍数または5の倍数の場合 { @Nested inner class _3の倍数または5の倍数の場合 {
@Test fun _15を渡したら文字列 FizzBuzzを返す() { assertEquals("FizzBuzz", fizzbuzz.convert(15)) } @Test fun _30を渡したら文字列 FizzBuzzを返す() { assertEquals("FizzBuzz", fizzbuzz.convert(30)) } } @Nested inner class _3の倍数の場合 { //　省略 } @Nested inner class _5の倍数の場合 { //　省略 } } @Nested inner class その他の場合 { //　省略 } Refactoring

@Nested inner class _3の倍数または5の倍数の場合 { @Nested inner class _3の倍数または5の倍数の場合 {
@Test fun _15を渡したら文字列 FizzBuzzを返す() { assertEquals("FizzBuzz", fizzbuzz.convert(15)) } @Test fun _30を渡したら文字列 FizzBuzzを返す() { assertEquals("FizzBuzz", fizzbuzz.convert(30)) } } @Nested inner class _3の倍数の場合 { //　省略 } @Nested inner class _5の倍数の場合 { //　省略 } } @Nested inner class その他の場合 { //　省略 } Refactoring 構造化

演習

TDDBC(TDD Boot Camp) http://devtesting.jp/tddbc/ TDDについて、実習形式で手を動かして体得することを目的とするイベント。定期的に開催されている。イベントで使った「お題」についても公開されているので興味があればどうぞ！

TDDチュートリアル

TDDチュートリアル

More Decks by Hodaka Suzuki

Other Decks in Programming

Featured

Transcript