pairtree 0.1.0 → 0.3.0

data/spec/pairtree/pairtree_encoding_spec.rb

@@ -1,56 +1,61 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require 'pairtree'
+require "spec_helper"
+require "pairtree"
 
 describe "Pairtree encoding" do
-
-  def roundtrip(id, expected_encoded=nil, expected_path=nil)
+  def roundtrip(id, expected_encoded = nil, expected_path = nil)
     encoded = Pairtree::Identifier.encode(id)
     unless expected_encoded.nil?
-      encoded.should == expected_encoded
+      expect(encoded).to eql(expected_encoded)
     end
     unless expected_path.nil?
       path = Pairtree::Path.id_to_path(id)
-      path.should == expected_path
+      expect(path).to eql(expected_path)
+    end
+    str = Pairtree::Identifier.decode(encoded)
+
+    if str.respond_to? :force_encoding
+      str.force_encoding("UTF-8")
     end
-    Pairtree::Identifier.decode(encoded).should == id
+
+    expect(str).to eql(id)
   end
-  
+
   it "should handle a" do
-    roundtrip('a', 'a', 'a/a')
+    roundtrip("a", "a", "a/a")
   end
 
   it "should handle ab" do
-    roundtrip('ab', 'ab', 'ab/ab')
+    roundtrip("ab", "ab", "ab/ab")
   end
 
   it "should handle abc" do
-    roundtrip('abc', 'abc', 'ab/c/abc')
+    roundtrip("abc", "abc", "ab/c/abc")
   end
 
   it "should handle abcd" do
-    roundtrip('abcd', 'abcd', 'ab/cd/abcd')
+    roundtrip("abcd", "abcd", "ab/cd/abcd")
   end
 
   it "should handle space" do
-    roundtrip('hello world', 'hello^20world', 'he/ll/o^/20/wo/rl/d/hello^20world')
+    roundtrip("hello world", "hello^20world", "he/ll/o^/20/wo/rl/d/hello^20world")
   end
 
   it "should handle slash" do
-    roundtrip("/","=",'=/=')
+    roundtrip("/", "=", "=/=")
   end
 
   it "should handle urn" do
-    roundtrip('http://n2t.info/urn:nbn:se:kb:repos-1','http+==n2t,info=urn+nbn+se+kb+repos-1','ht/tp/+=/=n/2t/,i/nf/o=/ur/n+/nb/n+/se/+k/b+/re/po/s-/1/http+==n2t,info=urn+nbn+se+kb+repos-1')
+    roundtrip("http://n2t.info/urn:nbn:se:kb:repos-1", "http+==n2t,info=urn+nbn+se+kb+repos-1", "ht/tp/+=/=n/2t/,i/nf/o=/ur/n+/nb/n+/se/+k/b+/re/po/s-/1/http+==n2t,info=urn+nbn+se+kb+repos-1")
   end
-  
+
   it "should handle wtf" do
-    roundtrip('what-the-*@?#!^!?', "what-the-^2a@^3f#!^5e!^3f", "wh/at/-t/he/-^/2a/@^/3f/#!/^5/e!/^3/f/what-the-^2a@^3f#!^5e!^3f")
+    roundtrip("what-the-*@?#!^!?", "what-the-^2a@^3f#!^5e!^3f", "wh/at/-t/he/-^/2a/@^/3f/#!/^5/e!/^3/f/what-the-^2a@^3f#!^5e!^3f")
   end
 
   it "should handle special characters" do
     roundtrip('\\"*+,<=>?^|', "^5c^22^2a^2b^2c^3c^3d^3e^3f^5e^7c")
   end
-  
+
   it "should roundtrip hardcore Unicode" do
     roundtrip(%{
        1. Euro Symbol: €.
@@ -69,9 +74,9 @@ describe "Pairtree encoding" do
       14. Thai: ฉันกินกระจกได้ แต่มันไม่ทำให้ฉันเจ็บ "
     })
   end
-  
+
   it "should roundtrip French" do
-    roundtrip('Années de Pèlerinage')
+    roundtrip("Années de Pèlerinage", "Ann^c3^a9es^20de^20P^c3^a8lerinage", "An/n^/c3/^a/9e/s^/20/de/^2/0P/^c/3^/a8/le/ri/na/ge/Ann^c3^a9es^20de^20P^c3^a8lerinage")
     roundtrip(%{
       Années de Pèlerinage (Years of Pilgrimage) (S.160, S.161,
       S.163) is a set of three suites by Franz Liszt for solo piano. Liszt's
@@ -82,6 +87,5 @@ describe "Pairtree encoding" do
       was composed well after the first two volumes and often displays less
       showy virtuosity and more harmonic experimentation.
     })
- end
-  
+  end
 end

data/spec/pairtree/pairtree_obj_spec.rb

@@ -1,69 +1,67 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require 'pairtree'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+require "pairtree"
 
 describe "Pairtree::Obj" do
-
   before(:all) do
     @base_path = File.join(File.dirname(__FILE__), "../test_data/working")
     Dir.chdir(File.join(File.dirname(__FILE__), "../test_data")) do
-      FileUtils.cp_r('fixtures/pairtree_root_spec', './working')
+      FileUtils.cp_r("fixtures/pairtree_root_spec", "./working")
     end
     @root = Pairtree.at(@base_path)
-    @obj = @root.get('pfx:abc123def')
+    @obj = @root.get("pfx:abc123def")
   end
-  
+
   after(:all) do
     FileUtils.rm_rf(@base_path)
   end
-  
+
   it "should read a file" do
-    @obj.read('content.xml').should == '<content/>'
+    expect(@obj.read("content.xml")).to eql("<content/>")
   end
 
   it "should have entries" do
-    @obj.entries.should == ['content.xml']
+    expect(@obj.entries).to eql(["content.xml"])
   end
-  
+
   it "should glob" do
-    @obj['*.xml'].should == ['content.xml']
-    @obj['*.txt'].should == []
+    expect(@obj["*.xml"]).to eql(["content.xml"])
+    expect(@obj["*.txt"]).to eql([])
   end
-  
+
   it "should be enumerable" do
-    block_body = mock('block_body')
-    block_body.should_receive(:yielded).with('content.xml')
+    block_body = double("block_body")
+    expect(block_body).to receive(:yielded).with("content.xml")
     @obj.each { |file| block_body.yielded(file) }
   end
-  
+
   describe "Call a bunch of File methods" do
     before(:each) do
-      @target = File.join(@base_path, 'pairtree_root/ab/c1/23/de/f/abc123def/content.xml')
+      @target = File.join(@base_path, "pairtree_root/ab/c1/23/de/f/abc123def/content.xml")
     end
-    
+
     it "should open a file" do
-      File.should_receive(:open).with(@target,'r')
-      @obj.open('content.xml','r')
+      expect(File).to receive(:open).with(@target, "r")
+      @obj.open("content.xml", "r")
     end
 
     it "should call delete" do
-      File.should_receive(:delete).with(@target)
-      @obj.delete('content.xml')
+      expect(File).to receive(:delete).with(@target)
+      @obj.delete("content.xml")
     end
 
     it "should call link" do
-      File.should_receive(:link).with(@target,@target + '.2')
-      @obj.link('content.xml','content.xml.2')
+      expect(File).to receive(:link).with(@target, @target + ".2")
+      @obj.link("content.xml", "content.xml.2")
     end
 
     it "should call rename" do
-      File.should_receive(:rename).with(@target,@target + '.new')
-      @obj.rename('content.xml','content.xml.new')
+      expect(File).to receive(:rename).with(@target, @target + ".new")
+      @obj.rename("content.xml", "content.xml.new")
     end
 
     it "should call utime" do
-      File.should_receive(:utime).with(0,1,@target)
-      @obj.utime(0,1,'content.xml')
+      expect(File).to receive(:utime).with(0, 1, @target)
+      @obj.utime(0, 1, "content.xml")
     end
   end
-  
-end
+end

data/spec/pairtree/pairtree_path_spec.rb

@@ -1,35 +1,33 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require 'pairtree'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+require "pairtree"
 
 describe "Pairtree::Path" do
-  
   after(:each) do
     Pairtree::Path.set_leaf { |id| id }
   end
-  
+
   it "should generate an encoded id as the leaf path by default" do
-    Pairtree::Path.leaf('abc/def').should == "abc=def"
+    expect(Pairtree::Path.leaf("abc/def")).to eql("abc=def")
   end
 
   it "should accept a nil override" do
     Pairtree::Path.set_leaf nil
-    Pairtree::Path.leaf('abc/def').should == ""
+    expect(Pairtree::Path.leaf("abc/def")).to eql("")
   end
-  
+
   it "should accept a scalar override" do
-    Pairtree::Path.set_leaf 'obj'
-    Pairtree::Path.leaf('abc/def').should == "obj"
+    Pairtree::Path.set_leaf "obj"
+    expect(Pairtree::Path.leaf("abc/def")).to eql("obj")
   end
-  
+
   it "should accept a Proc override" do
-    lp = Proc.new { |id| id.reverse }
+    lp = proc { |id| id.reverse }
     Pairtree::Path.set_leaf(lp)
-    Pairtree::Path.leaf('abc/def').should == "fed=cba"
+    expect(Pairtree::Path.leaf("abc/def")).to eql("fed=cba")
   end
-  
+
   it "should accept a block override" do
     Pairtree::Path.set_leaf { |id| id.reverse }
-    Pairtree::Path.leaf('abc/def').should == "fed=cba"
+    expect(Pairtree::Path.leaf("abc/def")).to eql("fed=cba")
   end
-  
 end

data/spec/pairtree/pairtree_root_spec.rb

@@ -1,63 +1,61 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require 'pairtree'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+require "pairtree"
 
 describe "Pairtree::Root" do
-  
   before(:all) do
     @base_path = File.join(File.dirname(__FILE__), "../test_data/working")
     Dir.chdir(File.join(File.dirname(__FILE__), "../test_data")) do
-      FileUtils.cp_r('fixtures/pairtree_root_spec', './working')
+      FileUtils.cp_r("fixtures/pairtree_root_spec", "./working")
     end
     @root = Pairtree.at(@base_path)
   end
-  
+
   after(:all) do
     FileUtils.rm_rf(@base_path)
   end
-  
+
   it "should have the correct prefix" do
-    @root.prefix.should == 'pfx:'
+    expect(@root.prefix).to eql("pfx:")
   end
-  
+
   it "should be in the correct location" do
-    File.expand_path(@root.path).should == File.expand_path(@base_path)
-    File.expand_path(@root.root).should == File.expand_path(File.join(@base_path, "pairtree_root"))
+    expect(File.expand_path(@root.path)).to eql(File.expand_path(@base_path))
+    expect(File.expand_path(@root.root)).to eql(File.expand_path(File.join(@base_path, "pairtree_root")))
   end
-  
+
   it "should list identifiers" do
-    @root.list.should == ['pfx:abc123def']
+    expect(@root.list).to eql(["pfx:abc123def"])
   end
-  
+
   it "should verify whether an identifier exists" do
-    @root.exists?('pfx:abc123def').should be_true
-    @root.exists?('pfx:abc123jkl').should be_false
+    expect(@root.exists?("pfx:abc123def")).to be true
+    expect(@root.exists?("pfx:abc123jkl")).to be false
   end
-  
+
   it "should raise an exception if an invalid prefix is used" do
-    lambda { @root.exists?('xfp:abc123def') }.should raise_error(Pairtree::IdentifierError)
+    expect { @root.exists?("xfp:abc123def") }.to raise_error(Pairtree::IdentifierError)
   end
-  
+
   it "should get a ppath for a valid ID" do
-    obj = @root.get 'pfx:abc123def'
-    obj.class.should == Pairtree::Obj
-    File.expand_path(obj.path).should == File.expand_path(File.join(@base_path, "pairtree_root/ab/c1/23/de/f/abc123def/"))
+    obj = @root.get "pfx:abc123def"
+    expect(obj.class).to eql(Pairtree::Obj)
+    expect(File.expand_path(obj.path)).to eql(File.expand_path(File.join(@base_path, "pairtree_root/ab/c1/23/de/f/abc123def/")))
   end
-  
+
   it "should raise an exception when attempting to get a ppath for an invalid ID" do
-    lambda { @root.get 'pfx:abc123jkl' }.should raise_error(Errno::ENOENT)
+    expect { @root.get "pfx:abc123jkl" }.to raise_error(Errno::ENOENT)
   end
-  
+
   it "should create a new ppath" do
-    obj = @root.mk 'pfx:abc123jkl'
-    obj.class.should == Pairtree::Obj
-    File.expand_path(obj.path).should == File.expand_path(File.join(@base_path, "pairtree_root/ab/c1/23/jk/l/abc123jkl/"))
-    @root.exists?('pfx:abc123jkl').should be_true
+    obj = @root.mk "pfx:abc123jkl"
+    expect(obj.class).to eql(Pairtree::Obj)
+    expect(File.expand_path(obj.path)).to eql(File.expand_path(File.join(@base_path, "pairtree_root/ab/c1/23/jk/l/abc123jkl/")))
+    expect(@root.exists?("pfx:abc123jkl")).to be true
   end
-  
+
   it "should delete a ppath" do
-    @root.exists?('pfx:abc123jkl').should be_true
-    @root.purge!('pfx:abc123jkl')
-    @root.exists?('pfx:abc123jkl').should be_false
+    expect(@root.exists?("pfx:abc123jkl")).to be true
+    @root.purge!("pfx:abc123jkl")
+    expect(@root.exists?("pfx:abc123jkl")).to be false
   end
-  
-end
+end

data/spec/pairtree/pairtree_spec.rb

@@ -1,14 +1,13 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require 'pairtree'
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+require "pairtree"
 
 describe "Pairtree" do
-  
   before(:all) do
     @base_path = File.join(File.dirname(__FILE__), "../test_data/working")
   end
-  
+
   it "should raise an error if a non-existent is specified without :create" do
-    lambda { Pairtree.at(@base_path) }.should raise_error(Pairtree::PathError)
+    expect { Pairtree.at(@base_path) }.to raise_error(Pairtree::PathError)
   end
 
   describe "new pairtree" do
@@ -17,53 +16,50 @@ describe "Pairtree" do
     end
 
     it "should create a new pairtree" do
-      prefix = 'my_prefix:'
-      pt = Pairtree.at(@base_path, :prefix => prefix, :create => true)
-      pt.prefix.should == prefix
-      File.read(File.join(@base_path, 'pairtree_prefix')).should == prefix
-      pt.root.should == File.join(@base_path, 'pairtree_root')
-      pt.pairtree_version.should == Pairtree::SPEC_VERSION
+      prefix = "my_prefix:"
+      pt = Pairtree.at(@base_path, prefix: prefix, create: true)
+      expect(pt.prefix).to eql(prefix)
+      expect(File.read(File.join(@base_path, "pairtree_prefix"))).to eql(prefix)
+      expect(pt.root).to eql(File.join(@base_path, "pairtree_root"))
+      expect(pt.pairtree_version).to eql(Pairtree::SPEC_VERSION)
     end
-    
   end
-  
+
   describe "existing pairtree" do
     before(:all) do
       Dir.chdir(File.join(File.dirname(__FILE__), "../test_data")) do
-        FileUtils.cp_r('fixtures/pairtree_root_spec', './working')
+        FileUtils.cp_r("fixtures/pairtree_root_spec", "./working")
       end
     end
-    
+
     after(:all) do
       FileUtils.rm_rf(@base_path)
     end
 
     it "should raise an error if a regular file is specified as a root" do
-      lambda { Pairtree.at(File.join(@base_path, "pairtree_prefix"), :create => true) }.should raise_error(Pairtree::PathError)
+      expect { Pairtree.at(File.join(@base_path, "pairtree_prefix"), create: true) }.to raise_error(Pairtree::PathError)
     end
-    
+
     it "should read the prefix if none is specified" do
-      Pairtree.at(@base_path).prefix.should == File.read(File.join(@base_path, 'pairtree_prefix'))
+      expect(Pairtree.at(@base_path).prefix).to eql(File.read(File.join(@base_path, "pairtree_prefix")))
     end
 
     it "should not complain if the given prefix matches the saved prefix" do
-      Pairtree.at(@base_path, :prefix => 'pfx:').prefix.should == File.read(File.join(@base_path, 'pairtree_prefix'))
+      expect(Pairtree.at(@base_path, prefix: "pfx:").prefix).to eql(File.read(File.join(@base_path, "pairtree_prefix")))
     end
-    
+
     it "should raise an error if the given prefix does not match the saved prefix" do
-      lambda { Pairtree.at(@base_path, :prefix => 'wrong-prefix:') }.should raise_error(Pairtree::IdentifierError)
+      expect { Pairtree.at(@base_path, prefix: "wrong-prefix:") }.to raise_error(Pairtree::IdentifierError)
     end
-    
+
     it "should not complain if the given version matches the saved version" do
-      Pairtree.at(@base_path, :version => Pairtree::SPEC_VERSION).pairtree_version.should == Pairtree::SPEC_VERSION
-      Pairtree.at(@base_path, :version => Pairtree::SPEC_VERSION, :create => true).pairtree_version.should == Pairtree::SPEC_VERSION
+      expect(Pairtree.at(@base_path, version: Pairtree::SPEC_VERSION).pairtree_version).to eql(Pairtree::SPEC_VERSION)
+      expect(Pairtree.at(@base_path, version: Pairtree::SPEC_VERSION, create: true).pairtree_version).to eql(Pairtree::SPEC_VERSION)
     end
 
     it "should raise an error if the given version does not match the saved version" do
-      lambda { Pairtree.at(@base_path, :version => 0.2) }.should raise_error(Pairtree::VersionMismatch)
-      lambda { Pairtree.at(@base_path, :version => 0.2, :create => true) }.should raise_error(Pairtree::VersionMismatch)
+      expect { Pairtree.at(@base_path, version: 0.2) }.to raise_error(Pairtree::VersionMismatch)
+      expect { Pairtree.at(@base_path, version: 0.2, create: true) }.to raise_error(Pairtree::VersionMismatch)
     end
-    
   end
-  
 end

data/spec/spec_helper.rb

@@ -1,10 +1,22 @@
 $LOAD_PATH.unshift(File.dirname(__FILE__))
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), "..", "lib"))
 
-require 'bundler/setup'
-require 'rspec'
-require 'rspec/autorun'
+require "bundler/setup"
+require "rspec"
+
+require "simplecov"
+require "simplecov-lcov"
+SimpleCov::Formatter::LcovFormatter.config do |c|
+  c.report_with_single_file = true
+  c.single_report_path = "coverage/lcov.info"
+end
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter.new([
+  SimpleCov::Formatter::HTMLFormatter,
+  SimpleCov::Formatter::LcovFormatter
+])
+SimpleCov.start
+
+require "pairtree"
 
 RSpec.configure do |config|
-  
 end

metadata

@@ -1,73 +1,85 @@
 --- !ruby/object:Gem::Specification
 name: pairtree
 version: !ruby/object:Gem::Version
-  version: 0.1.0
-  prerelease: 
+  version: 0.3.0
 platform: ruby
 authors:
-- Chris Beer
+- Chris Beer, Bryan Hockey, Michael Slone, Aaron Elkiss
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2010-12-23 00:00:00.000000000 Z
+date: 2023-03-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &2164523720 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164523720
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2164523000 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '3.12'
   type: :development
   prerelease: false
-  version_requirements: *2164523000
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.12'
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &2164522020 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164522020
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: RedCloth
-  requirement: &2164521500 !ruby/object:Gem::Requirement
-    none: false
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164521500
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 
-email: chris@cbeer.info
+email: 
 executables: []
 extensions: []
 extra_rdoc_files:
 - LICENSE.txt
-- README.textile
+- README.md
 files:
-- .document
-- .gitignore
+- ".document"
+- ".github/workflows/tests.yml"
+- ".gitignore"
 - Gemfile
 - LICENSE.txt
-- README.textile
+- README.md
 - Rakefile
 - lib/pairtree.rb
 - lib/pairtree/identifier.rb
@@ -85,39 +97,27 @@ files:
 - spec/test_data/fixtures/pairtree_root_spec/pairtree_prefix
 - spec/test_data/fixtures/pairtree_root_spec/pairtree_root/ab/c1/23/de/f/abc123def/content.xml
 - spec/test_data/fixtures/pairtree_root_spec/pairtree_version0_1
-homepage: http://github.com/microservices/pairtree
+homepage: http://github.com/ruby-microservices/pairtree
 licenses:
 - Apache2
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 3.3.5
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Ruby Pairtree implementation
-test_files:
-- spec/pairtree/pairtree_encoding_spec.rb
-- spec/pairtree/pairtree_obj_spec.rb
-- spec/pairtree/pairtree_path_spec.rb
-- spec/pairtree/pairtree_root_spec.rb
-- spec/pairtree/pairtree_spec.rb
-- spec/spec_helper.rb
-- spec/test_data/fixtures/pairtree_root_spec/pairtree_prefix
-- spec/test_data/fixtures/pairtree_root_spec/pairtree_root/ab/c1/23/de/f/abc123def/content.xml
-- spec/test_data/fixtures/pairtree_root_spec/pairtree_version0_1
-has_rdoc: 
+test_files: []