[ruby-dev:39586] Re: [ruby-cvs:32789] Ruby:r25571 (trunk): * test/ruby/test_enum.rb (test_count): test for [ruby-core:24794].

View: New views
2 Messages — Rating Filter:   Alert me  

Parent Message unknown [ruby-dev:39586] Re: [ruby-cvs:32789] Ruby:r25571 (trunk): * test/ruby/test_enum.rb (test_count): test for [ruby-core:24794].

by Yukihiro Matsumoto :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

まつもと ゆきひろです

In message "Re: [ruby-cvs:32789] Ruby:r25571 (trunk): * test/ruby/test_enum.rb (test_count): test for [ruby-core:24794]."
    on Fri, 30 Oct 2009 16:28:03 +0900 (JST), nobu@... writes:

|    * test/ruby/test_enum.rb (test_count): test for [ruby-core:24794].

[ruby-core:26420]にも書いたように、今回の対応は処理系依存では
ないかと考えています。つまり、処理系によってはこのテストは成
功しないで無限再帰になるかもしれないと。今回の修正を確認した
いと言う意図はわかるんですが、こういうのをテストに入れてもい
いんでしょうかね。


[ruby-dev:39587] Re: [ruby-cvs:32789] Ruby:r25571 (trunk): * test/ruby/test_enum.rb (test_count): test for [ruby-core:24794].

by NARUSE, Yui-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

2009/10/30 16:40, Yukihiro Matsumoto wrote:

> まつもと ゆきひろです
>
> In message "Re: [ruby-cvs:32789] Ruby:r25571 (trunk): * test/ruby/test_enum.rb (test_count): test for [ruby-core:24794]."
>      on Fri, 30 Oct 2009 16:28:03 +0900 (JST), nobu@... writes:
>
> |    * test/ruby/test_enum.rb (test_count): test for [ruby-core:24794].
>
> [ruby-core:26420]にも書いたように、今回の対応は処理系依存では
> ないかと考えています。つまり、処理系によってはこのテストは成
> 功しないで無限再帰になるかもしれないと。今回の修正を確認した
> いと言う意図はわかるんですが、こういうのをテストに入れてもい
> いんでしょうかね。

Ruby のリポジトリ内の test/ 下にあるテストは、
CRuby というか MRI というかのテストであるべきだと思います。
処理系非依存なテストは RubySpec に任せるべきでしょう。

今回の対応は CRuby においては意図を持って変更したわけで、
それが将来意図せず変わっていないかテストで確認することは
意義のあることだと思います。

rdoc や当該テストに処理系依存である旨書いておいて、
意図せず RubySpec に取り込まれないように、
注意する必要はあると思いますが。

--
NARUSE, Yui  <naruse@...>