Bug #1512: minitest のレビュー

r3497, r3498, r3499 で一部をレビューして #@todo を削除した。
#@todo を残している部分はよくわからなかったので残している。

MiniTest::Assertions#assert_in_delta に「期待値と実際の値の差の絶対値が与えられた差分以下である場合、検査にパスしたことになります。」とありますが，「与えられた差分」は変ではないでしょうか。同じく delta の説明の「許容する差分」もおかしいと思います。

MiniTest::Assertions#assert_in_epsilon に
 与えられた期待値と実際の値の計算機誤差が許容範囲内である場合、検査にパスしたことになります。
とあるのですが，計算機誤差はあまり関係ないように思います。
平たく言えば，assert_in_delta が絶対誤差を見るのに対し，assert_in_epsilon は相対誤差を見るということではないでしょうか。
Ruby 1.9.1-p378 のソースを見てみると，
 def assert_in_epsilon a, b, epsilon = 0.001, msg = nil
   assert_in_delta a, b, [a, b].min * epsilon, msg
 end
となっていますが，a, b が負数の場合はこのコードじゃダメそうな気が…。

library minitest/spec の「要約」に，
 このライブラリは Object に BDD 用の検査メソッドが追加します。 
とありますが，「検査メソッドが」は「検査メソッドを」が適当でしょう。
ところで，「BDD」の初出箇所を「BDD（behavior driven development，振舞い駆動開発）」などとしては如何でしょうか。

些細なタイポです。

Index: refm/api/src/minitest/MiniTest__Assertions
===================================================================
--- refm/api/src/minitest/MiniTest__Assertions	(revision 4230)
+++ refm/api/src/minitest/MiniTest__Assertions	(working copy)
@@ -85,7 +85,7 @@
 #@# 使っている用語が正しくないかもしれない。
 与えられた期待値と実際の値の相対誤差が許容範囲内である場合、検査にパスしたことになります。

-言い換えるとと以下の式が真である場合、検査をパスします。
+言い換えると以下の式が真である場合、検査をパスします。

   [expected, actual].min * epsilon >= (extected - actual).abs

Index: refm/api/src/minitest/spec.rd
===================================================================
--- refm/api/src/minitest/spec.rd	(revision 4230)
+++ refm/api/src/minitest/spec.rd	(working copy)
@@ -184,7 +184,7 @@
 @see [[m:MiniTest::Assertions#refute_empty]]

 --- wont_equal(actual) -> true
-自身とと実際の値が等くない場合、検査にパスしたことになります。
+自身と実際の値が等くない場合、検査にパスしたことになります。

 @param actual 実際の値を指定します。

• Status changed from Open to Closed

レビューしました。 必要なことは書いてありそうなので close します。