arxiv 0.0.4 → 0.0.5

data/lib/arxiv.rb

@@ -17,13 +17,26 @@ module Arxiv
     class MalformedId < StandardError ; end
   end
 
-  ID_FORMAT = /^\d{4}\.\d{4}(?:v\d+)?$/
+  # In 2007, the ArXiv API changed document ID formats:
+  #
+  #    http://arxiv.org/abs/math/0510097v1  (legacy)
+  #    http://arxiv.org/abs/1202.0819v1     (current)
+  #
+  # These constants help us deal with both use cases.
+  #
+  LEGACY_URL_FORMAT = /[^\/]+\/\d+(?:v\d+)?$/
+  CURRENT_URL_FORMAT = /\d{4}\.\d{4}(?:v\d+)?$/
 
-  def self.get(id)
+  LEGACY_ID_FORMAT = /^#{LEGACY_URL_FORMAT}/
+  ID_FORMAT = /^#{CURRENT_URL_FORMAT}/
 
-    id = parse_arxiv_id(id)
+  def self.get(identifier)
 
-    raise Arxiv::Error::MalformedId, "Manuscript ID format is invalid" unless id =~ ID_FORMAT
+    id = parse_arxiv_identifier(identifier)
+
+    unless id =~ ID_FORMAT || id =~ LEGACY_ID_FORMAT
+      raise Arxiv::Error::MalformedId, "Manuscript ID format is invalid"
+    end
 
     url = ::URI.parse("http://export.arxiv.org/api/query?id_list=#{id}")
     response = ::Nokogiri::XML(open(url)).remove_namespaces!
@@ -35,15 +48,32 @@ module Arxiv
 
   private
 
-  def self.parse_arxiv_id(id)
-    if id =~ ID_FORMAT
-      id
-    elsif id =~ /arxiv.org/
-      match = id.match(/[^\/]+$/)
-      match[0] if match
+  def self.parse_arxiv_identifier(identifier)
+    if valid_id?(identifier)
+      identifier
+    elsif valid_url?(identifier)
+      format = legacy_url?(identifier) ? LEGACY_URL_FORMAT : CURRENT_URL_FORMAT
+      identifier.match(/(#{format})/)[1]
     else
-      id
+      identifier # probably an error
     end
   end
 
+  def self.valid_id?(identifier)
+    identifier =~ ID_FORMAT || identifier =~ LEGACY_ID_FORMAT
+  end
+
+  def self.valid_url?(identifier)
+    identifier =~ LEGACY_URL_FORMAT || identifier =~ CURRENT_URL_FORMAT
+  end
+
+  def self.legacy_url?(identifier)
+    identifier =~ LEGACY_URL_FORMAT
+  end
+
+
+
+
+
+
 end

data/lib/arxiv/models/category.rb

@@ -24,7 +24,7 @@ module Arxiv
     end
 
     def long_description
-      "#{abbreviation} (#{description})"
+      description ? "#{abbreviation} (#{description})" : abbreviation
     end
 
   end

data/lib/arxiv/models/manuscript.rb

@@ -20,12 +20,20 @@ module Arxiv
       created_at != updated_at
     end
 
+    def legacy_article?
+      arxiv_url =~ Arxiv::LEGACY_URL_FORMAT
+    end
+
     def arxiv_id
-      arxiv_url.match(/([^\/]+)v\d+$/)[1]
+      arxiv_versioned_id.match(/([^v]+)v\d+$/)[1]
     end
 
     def arxiv_versioned_id
-      arxiv_url.match(/([^\/]+)$/)[1]
+      if legacy_article?
+        arxiv_url.match(/(#{Arxiv::LEGACY_URL_FORMAT})/)[1]
+      else
+        arxiv_url.match(/(#{Arxiv::CURRENT_URL_FORMAT})/)[1]
+      end
     end
 
     def version

data/lib/arxiv/string_scrubber.rb

@@ -1,6 +1,6 @@
 module Arxiv
   class StringScrubber
-    def self.scrub(string)
+     def self.scrub(string)
       string.gsub("\n", ' ').strip.squeeze(" ")
     end
   end

data/lib/arxiv/version.rb

@@ -1,3 +1,3 @@
 module Arxiv
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/spec/arxiv/arxiv_spec.rb

@@ -2,31 +2,43 @@ require 'spec_helper'
 
 module Arxiv
 
-  RSpec::Matchers.define :fetch_valid_manuscript do |expected|
+  RSpec::Matchers.define :fetch do |expected|
     match do |actual|
-      expected_title = "Laser frequency comb techniques for precise astronomical spectroscopy"
-      actual.is_a?(Arxiv::Manuscript) && actual.title == expected_title
+      actual.is_a?(Arxiv::Manuscript) && actual.title == expected
     end
   end
 
   describe "get" do
-    it "should fetch a manuscript when passed a valid id" do
-      Arxiv.get('1202.0819').should fetch_valid_manuscript
-    end
 
-    it "should fetch a manuscript when passed a valid id with a version number" do
-      Arxiv.get('1202.0819v1').should fetch_valid_manuscript
+    context "when using the current arXiv id format" do
+      it "should fetch a manuscript when passed an id" do
+        Arxiv.get('1202.0819').should fetch("Laser frequency comb techniques for precise astronomical spectroscopy")
+      end
+      it "should fetch a manuscript when passed a valid id with a version number" do
+        Arxiv.get('1202.0819v1').should fetch("Laser frequency comb techniques for precise astronomical spectroscopy")
+      end
+      it "should fetch a manuscript when passed full URL" do
+        Arxiv.get('http://arxiv.org/abs/1202.0819').should fetch("Laser frequency comb techniques for precise astronomical spectroscopy")
+      end
     end
 
-    it "should fetch a manuscript when passed full URL for a manuscript" do
-      Arxiv.get('http://arxiv.org/abs/1202.0819').should fetch_valid_manuscript
+    context "when using the legacy arXiv id format" do
+      it "should fetch a manuscript when passed an id" do
+        Arxiv.get('math.DG/0510097').should fetch("The differential topology of loop spaces")
+      end
+      it "should fetch a manuscript when passed a valid id with a version number" do
+        Arxiv.get('math.DG/0510097v1').should fetch("The differential topology of loop spaces")
+      end
+      it "should fetch a manuscript when passed full URL" do
+        Arxiv.get('http://arxiv.org/abs/math.DG/0510097').should fetch("The differential topology of loop spaces")
+      end
     end
 
-    context "errors" do
-      it "should raise a manuscript not found error when the manuscript cannot be found on arXiv" do
+    context "when something goes wrong" do
+      it "should raise an error if the manuscript cannot be found on arXiv" do
         lambda { Arxiv.get('1234.1234') }.should raise_error(Arxiv::Error::ManuscriptNotFound)
       end
-      it "should raise a malformed id error when the manuscript id has an incorrect format" do
+      it "should raise an error if the manuscript has an incorrectly formatted id" do
         lambda { Arxiv.get('cond-mat0709123') }.should raise_error(Arxiv::Error::MalformedId)
       end
     end

data/spec/arxiv/models/category_spec.rb

@@ -2,7 +2,10 @@ require 'spec_helper'
 
 module Arxiv
   describe Category do
-    before(:all) { @category = Arxiv.get('1202.0819').primary_category }
+    before(:all) do
+      @category = Arxiv.get('1202.0819').primary_category
+      @legacy_category = Arxiv.get('math.DG/0510097v1').categories.last
+    end
 
     describe "abbreviation" do
       it "should fetch the category's abbreviation" do
@@ -17,9 +20,12 @@ module Arxiv
     end
 
     describe "long_description" do
-      it "should fetch the category's #long_description" do
+      it "should fetch the category's abbreviation and description"do
         @category.long_description.should == "astro-ph.IM (Physics - Instrumentation and Methods for Astrophysics)"
       end
+      it "should just return the abbreviation when a description cannot be found (e.g. MSC classes)"do
+        @legacy_category.long_description.should == "58D15 (Primary); 58B10 (Secondary)"
+      end
     end
 
   end

data/spec/arxiv/models/manuscript_spec.rb

@@ -2,7 +2,10 @@ require 'spec_helper'
 
 module Arxiv
   describe Manuscript do
-    before(:all) { @manuscript = Arxiv.get('1202.0819') }
+    before(:all) do
+      @manuscript = Arxiv.get('1202.0819')
+      @legacy_manuscript = Arxiv.get('math.DG/0510097v1')
+    end
 
     describe "arxiv_url" do
       it "should fetch the link to the manuscript's page on arXiv" do
@@ -47,21 +50,30 @@ module Arxiv
     end
 
     describe "arxiv_versioned_id" do
-      it "should return the unique document id used by arXiv" do
+      it "should return the unique versioned document id used by arXiv for a current manuscript" do
         @manuscript.arxiv_versioned_id.should == '1202.0819v1'
       end
+      it "should return the unique versioned document id used by arXiv for a legacy manuscript" do
+        @legacy_manuscript.arxiv_versioned_id.should == 'math/0510097v1'
+      end
     end
 
     describe "arxiv_id" do
-      it "should return the unique document id used by arXiv" do
+      it "should return the unique document id used by arXiv for a current manuscript" do
         @manuscript.arxiv_id.should == '1202.0819'
       end
+      it "should return the unique document id used by arXiv for a legacy manuscript" do
+        @legacy_manuscript.arxiv_id.should == 'math/0510097'
+      end
     end
 
     describe "version" do
-      it "should return the manuscript's version number" do
+      it "should return the manuscript's version number for a current manuscript" do
         @manuscript.version.should == 1
       end
+      it "should return the manuscript's version number for a legacy manuscript" do
+        @legacy_manuscript.version.should == 1
+      end
     end
 
     describe "content_types" do
@@ -96,10 +108,18 @@ module Arxiv
     end
 
     describe "primary_category" do
-      it "should description" do
+      it "should description return the manuscript's primary category" do
         @manuscript.primary_category.abbreviation.should == "astro-ph.IM"
       end
     end
 
+    describe "legacy_article?" do
+      it "should return true if the manuscript was upload while the legacy API was still in use" do
+        @legacy_manuscript.should be_legacy_article
+      end
+      it "should return false if the manuscript was uploaded after the transition to the new API" do
+        @manuscript.should_not be_legacy_article
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: arxiv
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -11,11 +11,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-15 00:00:00.000000000Z
+date: 2012-02-16 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: happymapper
-  requirement: &2157600480 !ruby/object:Gem::Requirement
+  requirement: &2156723380 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -23,10 +23,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2157600480
+  version_requirements: *2156723380
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &2157600060 !ruby/object:Gem::Requirement
+  requirement: &2156722840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -34,10 +34,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2157600060
+  version_requirements: *2156722840
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2157599640 !ruby/object:Gem::Requirement
+  requirement: &2156722300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -45,7 +45,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2157599640
+  version_requirements: *2156722300
 description: Makes interacting with arXiv data really easy.
 email:
 - coryschires@gmail.com