Bug #3177

TestFileUtils#test_cp_r_symlink

Added by shyouhei (Shyouhei Urabe) about 2 years ago. Updated about 1 year ago.

[ruby-dev:41018]
Status:Closed Start date:04/20/2010
Priority:Normal Due date:
Assignee:yugui (Yuki Sonoda) % Done:

0%
Category:lib
Target version:1.9.2
ruby -v:ruby 1.9.2dev (2010-04-20 trunk 27400) [x86_64-linux]

Description

お気づきのことと思いますが、私のコミット以来trunkで1F増えています。
http://www.rubyist.net/~akr/chkbuild/debian/ruby-trunk/log/20100420T015800.diff.txt.gz

この原因は、FileUtils#cp_rの挙動を1.8と揃えたことにあります。
http://archive.good-day.net/~w3ml/w3ml.cgi/ruby-changes/msg/15498

なぜ1.8のほうに揃えたかというと、1.8のほうの挙動がそもそも意図されたものであるかのように見えるからです。
http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-dev/27149

したがってtrunkのこのテストは不適切ではないでしょうか。いきなりテストごと変えるのはよくないかなと思ったので、一旦チケットにします。

History

#1
Updated by mame (Yusuke Endoh) about 2 years ago

遠藤です。

2010年4月20日8:16 Shyouhei Urabe <redmine@ruby-lang.org>:
> お気づきのことと思いますが、私のコミット以来trunkで1F増えています。
> http://www.rubyist.net/~akr/chkbuild/debian/ruby-trunk/log/20100420T015800.diff.txt.gz
>
> この原因は、FileUtils#cp_rの挙動を1.8と揃えたことにあります。
> http://archive.good-day.net/~w3ml/w3ml.cgi/ruby-changes/msg/15498
>
> なぜ1.8のほうに揃えたかというと、1.8のほうの挙動がそもそも意図されたものであるかのように見えるからです。
> http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-dev/27149
>
> したがってtrunkのこのテストは不適切ではないでしょうか。いきなりテストごと変えるのはよくないかなと思ったので、一旦チケットにします。


時系列から見て、テストの直し忘れなんでしょうね。
テストを修正するのに賛成します。

-- 
Yusuke Endoh <mame@tsg.ne.jp>

#2
Updated by mame (Yusuke Endoh) about 2 years ago

  • Status changed from Assigned to Closed
遠藤です。

成瀬さんが r27550 で直してくれましたので close します。

-- 
Yusuke Endoh <mame@tsg.ne.jp>

Also available in: Atom PDF