回答編集履歴
3
文章の修正
test
CHANGED
|
@@ -20,7 +20,7 @@
|
|
|
20
20
|
|
|
21
21
|
|
|
22
22
|
|
|
23
|
-
使う側は、そのモジュールで用意されているpublicなメソッドを呼び、そのprivateメソッドを
|
|
23
|
+
使う側は、そのモジュールで用意されているpublicなメソッドを呼び、そのメソッド経由で呼ばれるprivateメソッドを呼び、最終的にpublicメソッドから意図した戻り値が返ってくればテストとして十分であるためです。
|
|
24
24
|
|
|
25
25
|
|
|
26
26
|
|
2
変な文章な気がしたので直した
test
CHANGED
|
@@ -8,11 +8,7 @@
|
|
|
8
8
|
|
|
9
9
|
|
|
10
10
|
|
|
11
|
-
ただ、それを試すのは
|
|
11
|
+
ただ、それを試すのは、**もうリリース済みで手を簡単には入れられないコードをテストしたいケース** に陥ったときの最終手段だと考えてください。
|
|
12
|
-
|
|
13
|
-
- **もうリリース済みで簡単に手を入れられないコード**
|
|
14
|
-
|
|
15
|
-
- **テストコードを書かずに作られたレガシーなコード**
|
|
16
12
|
|
|
17
13
|
|
|
18
14
|
|
|
@@ -20,14 +16,14 @@
|
|
|
20
16
|
|
|
21
17
|
|
|
22
18
|
|
|
23
|
-
privateメソッドに対するテストは上記のようなケースでしか
|
|
19
|
+
privateメソッドに対するテストは上記のようなケースでしか一般的には行いません。
|
|
24
20
|
|
|
25
21
|
|
|
26
22
|
|
|
27
|
-
使う側は、そのモジュールで用意されているpublicなメソッドを呼び、そのprivateメソッドを通して意図した戻り値が返ってくれば
|
|
23
|
+
使う側は、そのモジュールで用意されているpublicなメソッドを呼び、そのprivateメソッドを通して意図した戻り値が返ってくれば十分であるためです。
|
|
28
24
|
|
|
29
25
|
|
|
30
26
|
|
|
31
|
-
|
|
27
|
+
privateメソッドの責務が肥大している場合は、どうしてもprivateなメソッドをテストしたくなります。
|
|
32
28
|
|
|
33
|
-
その場合は、privateメソッドを
|
|
29
|
+
しかし、その場合は`PrivateObject`を使うのではなく、privateメソッドを複数のpublicメソッドに分割し、privateメソッドをテストしないよう見直す方が良いでしょう。
|
1
文章の修正
test
CHANGED
|
@@ -8,14 +8,26 @@
|
|
|
8
8
|
|
|
9
9
|
|
|
10
10
|
|
|
11
|
+
ただ、それを試すのは下記のようなケースに陥ったときの最終手段だと考えてください。
|
|
12
|
+
|
|
13
|
+
- **もうリリース済みで簡単に手を入れられないコード**
|
|
14
|
+
|
|
15
|
+
- **テストコードを書かずに作られたレガシーなコード**
|
|
16
|
+
|
|
17
|
+
|
|
18
|
+
|
|
11
19
|
### そもそも論
|
|
12
20
|
|
|
13
21
|
|
|
14
22
|
|
|
15
|
-
privateメソッドに対するテストは
|
|
23
|
+
privateメソッドに対するテストは上記のようなケースでしか普通は行いません。
|
|
16
|
-
|
|
17
|
-
使う側は、そのモジュールで用意されているpublicなメソッドを呼び、意図した戻り値が返ってくればそれで十分であるためです。
|
|
18
24
|
|
|
19
25
|
|
|
20
26
|
|
|
27
|
+
使う側は、そのモジュールで用意されているpublicなメソッドを呼び、そのprivateメソッドを通して意図した戻り値が返ってくればそれで十分であるためです。
|
|
28
|
+
|
|
29
|
+
|
|
30
|
+
|
|
31
|
+
それでもprivateなメソッドをテストしたいのであれば、恐らくそのメソッドの責務が肥大しているだと推測します。
|
|
32
|
+
|
|
21
|
-
|
|
33
|
+
その場合は、privateメソッドを各責務毎に分割し、その内のテストしたいメソッドをpublicにすることでprivateメソッドをテストしない、つまり`PrivateObject`を使わずにテストができるよう実装を見直すことが重要と思います。
|