stanford-mods 0.0.23 → 0.0.24

data/lib/stanford-mods/version.rb

@@ -1,6 +1,6 @@
 module Stanford
   module Mods
     # this is the Ruby Gem version
-    VERSION = "0.0.23"
+    VERSION = "0.0.24"
   end
 end

data/spec/searchworks_format_spec.rb

@@ -0,0 +1,103 @@
+# encoding: UTF-8
+require 'spec_helper'
+
+describe "Format field from Searchworks mixin for Stanford::Mods::Record" do
+
+  before(:all) do
+    @smods_rec = Stanford::Mods::Record.new
+    @ns_decl = "xmlns='#{Mods::MODS_NS}'"
+  end
+
+  it "should check genre as part of deciding format" do
+    m = "<mods #{@ns_decl}><typeOfResource>text</typeOfResouce><genre>thesis</genre></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == ['Thesis']
+  end
+  
+  it 'should work for datasets' do
+    m = "<mods #{@ns_decl}><typeOfResource>software, multimedia</typeOfResouce></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == ['Computer File']
+  end
+
+  it 'should work for books' do
+    m = "<mods #{@ns_decl}><typeOfResource>text</typeOfResouce><originInfo><issuance>monographic</issueance></originInfo></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == ['Book']
+  end
+
+  it "should work for a hydrus journal article" do
+    m = "<mods #{@ns_decl}><typeOfResource>text</typeOfResouce><genre>article</genre></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == ['Journal/Periodical']
+  end
+
+  it "should work for image" do
+    m = "<mods #{@ns_decl}><typeOfResource>still image</typeOfResouce></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == ['Image']
+  end
+
+  context "Hydrus mappings per GRYPHONDOR-207" do
+    it "should give a format of Computer File for <genre>game</genre> and <typeOfResource>software, multimedia</typeOfResouce>" do
+      m = "<mods #{@ns_decl}><genre>game</genre><typeOfResource>software, multimedia</typeOfResouce></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.format.should == ['Computer File']
+    end
+    it "should give a format of Video for <genre>motion picture</genre> and <typeOfResource>moving image</typeOfResouce>" do
+      m = "<mods #{@ns_decl}><genre>motion picture</genre><typeOfResource>moving image</typeOfResouce></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.format.should == ['Video']
+    end
+    it "should give a format of Sound Recording for <genre>sound</genre> and <typeOfResource>sound recording-nonmusical</typeOfResouce>" do
+      m = "<mods #{@ns_decl}><genre>sound</genre><typeOfResource>sound recording-nonmusical</typeOfResouce></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.format.should == ['Sound Recording']
+    end
+    it "should give a format of Conference Proceedings for <genre>conference publication</genre> and <typeOfResource>text</typeOfResouce>" do
+      m = "<mods #{@ns_decl}><genre>conference publication</genre><typeOfResource>text</typeOfResouce></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.format.should == ['Conference Proceedings']
+    end
+    it "should give a format of Book for <genre>technical report</genre> and <typeOfResource>text</typeOfResouce>" do
+      m = "<mods #{@ns_decl}><genre>technical report</genre><typeOfResource>text</typeOfResouce></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.format.should == ['Book']
+    end    
+  end
+
+  # Student Project Reports: spec via email from Vitus, August 16, 2013
+  it "should give a format of Other for <genre>student project report</genre> and <typeOfResource>text</typeOfResouce>" do
+    m = "<mods #{@ns_decl}><genre>student project report</genre><typeOfResource>text</typeOfResouce></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == ['Other']
+  end
+  
+  it "should give a format of Music - Score for <typeOfResource>notated music</typeOfResouce>" do
+    m = "<mods #{@ns_decl}><typeOfResource>notated music</typeOfResouce></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == ['Music - Score']
+  end
+  
+  it "should return nothing if there is no format info" do
+    m = "<mods #{@ns_decl}><originInfo>
+    <dateCreated>1904</dateCreated>
+    </originInfo></mods>"
+    @smods_rec = Stanford::Mods::Record.new
+    @smods_rec.from_str(m)
+    @smods_rec.format.should == []
+  end
+
+end

data/spec/searchworks_pub_dates_spec.rb

@@ -0,0 +1,236 @@
+# encoding: UTF-8
+require 'spec_helper'
+
+describe "Date methods in Searchworks mixin for Stanford::Mods::Record" do
+
+  before(:all) do
+    @smods_rec = Stanford::Mods::Record.new
+    @ns_decl = "xmlns='#{Mods::MODS_NS}'"
+  end
+  
+  context "pub_dates" do
+    it "should choose the first date" do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>1904</dateCreated>
+      <dateCreated>1904</dateCreated>
+      <dateIssued>1906</dateIssued>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_dates.should == ['1906','1904','1904']
+    end
+  end
+
+  context "pub_date" do
+    it "should choose the first date" do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>1904</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '1904'
+    end
+    it "should parse a date" do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>Aug. 3rd, 1886</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '1886'
+    end
+    it "should remove question marks and brackets" do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>Aug. 3rd, [18]86?</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '1886'
+    end
+    it 'should handle an s after the decade' do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>early 1890s</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '1890'
+    end
+    it 'should choose a date ending with CE if there are multiple dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateIssued>7192 AM (li-Adam) / 1684 CE</dateIssued><issuance>monographic</issuance></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '1684'
+    end
+    it 'should handle hyphenated range dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateIssued>1282 AH / 1865-6 CE</dateIssued><issuance>monographic</issuance></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '1865'
+    end
+    it 'should work with multiple 4 digit dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateCreated>Text dated June 4, 1594; miniatures added by 1596</dateCreated></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+       @smods_rec.pub_date.should == '1594'
+    end
+    it 'should work on 3 digit BC dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateCreated>300 B.C.</dateCreated></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+       @smods_rec.pub_year.should == '-700'
+       @smods_rec.pub_date.should == '-700'
+       @smods_rec.pub_date_sort.should =='-700'
+       @smods_rec.pub_date_facet.should == '300 B.C.'
+    end
+    it 'should handle century based dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateIssued>13th century AH / 19th CE</dateIssued><issuance>monographic</issuance></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date_facet.should == '19th century'
+      @smods_rec.pub_date_sort.should =='1800'
+      @smods_rec.pub_date.should == '18--'
+    end
+    it 'should handle multiple CE dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateIssued>6 Dhu al-Hijjah 923 AH / 1517 CE -- 7 Rabi I 924 AH / 1518 CE</dateIssued><issuance>monographic</issuance></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '1517'
+      @smods_rec.pub_date_sort.should =='1517'
+      @smods_rec.pub_date_facet.should == '1517'
+    end
+    it 'should handle this case from walters' do
+      m = "<mods #{@ns_decl}><originInfo><dateIssued>Late 14th or early 15th century CE</dateIssued><issuance>monographic</issuance></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '14--'
+      @smods_rec.pub_date_sort.should =='1400'
+      @smods_rec.pub_date_facet.should == '15th century'
+    end
+    it 'should work on 3 digit dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateIssued>966 CE</dateIssued><issuance>monographic</issuance></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date.should == '966'
+      @smods_rec.pub_date_sort.should =='0966'
+      @smods_rec.pub_date_facet.should == '966'
+    end
+    it 'should work on 3 digit dates' do
+      m = "<mods #{@ns_decl}><originInfo><dateIssued>3rd century AH / 9th CE</dateIssued><issuance>monographic</issuance></originInfo>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+  
+      @smods_rec.pub_date.should == '8--'
+      @smods_rec.pub_date_sort.should =='0800'
+      @smods_rec.pub_date_facet.should == '9th century'
+    end  
+  end # pub_date
+
+  context "dates with u notation (198u, 19uu)" do    
+    context "single digit u notation (198u)" do
+      before(:all) do
+        m = "<mods #{@ns_decl}>
+        <originInfo>
+          <dateIssued encoding=\"marc\" point=\"start\" keyDate=\"yes\">198u</dateIssued>
+          <dateIssued encoding=\"marc\" point=\"end\">9999</dateIssued>
+        </originInfo></mods>"
+        @smods_rec = Stanford::Mods::Record.new
+        @smods_rec.from_str(m)
+      end
+      it "get_u_year recognizes notation" do
+        dates = ["198u", "9999"]
+        uDate = @smods_rec.get_u_year dates
+        uDate.should eql("1980")
+      end
+      it 'pub_date: 198u = 1980' do
+        @smods_rec.pub_date.should == '1980'
+      end
+      it "pub_date_sort: 198u = 1980" do
+        @smods_rec.pub_date_sort.should =='1980'
+      end
+      it "pub_date_facet: 198u = 1980" do
+        @smods_rec.pub_date_facet.should == '1980'
+      end
+    end
+    context "double digit u notation (19uu)" do
+      before(:all) do
+        m = "<mods #{@ns_decl}>
+        <originInfo>
+          <dateIssued encoding=\"marc\" point=\"start\" keyDate=\"yes\">19uu</dateIssued>
+          <dateIssued encoding=\"marc\" point=\"end\">9999</dateIssued>
+        </originInfo></mods>"
+        @smods_rec = Stanford::Mods::Record.new
+        @smods_rec.from_str(m)
+      end
+      it "get_u_year recognizes notation" do
+        dates = ["19uu", "9999"]
+        uDate = @smods_rec.get_u_year dates
+        uDate.should eql("19--")
+      end
+      it 'pub_date: 19uu = 19--' do
+        @smods_rec.pub_date.should == '19--'
+      end
+      it "pub_date_sort: 19uu = 1900" do
+        @smods_rec.pub_date_sort.should =='1900'
+      end
+      it "pub_date_facet: 19uu = 20th century" do
+        @smods_rec.pub_date_facet.should == '20th century'
+      end
+    end
+  end # u notation
+
+
+
+
+  context 'pub_date_sort' do
+    before :all do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>Aug. 3rd, 1886</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+    end
+    it 'should work on normal dates' do
+      @smods_rec.stub(:pub_date).and_return('1945')
+      @smods_rec.pub_date_sort.should == '1945'
+    end
+    it 'should work on 3 digit dates' do
+      @smods_rec.stub(:pub_date).and_return('945')
+      @smods_rec.pub_date_sort.should == '0945'
+    end
+    it 'should work on century dates' do
+      @smods_rec.stub(:pub_date).and_return('16--')
+      @smods_rec.pub_date_sort.should == '1600'
+    end
+    it 'should work on 3 digit century dates' do
+      @smods_rec.stub(:pub_date).and_return('9--')
+      @smods_rec.pub_date_sort.should == '0900'
+    end
+  end
+
+  context "pub_date_groups" do
+    it 'should generate the groups' do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>1904</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date_groups(1904).should == ['More than 50 years ago']
+    end
+    it 'should work for a modern date too' do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>1904</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date_groups(2013).should == ["This year"]
+    end
+    it 'should work ok given a nil date' do
+      m = "<mods #{@ns_decl}><originInfo>
+      <dateCreated>1904</dateCreated>
+      </originInfo></mods>"
+      @smods_rec = Stanford::Mods::Record.new
+      @smods_rec.from_str(m)
+      @smods_rec.pub_date_groups(nil).should == nil
+    end
+  end #context pub date groups
+
+end

data/spec/searchworks_spec.rb

@@ -115,6 +115,7 @@ describe "Searchworks mixin for Stanford::Mods::Record" do
       m = "<mods #{@ns_decl}><titleInfo><title>Jerk</title><subTitle>A Tale of Tourettes</subTitle><nonSort>The</nonSort></titleInfo></mods>"
       @smods_rec.from_str m      
     end
+    
     context "short title (for title_245a_search, title_245a_display) " do
       it "should call :short_titles" do
         @smods_rec.should_receive(:short_titles) # in Mods gem
@@ -124,6 +125,7 @@ describe "Searchworks mixin for Stanford::Mods::Record" do
         @smods_rec.sw_short_title.should == 'The Jerk'
       end
     end
+    
     context "full title (for title_245_search, title_display, title_full_display)" do
       it "should call :full_titles" do
         @smods_rec.should_receive(:full_titles) # in Mods gem
@@ -140,6 +142,7 @@ describe "Searchworks mixin for Stanford::Mods::Record" do
         @smods_rec.sw_full_title.should == 'Pius V. Saint, [Michaele Gisleri)'     
       end
     end
+    
     context "additional titles (for title_variant_search)" do
       before(:all) do
         m = "<mods #{@ns_decl}>
@@ -170,6 +173,7 @@ describe "Searchworks mixin for Stanford::Mods::Record" do
           @addl_titles = @smods_rec.sw_addl_titles
       end
     end    
+    
     context "sort title" do
       it "should be a String" do
         @smods_rec.sw_sort_title.should be_an_instance_of(String)

data/spec/searchworks_subject_raw_spec.rb

@@ -0,0 +1,384 @@
+# encoding: UTF-8
+require 'spec_helper'
+
+describe "Searchworks mixin for Stanford::Mods::Record" do
+
+  before(:all) do
+    @smods_rec = Stanford::Mods::Record.new
+    @ns_decl = "xmlns='#{Mods::MODS_NS}'"
+  end
+
+  context "sw subject raw methods" do
+    before(:all) do
+      @genre = 'genre top level'
+      @cart_coord = '6 00 S, 71 30 E'
+      @s_genre = 'genre in subject'
+      @geo = 'Somewhere'
+      @geo_code = 'us'
+      @hier_geo_country = 'France'
+      @s_name = 'name in subject'
+      @occupation = 'worker bee'
+      @temporal = 'temporal'
+      @s_title = 'title in subject'
+      @topic = 'topic'
+      m = "<mods #{@ns_decl}>
+        <genre>#{@genre}</genre>
+        <subject><cartographics><coordinates>#{@cart_coord}</coordinates></cartographics></subject>
+        <subject><genre>#{@s_genre}</genre></subject>
+        <subject><geographic>#{@geo}</geographic></subject>
+        <subject><geographicCode authority='iso3166'>#{@geo_code}</geographicCode></subject>
+        <subject><hierarchicalGeographic><country>#{@hier_geo_country}</country></hierarchicalGeographic></subject>
+        <subject><name><namePart>#{@s_name}</namePart></name></subject>
+        <subject><occupation>#{@occupation}</occupation></subject>
+        <subject><temporal>#{@temporal}</temporal></subject>
+        <subject><titleInfo><title>#{@s_title}</title></titleInfo></subject>
+        <subject><topic>#{@topic}</topic></subject>      
+      </mods>"
+      @smods_rec.from_str m
+      @sw_geographic_search = @smods_rec.sw_geographic_search 
+      @sw_subject_titles = @smods_rec.sw_subject_titles
+      @sw_subject_names = @smods_rec.sw_subject_names
+    end
+    
+    context "sw_subject_names" do
+      it "should contain <subject><name><namePart> values" do
+        @sw_subject_names.should include(@s_name)
+      end
+      it "should not contain non-name subject subelements" do
+        @sw_subject_names.should_not include(@cart_coord)
+        @sw_subject_names.should_not include(@s_genre)
+        @sw_subject_names.should_not include(@geo)
+        @sw_subject_names.should_not include(@geo_code)
+        @sw_subject_names.should_not include(@hier_geo_country)
+        @sw_subject_names.should_not include(@occupation)
+        @sw_subject_names.should_not include(@temporal)
+        @sw_subject_names.should_not include(@topic)
+        @sw_subject_names.should_not include(@s_title)
+      end
+      it "should not contain subject/name/role" do
+        m = "<mods #{@ns_decl}>
+              <subject><name type='personal'>
+              	<namePart>Alterman, Eric</namePart>
+              	<displayForm>Eric Alterman</displayForm>
+              	<role>
+              	  	<roleTerm type='text'>creator</roleTerm>
+              	  	<roleTerm type='code'>cre</roleTerm>
+              	</role>
+              </name></subject></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_names.find { |sn| sn =~ /cre/ }.should == nil
+      end
+      it "should not contain subject/name/affiliation" do
+        m = "<mods #{@ns_decl}>
+              <subject><name type='personal'>
+              	<namePart type='termsOfAddress'>Dr.</namePart>
+              	<namePart>Brown, B. F.</namePart>
+              	<affiliation>Chemistry Dept., American University</affiliation>
+              </name></subject></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_names.find { |sn| sn =~ /Chemistry/ }.should == nil
+      end
+      it "should not contain subject/name/description" do
+        m = "<mods #{@ns_decl}>
+              <subject><name type='personal'>
+              	<namePart>Abrams, Michael</namePart>
+              	<description>American artist, 20th c.</description>
+              </name></subject></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_names.find { |sn| sn =~ /artist/ }.should == nil
+      end
+      it "should not include top level name element" do
+        m = "<mods #{@ns_decl}>
+              <name type='personal'>
+              	<namePart>Abrams, Michael</namePart>
+              	<description>American artist, 20th c.</description>
+              </name></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_names.should == []
+      end
+      it "should have one value for each name element" do
+        m = "<mods #{@ns_decl}>
+              <subject>
+                <name><namePart>first</namePart></name>
+                <name><namePart>second</namePart></name>
+              </subject>
+              <subject>
+                <name><namePart>third</namePart></name>
+              </subject>
+              </mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_names.should == ['first', 'second', 'third']
+      end
+      it "should be an empty Array if there are no values in the mods" do
+        m = "<mods #{@ns_decl}><note>notit</note></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_names.should == []
+      end
+      it "should be an empty Array if there are empty values in the mods" do
+        m = "<mods #{@ns_decl}><subject><name><namePart/></name></subject><note>notit</note></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_names.should == []
+      end
+      context "combining subelements" do
+        before(:all) do
+          m = "<mods #{@ns_decl}>
+                <subject>
+                  <name>
+                    <namePart>first</namePart>
+                    <namePart>second</namePart>
+                  </name>
+                </subject>
+              </mods>"
+          @smods_rec.from_str m
+        end
+        it "uses a ', ' as the separator by default" do
+          @smods_rec.sw_subject_names.should == ['first, second']
+        end          
+        it "honors any string value passed in for the separator" do
+          @smods_rec.sw_subject_names(' --').should == ['first --second']
+        end
+      end
+    end # sw_subject_names
+    
+    context "sw_subject_titles" do
+      it "should contain <subject><titleInfo> subelement values" do
+        @sw_subject_titles.should include(@s_title)
+      end
+      it "should not contain non-name subject subelements" do
+        @sw_subject_titles.should_not include(@cart_coord)
+        @sw_subject_titles.should_not include(@s_genre)
+        @sw_subject_titles.should_not include(@geo)
+        @sw_subject_titles.should_not include(@geo_code)
+        @sw_subject_titles.should_not include(@hier_geo_country)
+        @sw_subject_titles.should_not include(@s_name)
+        @sw_subject_titles.should_not include(@occupation)
+        @sw_subject_titles.should_not include(@temporal)
+        @sw_subject_titles.should_not include(@topic)
+      end
+      it "should not include top level titleInfo element" do
+        m = "<mods #{@ns_decl}><titleInfo><title>Oklahoma</title></titleInfo></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_titles.should == []
+      end
+      it "should have one value for each titleInfo element" do
+        m = "<mods #{@ns_decl}>
+              <subject>
+                <titleInfo><title>first</title></titleInfo>
+                <titleInfo><title>second</title></titleInfo>
+              </subject>
+              <subject>
+                <titleInfo><title>third</title></titleInfo>
+              </subject>
+              </mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_titles.should == ['first', 'second', 'third']
+      end
+      it "should be an empty Array if there are no values in the mods" do
+        m = "<mods #{@ns_decl}><note>notit</note></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_titles.should == []
+      end
+      it "should be an empty Array if there are empty values in the mods" do
+        m = "<mods #{@ns_decl}><subject><titleInfo><title/></titleInfo></subject><note>notit</note></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_subject_titles.should == []
+      end
+      context "combining subelements" do
+        before(:all) do
+          m = "<mods #{@ns_decl}>
+                <subject>
+                  <titleInfo>
+                    <title>first</title>
+                    <subTitle>second</subTitle>
+                  </titleInfo>
+                </subject>
+              </mods>"
+          @smods_rec.from_str m
+        end
+        it "uses a ' ' as the separator by default" do
+          @smods_rec.sw_subject_titles.should == ['first second']
+        end          
+        it "honors any string value passed in for the separator" do
+          @smods_rec.sw_subject_titles(' --').should == ['first --second']
+        end
+        it "includes all subelements in the order of occurrence" do
+          m = "<mods #{@ns_decl}>
+                <subject>
+                  <titleInfo>
+                    <partName>1</partName>
+                    <nonSort>2</nonSort>
+                    <partNumber>3</partNumber>
+                    <title>4</title>
+                    <subTitle>5</subTitle>
+                  </titleInfo>
+                </subject>
+              </mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_subject_titles.should == ['1 2 3 4 5']
+        end
+      end
+    end # sw_subject_titles
+    
+    
+    context "sw_geographic_search" do
+      it "should contain subject <geographic> subelement data" do
+        @sw_geographic_search.should include(@geo)
+      end
+      it "should contain subject <hierarchicalGeographic> subelement data" do
+        @sw_geographic_search.should include(@hier_geo_country)
+      end
+      it "should contain translation of <geographicCode> subelement data with translated authorities" do
+        m = "<mods #{@ns_decl}><subject><geographicCode authority='marcgac'>e-er</geographicCode></subject></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_geographic_search.should include('Estonia')
+      end
+      it "should not contain other subject element data" do
+        @sw_geographic_search.should_not include(@genre)
+        @sw_geographic_search.should_not include(@cart_coord)
+        @sw_geographic_search.should_not include(@s_genre)
+        @sw_geographic_search.should_not include(@s_name)
+        @sw_geographic_search.should_not include(@occupation)
+        @sw_geographic_search.should_not include(@temporal)
+        @sw_geographic_search.should_not include(@topic)
+        @sw_geographic_search.should_not include(@s_title)
+      end
+      it "should be [] if there are no values in the MODS" do
+        m = "<mods #{@ns_decl}><note>notit</note></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_geographic_search.should == []
+      end
+      it "should not be empty Array if there are only subject/geographic elements" do
+        m = "<mods #{@ns_decl}><subject><geographic>#{@geo}</geographic></subject></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_geographic_search.should == [@geo]
+      end
+      it "should not be empty Array if there are only subject/hierarchicalGeographic" do
+        m = "<mods #{@ns_decl}><subject><hierarchicalGeographic><country>#{@hier_geo_country}</country></hierarchicalGeographic></subject></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_geographic_search.should == [@hier_geo_country]
+      end
+      it "should not be empty Array if there are only subject/geographicCode elements" do
+        m = "<mods #{@ns_decl}><subject><geographicCode authority='marcgac'>e-er</geographicCode></subject></mods>"
+        @smods_rec.from_str m
+        @smods_rec.sw_geographic_search.should == ['Estonia']
+      end
+      context "geographic subelement" do
+        it "should have a separate value for each geographic element" do
+          m = "<mods #{@ns_decl}>
+                <subject>
+                <geographic>Mississippi</geographic>
+                <geographic>Tippah County</geographic>
+                </subject>
+                <subject><geographic>Washington (D.C.)</geographic></subject>
+              </mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.should == ['Mississippi', 'Tippah County', 'Washington (D.C.)']
+        end
+        it "should be empty Array if there are only empty values in the MODS" do
+          m = "<mods #{@ns_decl}><subject><geographic/></subject><note>notit</note></mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.should == []
+        end
+      end
+      context "hierarchicalGeographic subelement" do
+        it "should have a separate value for each hierarchicalGeographic element" do
+          m = "<mods #{@ns_decl}>
+                <subject>
+                  <hierarchicalGeographic><area>first</area></hierarchicalGeographic>
+                  <hierarchicalGeographic><area>second</area></hierarchicalGeographic>
+                </subject>
+                <subject><hierarchicalGeographic><area>third</area></hierarchicalGeographic></subject>
+              </mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.should == ['first', 'second', 'third']
+        end
+        it "should be empty Array if there are only empty values in the MODS" do
+          m = "<mods #{@ns_decl}><subject><hierarchicalGeographic/></subject><note>notit</note></mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.should == []
+        end
+        context "combining subelements" do
+          before(:all) do
+            m = "<mods #{@ns_decl}>
+            <subject>
+              <hierarchicalGeographic>
+              	<country>Canada</country>
+              	<province>British Columbia</province>
+              	<city>Vancouver</city>
+              </hierarchicalGeographic>
+            </subject></mods>"
+            @smods_rec.from_str m
+          end
+          it "uses a space as the separator by default" do
+            @smods_rec.sw_geographic_search.should == ['Canada British Columbia Vancouver']
+          end          
+          it "honors any string value passed in for the separator" do
+            @smods_rec.sw_geographic_search(' --').should == ['Canada --British Columbia --Vancouver']
+          end
+        end
+      end # hierarchicalGeographic
+      context "geographicCode subelement" do
+        before(:all) do
+          m = "<mods #{@ns_decl}>
+            <subject><geographicCode authority='marcgac'>n-us-md</geographicCode></subject>
+            <subject><geographicCode authority='marcgac'>e-er</geographicCode></subject>
+            <subject><geographicCode authority='marccountry'>mg</geographicCode></subject>
+            <subject><geographicCode authority='iso3166'>us</geographicCode></subject>
+          </mods>"
+          @smods_rec.from_str m
+          @geo_search_from_codes = @smods_rec.sw_geographic_search   
+        end
+        it "should not add untranslated values" do
+          @geo_search_from_codes.should_not include('n-us-md')
+          @geo_search_from_codes.should_not include('e-er')
+          @geo_search_from_codes.should_not include('mg')
+          @geo_search_from_codes.should_not include('us')
+        end
+        it "should translate marcgac codes" do
+          @geo_search_from_codes.should include('Estonia')
+        end
+        it "should translate marccountry codes" do
+          @geo_search_from_codes.should include('Madagascar')
+        end
+        it "should not translate other codes" do
+          @geo_search_from_codes.should_not include('United States')
+        end
+        it "should have a separate value for each geographicCode element" do
+          m = "<mods #{@ns_decl}>
+                <subject>
+                  <geographicCode authority='marcgac'>e-er</geographicCode>
+                	<geographicCode authority='marccountry'>mg</geographicCode>
+                </subject>
+                <subject><geographicCode authority='marcgac'>n-us-md</geographicCode></subject>
+              </mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.should == ['Estonia', 'Madagascar', 'Maryland']
+        end
+        it "should be empty Array if there are only empty values in the MODS" do
+          m = "<mods #{@ns_decl}><subject><geographicCode/></subject><note>notit</note></mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.should == []
+        end
+        it "should add the translated value if it wasn't present already" do
+          m = "<mods #{@ns_decl}>
+            <subject><geographic>Somewhere</geographic></subject>
+            <subject><geographicCode authority='marcgac'>e-er</geographicCode></subject>
+          </mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.size.should == 2
+          @smods_rec.sw_geographic_search.should include('Estonia')
+        end
+        it "should not add the translated value if it was already present" do
+          m = "<mods #{@ns_decl}>
+            <subject><geographic>Estonia</geographic></subject>
+            <subject><geographicCode authority='marcgac'>e-er</geographicCode></subject>
+          </mods>"
+          @smods_rec.from_str m
+          @smods_rec.sw_geographic_search.size.should == 1
+          @smods_rec.sw_geographic_search.should == ['Estonia']
+        end
+      end
+    end # sw_geographic_search
+  end # context sw subject methods
+
+end