タイムラインに流していた話題と似ていますが、
このテストの位置付けによりますかね。。

テストがあくまで「開発者のコーディング手助け」程度ならば、
テストが粗くても問題ないですし(語弊あるかもですが)、
これを「品質保証のための単体テスト」と捉えるならばきちんと検証すべきですね。

あとはこれは自分でもまだ悩みどころなんですが、
動的言語のテストにおいてどこまで厳密にやるべきなんでしょうね。
型を意識せずコーディングできるのが動的言語の利点ですが、
テストで型を意識すると結局本体もそれを考慮した作りになって、
動的言語の良さが失われるんですよね。(むしろ不利)

とはいえ、自分がレビューする側だと気になるんですけどｗ

どのみちこのあたりも一回修正します。