* lib/pathname.rb (Pathname#descend): Pathname.new("./a/b/c").descend

didn't yield "."
  (Pathname#ascend): ditto.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@9073 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

ChangeLog

@@ -1,3 +1,9 @@
+Sat Sep  3 13:59:31 2005  Tanaka Akira  <akr@m17n.org>
+
+	* lib/pathname.rb (Pathname#descend): Pathname.new("./a/b/c").descend
+	  didn't yield "."
+	  (Pathname#ascend): ditto.
+
 Fri Sep  2 23:51:54 2005  Nobuyoshi Nakada  <nobu@ruby-lang.org>
 
 	* parse.y (f_arg): f_norm_arg is a VALUE in ripper, not an ID.

lib/pathname.rb

@@ -448,10 +448,11 @@ class Pathname
       end
       paths << v
     else
-      until v.to_s == '.'
+      until v.basename == v
         paths << v
         v = v.dirname
       end
+      paths << v
     end
     paths.reverse_each {|path| yield path }
   end
@@ -482,10 +483,11 @@ class Pathname
       end
       paths << v
     else
-      until v.to_s == '.'
+      until v.basename == v
         paths << v
         v = v.dirname
       end
+      paths << v
     end
     paths.each {|path| yield path }
   end
@@ -1273,31 +1275,40 @@ if $0 == __FILE__
     end
 
     def test_descend_abs
-      rs = %w[/ /a /a/b /a/b/c]
+      rs = %w[/ /a /a/b /a/b/c].map {|s| Pathname.new(s) }
-      Pathname.new("/a/b/c").descend {|v|
+      Pathname.new("/a/b/c").descend {|v| assert_equal(rs.shift, v) }
-        assert_equal(Pathname.new(rs.shift), v)
+      assert_equal([], rs)
-      }
     end
 
     def test_descend_rel
-      rs = %w[a a/b a/b/c]
+      rs = %w[a a/b a/b/c].map {|s| Pathname.new(s) }
-      Pathname.new("a/b/c").descend {|v|
+      Pathname.new("a/b/c").descend {|v| assert_equal(rs.shift, v) }
-        assert_equal(Pathname.new(rs.shift), v)
+      assert_equal([], rs)
-      }
+    end
+
+    def test_descend_rel_with_current_dir
+      rs = %w[. ./a ./a/b ./a/b/c].map {|s| Pathname.new(s) }
+      Pathname.new("./a/b/c").descend {|v| assert_equal(rs.shift, v) }
+      assert_equal([], rs)
     end
 
     def test_ascend_abs
-      rs = %w[/a/b/c /a/b /a /]
+      rs = %w[/a/b/c /a/b /a /].map {|s| Pathname.new(s) }
-      Pathname.new("/a/b/c").ascend {|v|
+      Pathname.new("/a/b/c").ascend {|v| assert_equal(rs.shift, v) }
-        assert_equal(Pathname.new(rs.shift), v)
+      assert_equal([], rs)
-      }
     end
 
     def test_ascend_rel
-      rs = %w[a/b/c a/b a]
+      rs = %w[a/b/c a/b a].map {|s| Pathname.new(s) }
-      Pathname.new("a/b/c").ascend {|v|
+      Pathname.new("a/b/c").ascend {|v| assert_equal(rs.shift, v) }
-        assert_equal(Pathname.new(rs.shift), v)
+      assert_equal([], rs)
-      }
     end
+
+    def test_ascend_rel_with_current_dir
+      rs = %w[./a/b/c ./a/b ./a .].map {|s| Pathname.new(s) }
+      Pathname.new("./a/b/c").ascend {|v| assert_equal(rs.shift, v) }
+      assert_equal([], rs)
+    end
+
   end
 end