pennmarc 1.0.12.pre1 → 1.0.14

data/lib/pennmarc/helpers/location.rb

@@ -3,10 +3,12 @@
 module PennMARC
   # Methods that return Library and Location values from Alma enhanced MARC fields
   class Location < Helper
+    ONLINE_LIBRARY = 'Online library'
+
     class << self
       # Retrieves library location from enriched marc 'itm' or 'hld' fields, giving priority to the item location over
       # the holdings location. Returns item's location if available. Otherwise, returns holding's location.
-      # {PennMARC::EnrichedMarc} maps enriched marc fields and subfields created during Alma publishing.
+      # {PennMARC::Enriched} maps enriched marc fields and subfields created during Alma publishing.
       # @see https://developers.exlibrisgroup.com/alma/apis/docs/bibs/R0VUIC9hbG1hd3MvdjEvYmlicy97bW1zX2lkfQ==/
       #   Alma documentation for these added fields
       # @param [MARC::Record] record
@@ -19,7 +21,7 @@ module PennMARC
       # Retrieves the specific location from enriched marc 'itm' or 'hld' fields, giving priority to the item location
       # over the holdings location. Returns item library location if available. Otherwise, returns holdings library
       # location.
-      # {PennMARC::EnrichedMarc} maps enriched marc fields and subfields created during Alma publishing.
+      # {PennMARC::Enriched} maps enriched marc fields and subfields created during Alma publishing.
       # @see https://developers.exlibrisgroup.com/alma/apis/docs/bibs/R0VUIC9hbG1hd3MvdjEvYmlicy97bW1zX2lkfQ==/
       #   Alma documentation for these added fields
       # @param [MARC::Record] record
@@ -31,7 +33,7 @@ module PennMARC
 
       # Base method to retrieve location data from enriched marc 'itm' or 'hld' fields, giving priority to the item
       # location over the holdings location. Returns item location if available. Otherwise, returns holdings location.
-      # {PennMARC::EnrichedMarc} maps enriched marc fields and subfields created during Alma publishing.
+      # {PennMARC::Enriched} maps enriched marc fields and subfields created during Alma publishing.
       # @see https://developers.exlibrisgroup.com/alma/apis/docs/bibs/R0VUIC9hbG1hd3MvdjEvYmlicy97bW1zX2lkfQ==/
       #   Alma documentation for these added fields
       # @param [MARC::Record] record
@@ -61,7 +63,9 @@ module PennMARC
             location_map[subfield.value.to_sym][display_value.to_sym]
           }.flatten.compact_blank
         }.uniq
-        locations << 'Online library' if record.fields(PennMARC::EnrichedMarc::TAG_ELECTRONIC_INVENTORY).any?
+        if record.tags.intersect?([Enriched::Pub::ELEC_INVENTORY_TAG, Enriched::Api::ELEC_INVENTORY_TAG])
+          locations << ONLINE_LIBRARY
+        end
         locations
       end
 
@@ -82,17 +86,18 @@ module PennMARC
         # Since item records may reflect locations more accurately, we use them if they exist;
         # if not, we use the holdings.
 
-        tag = PennMARC::EnrichedMarc::TAG_HOLDING
-        subfield_code = PennMARC::EnrichedMarc::SUB_HOLDING_SHELVING_LOCATION
-
         # if the record has an enriched item field present, use it
-        if field_defined?(record, PennMARC::EnrichedMarc::TAG_ITEM)
-          tag = PennMARC::EnrichedMarc::TAG_ITEM
-          subfield_code = PennMARC::EnrichedMarc::SUB_ITEM_CURRENT_LOCATION
-        # otherwise, use the api enriched field value
-        elsif field_defined?(record, EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY)
-          tag = EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY
-          subfield_code = EnrichedMarc::AlmaApi::SUB_HOLDING_LOCATION_CODE
+        if field_defined?(record, Enriched::Pub::ITEM_TAG)
+          tag = Enriched::Pub::ITEM_TAG
+          subfield_code = Enriched::Pub::ITEM_CURRENT_LOCATION
+        # if the record has API inventory tags, use them
+        elsif field_defined?(record, Enriched::Api::PHYS_INVENTORY_TAG)
+          tag = Enriched::Api::PHYS_INVENTORY_TAG
+          subfield_code = Enriched::Api::PHYS_LOCATION_CODE
+        # otherwise use Pub holding tags
+        else
+          tag = Enriched::Pub::PHYS_INVENTORY_TAG
+          subfield_code = Enriched::Pub::PHYS_LOCATION_CODE
         end
 
         { tag: tag, subfield_code: subfield_code }

data/lib/pennmarc/helpers/subject.rb

@@ -173,9 +173,9 @@ module PennMARC
 
         case type.to_sym
         when :facet
-          "#{term[:parts].join('--')} #{term[:lasts].join(' ')}".strip
+          term[:parts].join('--').strip
         when :display
-          "#{term[:parts].join('--')} #{term[:lasts].join(' ')} #{term[:append].join(' ')}".strip
+          "#{term[:parts].join('--')} #{term[:append].join(' ')}".strip
         end
       end
 
@@ -227,9 +227,9 @@ module PennMARC
       # @todo do i need all this?
       # @todo do i need to handle punctuation? see append_new_part
       # @param [MARC::DataField] field
-      # @return [Hash{Symbol->Integer | Array}]
+      # @return [Hash{Symbol => Integer, Array}, Nil]
       def build_subject_hash(field)
-        term_info = { count: 0, parts: [], append: [], lasts: [], uri: nil,
+        term_info = { count: 0, parts: [], append: [], uri: nil,
                       local: field.indicator2 == '4' || field.tag.starts_with?('69'), # local subject heading
                       vernacular: field.tag == '880' }
         field.each do |subfield|
@@ -251,8 +251,8 @@ module PennMARC
             # 'e' is relator term; not sure what 'w' is. These are used to append for record-view display only
             term_info[:append] << subfield.value.strip # TODO: map relator code?
           when 'b', 'c', 'd', 'p', 'q', 't'
-            # these are appended to the last component if possible (i.e., when joined, should have no delimiter)
-            term_info[:lasts] << subfield.value.strip
+            # these are appended to the last component (part) if possible (i.e., when joined, should have no delimiter)
+            term_info[:parts].last << ", #{subfield.value.strip}"
             term_info[:count] += 1
           else
             # the usual case; add a new component to `parts`

data/lib/pennmarc/mappings/locations.yml

@@ -588,6 +588,10 @@ finecore:
   specific_location: Fisher Fine Arts Library - Core Reading Collection
   library: Fisher Fine Arts Library
   display: Fine Arts Library - Core Reading Collection
+fineexhib:
+  specific_location: Fisher Fine Arts Library - Exhibitions Corridor
+  library: Fisher Fine Arts Library
+  display: Fine Arts Library - Exhibitions Corridor
 finelock:
   specific_location: Fisher Fine Arts Library - Locked Case
   library:

data/lib/pennmarc/util.rb

@@ -14,7 +14,7 @@ module PennMARC
     end
 
     # Join subfields from a field selected based on a provided proc
-    # @param [MARC::DataField] field
+    # @param [MARC::DataField, nil] field
     # @param [Proc] selector
     # @return [String]
     def join_subfields(field, &selector)
@@ -125,6 +125,21 @@ module PennMARC
       string.sub map[trailer.to_sym], ''
     end
 
+    # Intelligently append given punctuation to the end of a string
+    # @param [Symbol] trailer
+    # @param [String] string
+    # @return [String]
+    def append_trailing(trailer, string)
+      return string if string.end_with?('.', '-')
+
+      map = { semicolon: ';',
+              colon: ':',
+              slash: '/',
+              comma: ',',
+              period: '.' }
+      string + map[trailer.to_sym]
+    end
+
     # MARC 880 field "Alternate Graphic Representation" contains text "linked" to another
     # field (e.g., 254 [Title]) used as an alternate representation. Often used to hold
     # translations of title values. A common need is to extract subfields as selected by

data/lib/pennmarc/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PennMARC
-  VERSION = '1.0.12.pre1'
+  VERSION = '1.0.14'
 end

data/spec/lib/pennmarc/helpers/access_spec.rb

@@ -10,7 +10,7 @@ describe 'PennMARC::Access' do
       let(:record) { marc_record fields: [marc_field(tag: tag)] }
 
       context 'with enrichment via the Alma publishing process' do
-        let(:tag) { PennMARC::EnrichedMarc::TAG_ELECTRONIC_INVENTORY }
+        let(:tag) { PennMARC::Enriched::Pub::ELEC_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::ONLINE)
@@ -18,7 +18,7 @@ describe 'PennMARC::Access' do
       end
 
       context 'with enrichment with availability info via the Alma API' do
-        let(:tag) { PennMARC::EnrichedMarc::AlmaApi::TAG_ELECTRONIC_INVENTORY }
+        let(:tag) { PennMARC::Enriched::Api::ELEC_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::ONLINE)
@@ -30,7 +30,7 @@ describe 'PennMARC::Access' do
       let(:record) { marc_record fields: [marc_field(tag: tag)] }
 
       context 'with enrichment via the Alma publishing process' do
-        let(:tag) { PennMARC::EnrichedMarc::TAG_HOLDING }
+        let(:tag) { PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::AT_THE_LIBRARY)
@@ -38,7 +38,7 @@ describe 'PennMARC::Access' do
       end
 
       context 'with enrichment with availability info via the Alma API' do
-        let(:tag) { PennMARC::EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY }
+        let(:tag) { PennMARC::Enriched::Api::PHYS_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::AT_THE_LIBRARY)
@@ -48,7 +48,7 @@ describe 'PennMARC::Access' do
 
     context 'with a record containing a link to a finding aid (as a handle link)' do
       let(:record) do
-        marc_record fields: [marc_field(tag: PennMARC::EnrichedMarc::TAG_HOLDING),
+        marc_record fields: [marc_field(tag: PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG),
                              marc_field(tag: '856', subfields: location_and_access_subfields, **indicators)]
       end
 

data/spec/lib/pennmarc/helpers/classification_spec.rb

@@ -17,9 +17,9 @@ describe 'PennMARC::Classification' do
 
   describe '.facet' do
     context 'with enrichment via the Alma publishing process' do
-      let(:tag) { PennMARC::EnrichedMarc::TAG_ITEM }
-      let(:call_number_type_sf) { PennMARC::EnrichedMarc::SUB_ITEM_CALL_NUMBER_TYPE }
-      let(:call_number_sf) { PennMARC::EnrichedMarc::SUB_ITEM_CALL_NUMBER }
+      let(:tag) { PennMARC::Enriched::Pub::ITEM_TAG }
+      let(:call_number_type_sf) { PennMARC::Enriched::Pub::ITEM_CALL_NUMBER_TYPE }
+      let(:call_number_sf) { PennMARC::Enriched::Pub::ITEM_CALL_NUMBER }
 
       it 'returns expected values' do
         expect(helper.facet(record)).to contain_exactly('T - Technology', '600 - Technology',
@@ -28,9 +28,9 @@ describe 'PennMARC::Classification' do
     end
 
     context 'with enrichment with availability info via Alma Api' do
-      let(:tag) { PennMARC::EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY }
-      let(:call_number_type_sf) { PennMARC::EnrichedMarc::AlmaApi::SUB_PHYSICAL_CALL_NUMBER_TYPE }
-      let(:call_number_sf) { PennMARC::EnrichedMarc::AlmaApi::SUB_PHYSICAL_CALL_NUMBER }
+      let(:tag) { PennMARC::Enriched::Api::PHYS_INVENTORY_TAG }
+      let(:call_number_type_sf) { PennMARC::Enriched::Api::PHYS_CALL_NUMBER_TYPE }
+      let(:call_number_sf) { PennMARC::Enriched::Api::PHYS_CALL_NUMBER }
 
       it 'returns expected values' do
         expect(helper.facet(record)).to contain_exactly('T - Technology', '600 - Technology',

data/spec/lib/pennmarc/helpers/creator_spec.rb

@@ -17,11 +17,11 @@ describe 'PennMARC::Creator' do
       end
 
       it 'contains the expected search field values for a single author work' do
-        expect(helper.search(record, relator_map: mapping)).to eq [
+        expect(helper.search(record, relator_map: mapping)).to contain_exactly(
           'Name Surname http://cool.uri/12345 author 1900-2000.',
           'Surname, Name http://cool.uri/12345 author 1900-2000.',
           'Alternative Surname'
-        ]
+        )
       end
     end
 
@@ -32,8 +32,33 @@ describe 'PennMARC::Creator' do
       end
 
       it 'contains the expected search field values for a corporate author work' do
-        expect(helper.search(record, relator_map: mapping)).to eq ['Group of People Annual Meeting Author.',
-                                                                   'Alt. Group Name Alt. Annual Meeting']
+        expect(helper.search(record, relator_map: mapping)).to contain_exactly(
+          'Group of People Annual Meeting, Author.',
+          'Alt. Group Name Alt. Annual Meeting'
+        )
+      end
+    end
+  end
+
+  describe '.search_aux' do
+    let(:record) { marc_record fields: fields }
+
+    context 'with a record that has an added name in the 7xx field' do
+      let(:fields) do
+        [marc_field(tag: '100', subfields: { a: 'Author', c: 'Fancy', d: 'active 24th century AD', '4': 'aut' }),
+         marc_field(tag: '700', subfields: { a: 'Author, Added' }),
+         marc_field(tag: '880', subfields: { '6': '100', a: 'Alt Author', c: 'Alt Fanciness' }),
+         marc_field(tag: '880', subfields: { '6': '700', a: 'Alt Added Author' })]
+      end
+
+      it 'contains the expected search_aux field values for a single author work' do
+        expect(helper.search_aux(record, relator_map: mapping)).to contain_exactly(
+          'Author Fancy active 24th century AD, Author.',
+          'Author, Added.',
+          'Added Author.',
+          'Alt Author Alt Fanciness',
+          'Alt Added Author'
+        )
       end
     end
   end
@@ -187,6 +212,18 @@ describe 'PennMARC::Creator' do
     end
   end
 
+  describe '.conference_search' do
+    let(:record) do
+      marc_record fields: [
+        marc_field(tag: '111', subfields: { a: 'MARC History Symposium', c: 'Moscow', '4': 'aut' })
+      ]
+    end
+
+    it 'returns conference name information for searching without relator value' do
+      expect(helper.conference_search(record)).to eq ['MARC History Symposium Moscow']
+    end
+  end
+
   describe '.contributor_show' do
     let(:record) do
       marc_record fields: [
@@ -217,7 +254,4 @@ describe 'PennMARC::Creator' do
       )
     end
   end
-
-  # describe '.conference_search'
-  # describe '.search_aux'
 end

data/spec/lib/pennmarc/helpers/format_spec.rb

@@ -68,9 +68,9 @@ describe 'PennMARC::Format' do
       context 'with API enriched fields' do
         let(:record) do
           marc_record fields: [
-            marc_field(tag: PennMARC::EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY, subfields: {
+            marc_field(tag: PennMARC::Enriched::Api::PHYS_INVENTORY_TAG, subfields: {
                          :h => 'AB123',
-                         PennMARC::EnrichedMarc::AlmaApi::SUB_PHYSICAL_CALL_NUMBER_TYPE => '.456 Microfilm'
+                         PennMARC::Enriched::Api::PHYS_CALL_NUMBER_TYPE => '.456 Microfilm'
                        })
           ]
         end
@@ -83,9 +83,9 @@ describe 'PennMARC::Format' do
       context 'with publishing enriched fields' do
         let(:record) do
           marc_record fields: [
-            marc_field(tag: PennMARC::EnrichedMarc::TAG_HOLDING,
+            marc_field(tag: PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG,
                        subfields: { :h => 'AB123',
-                                    PennMARC::EnrichedMarc::SUB_ITEM_CALL_NUMBER_TYPE => '.456 Microfilm' })
+                                    PennMARC::Enriched::Pub::ITEM_CALL_NUMBER_TYPE => '.456 Microfilm' })
           ]
         end
 

data/spec/lib/pennmarc/helpers/inventory_spec.rb

@@ -0,0 +1,129 @@
+# frozen_string_literal: true
+
+describe 'PennMARC::Inventory' do
+  include MarcSpecHelpers
+
+  let(:helper) { PennMARC::Inventory }
+  let(:record) do
+    marc_record fields: fields
+  end
+
+  describe 'physical' do
+    let(:fields) do
+      [marc_field(tag: mapper::PHYS_INVENTORY_TAG,
+                  subfields: subfields)]
+    end
+
+    context 'with API enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Api }
+      let(:subfields) do
+        { mapper::PHYS_CALL_NUMBER => 'AB123.4',
+          mapper::PHYS_HOLDING_ID => '123456789',
+          mapper::PHYS_LOCATION_CODE => 'vanpelt',
+          mapper::PHYS_LOCATION_NAME => 'Van Pelt Library',
+          mapper::PHYS_PRIORITY => '1' }
+      end
+
+      it 'returns expected array of hash values' do
+        expect(helper.physical(record)).to contain_exactly(
+          { call_num: 'AB123.4', holding_id: '123456789', location_code: 'vanpelt',
+            location_name: 'Van Pelt Library', priority: '1' }
+        )
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Pub }
+      let(:subfields) do
+        { mapper::HOLDING_CLASSIFICATION_PART => 'AB123',
+          mapper::HOLDING_ITEM_PART => '.4',
+          mapper::PHYS_HOLDING_ID => '123456789',
+          mapper::PHYS_LOCATION_CODE => 'vanpelt',
+          mapper::PHYS_LOCATION_NAME => 'Van Pelt Library' }
+      end
+
+      it 'returns expected array of hash values' do
+        expect(helper.physical(record)).to contain_exactly(
+          { call_num: 'AB123.4', holding_id: '123456789', location_code: 'vanpelt',
+            location_name: 'Van Pelt Library', priority: nil }
+        )
+      end
+    end
+  end
+
+  describe 'electronic' do
+    let(:fields) do
+      [marc_field(tag: mapper::ELEC_INVENTORY_TAG,
+                  subfields: subfields)]
+    end
+    let(:subfields) do
+      { mapper::ELEC_PORTFOLIO_ID => '234567890',
+        mapper::ELEC_SERVICE_URL => 'https://www.iwish.com',
+        mapper::ELEC_COLLECTION_NAME => 'All Articles Repo',
+        mapper::ELEC_COVERAGE_STMT => 'All time',
+        mapper::ELEC_PUBLIC_NOTE => 'Portfolio public note' }
+    end
+
+    context 'with API enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Api }
+
+      it 'returns expected array of hash values' do
+        expect(helper.electronic(record)).to contain_exactly(
+          { portfolio_id: '234567890', url: 'https://www.iwish.com', collection_name: 'All Articles Repo',
+            coverage: 'All time', note: 'Portfolio public note' }
+        )
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Pub }
+
+      it 'returns expected array of hash values' do
+        expect(helper.electronic(record)).to contain_exactly(
+          { portfolio_id: '234567890', url: 'https://www.iwish.com', collection_name: 'All Articles Repo',
+            coverage: 'All time', note: 'Portfolio public note' }
+        )
+      end
+    end
+  end
+
+  describe 'electronic_portfolio_count' do
+    let(:fields) { [marc_field(tag: inventory_tag), marc_field(tag: inventory_tag)] }
+
+    context 'with API enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Api::ELEC_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.electronic_portfolio_count(record)).to eq 2
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Pub::ELEC_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.electronic_portfolio_count(record)).to eq 2
+      end
+    end
+  end
+
+  describe 'physical_holding_count' do
+    let(:fields) { [marc_field(tag: inventory_tag), marc_field(tag: inventory_tag)] }
+
+    context 'with API enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Api::PHYS_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.physical_holding_count(record)).to eq 2
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.physical_holding_count(record)).to eq 2
+      end
+    end
+  end
+end

data/spec/lib/pennmarc/helpers/location_spec.rb

@@ -4,11 +4,15 @@ describe 'PennMARC::Location' do
   include MarcSpecHelpers
 
   let(:helper) { PennMARC::Location }
+  let(:enriched_marc) { PennMARC::Enriched }
   let(:mapping) { location_map }
 
   describe 'location' do
     context "with only 'itm' field present" do
-      let(:record) { marc_record(fields: [marc_field(tag: 'itm', subfields: { g: 'stor' })]) }
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' })])
+      end
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
@@ -19,7 +23,10 @@ describe 'PennMARC::Location' do
     end
 
     context "with only 'hld' field present" do
-      let(:record) { marc_record(fields: [marc_field(tag: 'hld', subfields: { c: 'stor' })]) }
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
+                                        subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'stor' })])
+      end
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
@@ -29,10 +36,12 @@ describe 'PennMARC::Location' do
       end
     end
 
-    context "with both 'hld' and 'itm' fields present" do
+    context 'with both holding and item tag fields present=' do
       let(:record) do
-        marc_record(fields: [marc_field(tag: 'itm', subfields: { g: 'stor' }),
-                             marc_field(tag: 'hld', subfields: { c: 'dent' })])
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' }),
+                             marc_field(tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
+                                        subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'dent' })])
       end
 
       it 'returns item location' do
@@ -42,7 +51,7 @@ describe 'PennMARC::Location' do
     end
 
     context 'with multiple library locations' do
-      let(:record) { marc_record(fields: [marc_field(tag: 'itm', subfields: { g: %w[dent] })]) }
+      let(:record) { marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG, subfields: { g: %w[dent] })]) }
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
@@ -60,17 +69,26 @@ describe 'PennMARC::Location' do
     end
 
     context 'with electronic inventory tag' do
-      let(:record) { marc_record(fields: [marc_field(tag: 'itm', subfields: { g: 'stor' }), marc_field(tag: 'prt')]) }
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' }),
+                             marc_field(tag: enriched_marc::Pub::ELEC_INVENTORY_TAG)])
+      end
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
-                               display_value: :library)).to contain_exactly('LIBRA', 'Online library')
+                               display_value: :library)).to contain_exactly('LIBRA', helper::ONLINE_LIBRARY)
       end
     end
 
     context 'with AVA fields' do
       let(:record) do
-        marc_record(fields: [marc_field(tag: 'AVA', subfields: { b: 'Libra', c: 'LIBRA', j: 'stor' })])
+        marc_record(fields: [marc_field(tag: enriched_marc::Api::PHYS_INVENTORY_TAG,
+                                        subfields: {
+                                          enriched_marc::Api::PHYS_LIBRARY_CODE => 'Libra',
+                                          enriched_marc::Api::PHYS_LOCATION_NAME => 'LIBRA',
+                                          enriched_marc::Api::PHYS_LOCATION_CODE => 'stor'
+                                        })])
       end
 
       it 'returns expected values' do
@@ -79,5 +97,18 @@ describe 'PennMARC::Location' do
         )
       end
     end
+
+    context 'with AVE fields' do
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Api::ELEC_INVENTORY_TAG,
+                                        subfields: { enriched_marc::Api::ELEC_COLLECTION_NAME => 'Nature' })])
+      end
+
+      it 'returns expected values' do
+        expect(helper.location(record: record, location_map: mapping, display_value: :library)).to(
+          contain_exactly(helper::ONLINE_LIBRARY)
+        )
+      end
+    end
   end
 end