xaicron/p5-Test-Flatten — pull request #3 by hixi-hyi (2014-11-17 09:07:11+00:00)
Title: change override method, Test::More to Test::Builder

@xaicron さん

ぶっちゃけどっちでもいい！と言われそうですが、
Test系のモジュールを作るときはTest::Builderをオーバーライドした方がいいかな。
と思っております。

Test系のモジュールを作るときは、Test::Builderをオーバライドしているケースが多いのでは？っていう理由だけです。
* https://metacpan.org/source/TOKUHIROM/Test-Pretty-0.30/lib/Test/Pretty.pm#L52
* https://metacpan.org/source/SATOH/Test-Name-FromLine-0.13/lib/Test/Name/FromLine.pm#L18

で、僕もテスト用のモジュールを作った時に、Test::Moreではなく、Test::Builderの方のsubtestを使うように書いちゃったので、
https://metacpan.org/source/HIXI/Test-Clear-0.02/lib/Test/Clear.pm#L32
あー、Test::Flattenのsubtest使えないなーと思って、PR送ってしまいました。

利点は、上のようなテスト用モジュールと共有できるようになる。
欠点は、Test::Moreのsubtestをオーバライドするようなモジュールを作られた時に撃ち負ける。

です。お考えを聞かせていただくm m