pathname2 1.7.4 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 84212bcc430946a16d891b581e2542f0612a2fb8
-  data.tar.gz: 29523654aa88c6c8457a4e340aaf6fbcb793bee3
+  metadata.gz: 113b69a61947d4b98d3c4a44c1a030d2a49e193e
+  data.tar.gz: 03c39ef8fdfbf343c04ce7ab3da048af9f5da3db
 SHA512:
-  metadata.gz: f533ba4680d65321f23d14353f81d181fbaf9afb04008ea84c21f53c9262a0bb66a73750f349de344522824966845c0d6f3d1eb5e405cd8b7a77b6444323352e
-  data.tar.gz: 929e7cabf8112de8e7093323296ef4549354b652eef8ec53fef925529d4926f6cfa224ae58d91d06fe1dee1d783001559cbea6ac59a5c12601b819f0d746cf2e
+  metadata.gz: fec8d35e327db644ea24c8777288df0690c6e8e6d85627b7646aae83d99266bbeb0ef8c256727afd77342c89ed5cf36ca9923c444aa4c01edcb00a3d435cc98d
+  data.tar.gz: 80eadfb04099f368725d206f487ab3195515819053bb365173a90b7ee64a1c136e3a0cbb36bd18622174ae9d582ee125b84cf5f2b3918ecb58adadd5eb0ee72a

data/CHANGES

@@ -1,3 +1,10 @@
+== 1.8.0 - 19-Jun-2016
+* Changed license to Apache 2.0.
+* Some cleanup and updates to the Rakefile and benchmarks.
+* Refactored some realpath tests so they're not touching system files like
+  /dev/stdin any more. Thanks go to Michael R. Crusoe for pointing out the
+  potential pitfalls of doing that.
+
 == 1.7.4 - 7-Sep-2015
 * This gem is now signed.
 * Rakefile now assumes Rubygems 2.x for some tasks.

data/README

@@ -72,21 +72,20 @@
   method that is used by the pathname2 library.
    
 == Known Issues
-  In Ruby 1.8.3 and 1.8.4 you will see a failure in the test suite regarding
-  'fu_world_writable?' from FileUtils.  You can ignore this.  That method is
-  supposed to be private. See ruby-core:7383.
+  On MS Windows, some methods may not work on pathnames greater than 260
+  characters because of internal function limitations.
    
-  Any other issues should be reported on the project page at
+  Any issues you find should be reported on the project page at
   https://github.com/djberg96/pathname2
 
 == Future Plans
   Suggestions welcome.
    
 == License
-  Artistic 2.0
+  Apache 2.0
    
 == Copyright
-  (C) 2003-2015 Daniel J. Berger
+  (C) 2003-2016 Daniel J. Berger
   All rights reserved.
 
 == Warranty

data/Rakefile

@@ -10,7 +10,7 @@ namespace :gem do
     require 'rubygems/package'
     spec = eval(IO.read('pathname2.gemspec'))
     spec.signing_key = File.join(Dir.home, '.ssh', 'gem-private_key.pem')
-    Gem::Package.build(spec)
+    Gem::Package.build(spec, true)
   end
 
   desc "Install the pathname2 gem"

data/benchmarks/bench_pathname.rb

@@ -12,7 +12,7 @@ require 'benchmark'
 require 'pathname2'
 require 'rbconfig'
 
-if Config::CONFIG['host_os'].match("mswin")
+if RbConfig::CONFIG['host_os'] =~ /mingw|mswin/i
    path1 = Pathname.new("C:\\Program Files\\Windows NT")
    path2 = Pathname.new("Accessories")
    path3 = Pathname.new("C:\\Program Files\\..\\.\\Windows NT")
@@ -26,102 +26,102 @@ end
 MAX = 10000
 
 Benchmark.bm(25) do |bench|
-   bench.report("Pathname.new(path)"){
-      MAX.times{ Pathname.new("/usr/local/bin") }
-   }
-
-   bench.report("Pathname#+(Pathname)"){
-      MAX.times{ path1 + path2 }
-   }
-
-   bench.report("Pathname#+(String)"){
-      MAX.times{ path1 + path2 }
-   }
-
-   bench.report("Pathname#children"){
-      MAX.times{ path1.children }
-   }
-
-   bench.report("Pathname#pstrip"){
-      MAX.times{ path1.pstrip }
-   }
-
-   bench.report("Pathname#pstrip!"){
-      MAX.times{ path1.pstrip! }
-   }
-
-   bench.report("Pathname#to_a"){
-      MAX.times{ path1.to_a }
-   }
-
-   bench.report("Pathname#descend"){
-      MAX.times{ path1.descend{} }
-   }
-
-   bench.report("Pathname#ascend"){
-      MAX.times{ path1.ascend{} }
-   }
-
-   bench.report("Pathname#root"){
-      MAX.times{ path1.root }
-   }
-
-   bench.report("Pathname#root?"){
-      MAX.times{ path1.root? }
-   }
-
-   bench.report("Pathname#<=>"){
-      MAX.times{ path1 <=> path2 }
-   }
-
-   bench.report("Pathname#absolute?"){
-      MAX.times{ path1.absolute? }
-   }
-
-   bench.report("Pathname#relative?"){
-      MAX.times{ path1.relative? }
-   }
-
-   bench.report("Pathname#clean"){
-      MAX.times{ path3.clean }
-   }
-
-   bench.report("Pathname#clean!"){
-      MAX.times{ path3.clean! }
-   }
-
-   # Platform specific tests
-   if Config::CONFIG['host_os'].match("mswin")
-      bench.report("Pathname.new(file_url)"){
-         MAX.times{ Pathname.new("file:///C:/usr/local/bin") }
-      }
-
-      bench.report("Pathname#drive_number"){
-         MAX.times{ path1.drive_number }
-      }
-
-      bench.report("Pathname#unc?"){
-         MAX.times{ path1.unc? }
-      }
-
-      bench.report("Pathname#undecorate"){
-         MAX.times{ path1.undecorate }
-      }
-
-      bench.report("Pathname#undecorate!"){
-         MAX.times{ path1.undecorate! }
-      }
-
-      bench.report("Pathname#short_path"){
-         MAX.times{ path1.short_path }
-      }
-
-      bench.report("Pathname#long_path"){
-         MAX.times{ path1.long_path }
-      }
-   else      
-      bench.report("Pathname#realpath"){
-         MAX.times{ path4.realpath }
-      }
-   end
+  bench.report("Pathname.new(path)"){
+    MAX.times{ Pathname.new("/usr/local/bin") }
+  }
+
+  bench.report("Pathname#+(Pathname)"){
+    MAX.times{ path1 + path2 }
+  }
+
+  bench.report("Pathname#+(String)"){
+    MAX.times{ path1 + path2 }
+  }
+
+  bench.report("Pathname#children"){
+    MAX.times{ path1.children }
+  }
+
+  bench.report("Pathname#pstrip"){
+    MAX.times{ path1.pstrip }
+  }
+
+  bench.report("Pathname#pstrip!"){
+    MAX.times{ path1.pstrip! }
+  }
+
+  bench.report("Pathname#to_a"){
+    MAX.times{ path1.to_a }
+  }
+
+  bench.report("Pathname#descend"){
+    MAX.times{ path1.descend{} }
+  }
+
+  bench.report("Pathname#ascend"){
+    MAX.times{ path1.ascend{} }
+  }
+
+  bench.report("Pathname#root"){
+    MAX.times{ path1.root }
+  }
+
+  bench.report("Pathname#root?"){
+    MAX.times{ path1.root? }
+  }
+
+  bench.report("Pathname#<=>"){
+    MAX.times{ path1 <=> path2 }
+  }
+
+  bench.report("Pathname#absolute?"){
+    MAX.times{ path1.absolute? }
+  }
+
+  bench.report("Pathname#relative?"){
+    MAX.times{ path1.relative? }
+  }
+
+  bench.report("Pathname#clean"){
+    MAX.times{ path3.clean }
+  }
+
+  bench.report("Pathname#clean!"){
+    MAX.times{ path3.clean! }
+  }
+
+  # Platform specific tests
+  if RbConfig::CONFIG['host_os'] =~ /mingw|mswin/i
+    bench.report("Pathname.new(file_url)"){
+      MAX.times{ Pathname.new("file:///C:/usr/local/bin") }
+    }
+
+    bench.report("Pathname#drive_number"){
+      MAX.times{ path1.drive_number }
+    }
+
+    bench.report("Pathname#unc?"){
+      MAX.times{ path1.unc? }
+    }
+
+    bench.report("Pathname#undecorate"){
+      MAX.times{ path1.undecorate }
+    }
+
+    bench.report("Pathname#undecorate!"){
+      MAX.times{ path1.undecorate! }
+    }
+
+    bench.report("Pathname#short_path"){
+      MAX.times{ path1.short_path }
+    }
+
+    bench.report("Pathname#long_path"){
+      MAX.times{ path1.long_path }
+    }
+  else
+    bench.report("Pathname#realpath"){
+      MAX.times{ path4.realpath }
+    }
+  end
 end

data/lib/pathname2.rb

@@ -93,7 +93,7 @@ class Pathname < String
   public
 
   # The version of the pathname2 library
-  VERSION = '1.7.4'
+  VERSION = '1.8.0'
 
   # The maximum length of a path
   MAXPATH = 1024 unless defined? MAXPATH # Yes, I willfully violate POSIX

data/pathname2.gemspec

@@ -2,9 +2,9 @@ require 'rubygems'
 
 Gem::Specification.new do |spec|
   spec.name        = 'pathname2'
-  spec.version     = '1.7.4'
+  spec.version     = '1.8.0'
   spec.author      = 'Daniel J. Berger'
-  spec.license     = 'Artistic 2.0'
+  spec.license     = 'Apache 2.0'
   spec.email       = 'djberg96@gmail.com'
   spec.homepage    = 'https://github.com/djberg96/pathname2'
   spec.summary     = 'An alternate implementation of the Pathname class'

data/test/test_pathname.rb

@@ -5,6 +5,7 @@
 # should be run via the test rake task.
 ##############################################################################
 require 'pathname2'
+require 'fileutils'
 require 'rbconfig'
 require 'test-unit'
 include RbConfig
@@ -30,6 +31,10 @@ class TC_Pathname < Test::Unit::TestCase
     @rel_array = []
 
     @mypath = MyPathname.new('/usr/bin')
+
+    @test_file  = 'realpath_test.txt'
+    @link_file  = 'realpath_symlink.txt'
+    @link_file2 = 'realpath_symlink2.txt'
   end
 
   # Convenience method to verify that the receiver was not modified
@@ -62,41 +67,30 @@ class TC_Pathname < Test::Unit::TestCase
 
   # Convenience method for test_relative_path_from_expected_errors
   def assert_relpath_err(to, from)
-    assert_raise(ArgumentError) {
-      Pathname.new(to).relative_path_from(from)
-    }
+    assert_raise(ArgumentError) { Pathname.new(to).relative_path_from(from) }
   end
 
-  def test_file_url_path
+  test "url_path returns expected result" do
     assert_equal('/foo bar/baz', @url_path)
   end
 
-  def test_realpath
+  test "realpath basic functionality" do
+    FileUtils.touch(@test_file) && File.symlink(@test_file, @link_file)
     assert_respond_to(@abs_path, :realpath)
     assert_equal(@@pwd, Pathname.new('.').realpath)
-    assert_kind_of(Pathname, Pathname.new('/dev/stdin').realpath)
-    assert(Pathname.new('/dev/stdin') != Pathname.new('/dev/stdin').realpath)
-    if CONFIG['host_os'] =~ /bsd|darwin|mac/i
-      assert_raises(Errno::ENOENT){ Pathname.new('../blahblah/bogus').realpath }
-    else
-      assert_raises(Errno::ENOENT){ Pathname.new('../bogus').realpath }
-    end
-  end
-
-  def test_realpath_platform
-    case CONFIG['host_os']
-      when /linux/i
-        path1 = '/dev/stdin'
-        assert_true(['/dev/pts/0', '/dev/proc/self/fd/0'].include?(Pathname.new(path1).realpath))
-      when /sunos|solaris/i
-        path1 = '/dev/null'
-        path2 = '/dev/stdin'
-        path3 = '/dev/fd0'   # Multiple symlinks
-
-        assert_equal('/devices/pseudo/mm@0:null', Pathname.new(path1).realpath)
-        assert_equal('/dev/fd/0', Pathname.new(path2).realpath)
-        assert_equal('/devices/pci@1f,0/isa@7/dma@0,0/floppy@0,3f0:c', Pathname.new(path3).realpath)
-    end
+    assert_kind_of(Pathname, Pathname.new(@link_file).realpath)
+  end
+
+  test "realpath returns expected result for simple symlink" do
+    FileUtils.touch(@test_file) && File.symlink(@test_file, @link_file)
+    assert_true(Pathname.new(@link_file) != Pathname.new(@link_file).realpath)
+    assert_raises(Errno::ENOENT){ Pathname.new('../bogus').realpath }
+  end
+
+  test "realpath returns expected result for nested symlink" do
+    FileUtils.touch(@test_file) && File.symlink(@test_file, @link_file) && File.symlink(@link_file, @link_file2)
+    assert_true(Pathname.new(@link_file) != Pathname.new(@link_file2).realpath)
+    assert_equal(Pathname.new(@link_file).realpath, Pathname.new(@link_file2).realpath)
   end
 
   # These tests taken directly from Tanaka's pathname.rb. The one failure
@@ -476,6 +470,14 @@ class TC_Pathname < Test::Unit::TestCase
     @mypath = nil
     @abs_array.clear
     @rel_array.clear
+
+    File.delete(@link_file2) if File.exist?(@link_file2)
+    File.delete(@link_file) if File.exist?(@link_file)
+    File.delete(@test_file) if File.exist?(@test_file)
+
+    @link_file2 = nil
+    @link_file  = nil
+    @test_file  = nil
   end
 
   def self.shutdown

data/test/test_version.rb

@@ -8,6 +8,6 @@ require 'test-unit'
 
 class TC_Pathname_Version < Test::Unit::TestCase
   test "version is set to expected value" do
-    assert_equal('1.7.4', Pathname::VERSION)
+    assert_equal('1.8.0', Pathname::VERSION)
   end
 end

data/test/windows/test_facade.rb

@@ -51,11 +51,11 @@ class TC_Pathname_Facade < Test::Unit::TestCase
     assert_respond_to(@path, :sysopen)
   end
 
-  test "exists? facade works as expected" do
-    assert_respond_to(@path, :exists?)
-    assert_nothing_raised{ @path.exists? }
-    assert_true(Pathname.new("C:\\").exists?)
-    assert_false(Pathname.new("X:\\foo\\bar\\baz").exists?)
+  test "exist? facade works as expected" do
+    assert_respond_to(@path, :exist?)
+    assert_nothing_raised{ @path.exist? }
+    assert_true(Pathname.new("C:\\").exist?)
+    assert_false(Pathname.new("X:\\foo\\bar\\baz").exist?)
   end
 
   def teardown

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pathname2
 version: !ruby/object:Gem::Version
-  version: 1.7.4
+  version: 1.8.0
 platform: ruby
 authors:
 - Daniel J. Berger
@@ -30,7 +30,7 @@ cert_chain:
   EJYzxdPOrx2n6NYR3Hk+vHP0U7UBSveI6+qx+ndQYaeyCn+GRX2PKS9h66YF/Q1V
   tGSHgAmcLlkdGgan182qsE/4kKM=
   -----END CERTIFICATE-----
-date: 2015-09-07 00:00:00.000000000 Z
+date: 2016-06-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: facade
@@ -90,18 +90,24 @@ extra_rdoc_files:
 - CHANGES
 - MANIFEST
 files:
-- CHANGES
-- MANIFEST
-- README
-- Rakefile
+- benchmarks
 - benchmarks/bench_pathname.rb
 - benchmarks/bench_plus.rb
+- certs
 - certs/djberg96_pub.pem
+- CHANGES
+- examples
 - examples/example_pathname.rb
+- lib
 - lib/pathname2.rb
+- MANIFEST
 - pathname2.gemspec
+- Rakefile
+- README
+- test
 - test/test_pathname.rb
 - test/test_version.rb
+- test/windows
 - test/windows/test_append.rb
 - test/windows/test_aref.rb
 - test/windows/test_ascend.rb
@@ -132,7 +138,7 @@ files:
 - test/windows/test_undecorate_bang.rb
 homepage: https://github.com/djberg96/pathname2
 licenses:
-- Artistic 2.0
+- Apache 2.0
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -150,7 +156,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.6.4
 signing_key: 
 specification_version: 4
 summary: An alternate implementation of the Pathname class