pennmarc 1.0.23 → 1.0.25

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7c906223a9bcdb4ba4792ac50462b8c57e41e71fa1e83cd8e63018d360ebfe0a
-  data.tar.gz: 5d1a420e07734370c95ab1f83d3c519a1d75a7faaa807f1d3ee436808779c586
+  metadata.gz: 8b540ecd84333f097b50a3682732229854b188fcde76921eb0ec184622a690dd
+  data.tar.gz: 9203fadee45b9ad65ec20880a1acbc23356000b748d32d07428ec093590cd55a
 SHA512:
-  metadata.gz: 32e47c254c1a2bb78e861003c864bdbc22833439cff22ebe00148df081c74fed85f60e2f743b5ea65065e5db576e3a304710b1a68b2cdc1e08a1a5f7808e5fc3
-  data.tar.gz: 6051840a95afdcb41640abf2c42ce699deddc93970c5d117cb445474d4f5998996ee05651ec1e59da3e45886e00f6c058b881b1bf9103de92deb5cf6d868ce80
+  metadata.gz: 6770815e8f5f828c15451fc71c04dbef6a131203e8c58d721c063b15f218d5814a71b37fd6c4f088a9c7164d5d989ff6933abc820094068921f218c937a87410
+  data.tar.gz: 94eee63c29ae06151816a532a5afb7db1f27490de798651c909dc07fd0f33de1b4a1ee26057b6e3a8abc5d15c54d09709df8942e6e21ad828c1296fe3191725e

data/lib/pennmarc/enriched.rb

@@ -12,6 +12,7 @@ module PennMARC
       PHYS_INVENTORY_TAG = 'hld'
       ELEC_INVENTORY_TAG = 'prt'
       ITEM_TAG = 'itm'
+      RELATED_RECORD_TAG = 'rel'
 
       # Subfields for HLD tags
       # Follow MARC 852 spec: https://www.loc.gov/marc/holdings/hd852.html, but names are translated into Alma parlance

data/lib/pennmarc/helpers/access.rb

@@ -5,14 +5,13 @@ module PennMARC
   class Access < Helper
     ONLINE = 'Online'
     AT_THE_LIBRARY = 'At the library'
+    RESOURCE_LINK_BASE_URL = 'hdl.library.upenn.edu'
 
     class << self
       # Based on enhanced metadata fields added by Alma publishing process or API, determine if the record has
       # electronic access or has physical holdings, and is therefore "Online" or "At the library". If a record is "At
       # the library", but has a link to a finding aid in the 856 field (matching certain criteria), also add 'Online' as
       # an access method.
-      # @todo What if none of these criteria match? Should we include "At the library" by default? Records with no value
-      #       in this field would be lost if the user selects a facet value.
       # @param [MARC::Record] record
       # @return [Array]
       def facet(record)
@@ -24,7 +23,7 @@ module PennMARC
         return values if values.size == 2 # return early if all values are already present
 
         # only check if ONLINE isn't already there
-        values << ONLINE if values.exclude?(ONLINE) && finding_aid_linkage?(record)
+        values << ONLINE if values.exclude?(ONLINE) && resource_link?(record)
         values.uniq
       end
 
@@ -44,20 +43,23 @@ module PennMARC
         field.tag.in? [Enriched::Pub::PHYS_INVENTORY_TAG, Enriched::Api::PHYS_INVENTORY_TAG]
       end
 
-      # Check if a record contains an 856 entry for an online finding aid, meeting these criteria:
+      # Check if a record contains an 856 entry with a Penn Handle server link meeting these criteria:
       # 1. Indicator 1 is 4 (HTTP resource)
       # 2. Indicator 2 is NOT 2 (indicating the linkage is to a "related" thing)
       # 3. The URL specified in subfield u (URI) is a Penn Handle link
+      # 4. The subfield z does NOT include the string 'Finding aid'
       # See: https://www.loc.gov/marc/bibliographic/bd856.html
+      # @note Some electronic records do not have Portfolios in Alma, so we rely upon the Resource Link in the 856 to
+      #       get these records included in the Online category.
       # @param [MARC::Record] record
       # @return [Boolean]
-      def finding_aid_linkage?(record)
+      def resource_link?(record)
         record.fields('856').filter_map do |field|
           next if field.indicator2 == '2' || field.indicator1 != '4'
 
           subz = subfield_values(field, 'z')
           subfield_values(field, 'u').filter_map do |value|
-            return true if subz.include?('Finding aid') && value.include?('hdl.library.upenn.edu')
+            return true if subz.exclude?('Finding aid') && value.include?(RESOURCE_LINK_BASE_URL)
           end
         end
         false

data/lib/pennmarc/helpers/creator.rb

@@ -21,6 +21,12 @@ module PennMARC
 
       CONTRIBUTOR_TAGS = %w[700 710].freeze
 
+      FACET_SOURCE_MAP = {
+        100 => 'abcdjq', 110 => 'abcdjq', 111 => 'abcen',
+        700 => 'abcdjq', 710 => 'abcdjq', 711 => 'abcen',
+        800 => 'abcdjq', 810 => 'abcdjq', 811 => 'abcen'
+      }.freeze
+
       # Author/Creator search field. Includes all subfield values (even ǂ0 URIs) from
       # {https://www.oclc.org/bibformats/en/1xx/100.html 100 Main Entry--Personal Name} and
       # {https://www.oclc.org/bibformats/en/1xx/110.html 110 Main Entry--Corporate Name}. Maps any relator codes found
@@ -51,19 +57,33 @@ module PennMARC
       end
 
       # Retrieve creator values for display from fields {https://www.loc.gov/marc/bibliographic/bd100.html 100}
-      # and {https://www.loc.gov/marc/bibliographic/bd110.html 110} and their linked alternates. Appends any encoded
-      # relationships found in $4. If there are no valid encoded relationships, uses the value found in $e.
+      # and {https://www.loc.gov/marc/bibliographic/bd110.html 110} and their linked alternates. First, joins subfields
+      # other than $0, $1, $4, $6, $8, $e, and w. Then, appends any encoded relators found in $4.
+      # If there are no valid encoded relators, uses the value found in $e.
       # @param [MARC::Record] record
       # @return [Array<String>] array of author/creator values for display
       def show(record, relator_map: Mappers.relator)
         fields = record.fields(TAGS)
         fields += record.fields('880').select { |field| subfield_value?(field, '6', /^(#{TAGS.join('|')})/) }
         fields.filter_map { |field|
-          creator = join_subfields(field, &subfield_not_in?(%w[0 1 4 6 8 e w]))
-          append_relator(field: field, joined_subfields: creator, relator_term_sf: 'e', relator_map: relator_map)
+          parse_show_value(field, relator_map: relator_map)
         }.uniq
       end
 
+      # Hash with main creator show values as the fields and the corresponding facet as the values.
+      # Does not include linked 880s.
+      # @param [MARC::Record] record
+      # @param [Hash] relator_map
+      # @return [Hash]
+      def show_facet_map(record, relator_map: Mappers.relator)
+        creators = record.fields(TAGS).filter_map do |field|
+          show = parse_show_value(field, relator_map: relator_map)
+          facet = parse_facet_value(field, FACET_SOURCE_MAP[field.tag.to_i].chars)
+          { show: show, facet: facet }
+        end
+        creators.to_h { |h| [h[:show], h[:facet]] }
+      end
+
       # All author/creator values for display (like #show, but multivalued?) - no 880 linkage
       # Performs additional normalization of author names
       # @note ported from get_author_creator_values (indexed as author_creator_a) - shown on results page
@@ -94,14 +114,9 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>] array of author/creator values for faceting
       def facet(record)
-        source_map = {
-          100 => 'abcdjq', 110 => 'abcdjq', 111 => 'abcen',
-          700 => 'abcdjq', 710 => 'abcdjq', 711 => 'abcen',
-          800 => 'abcdjq', 810 => 'abcdjq', 811 => 'abcen'
-        }
-        source_map.flat_map { |field_num, subfields|
+        FACET_SOURCE_MAP.flat_map { |field_num, subfields|
           record.fields(field_num.to_s).map do |field|
-            trim_punctuation(join_subfields(field, &subfield_in?(subfields.chars)))
+            parse_facet_value(field, subfields.chars)
           end
         }.uniq
       end
@@ -117,7 +132,11 @@ module PennMARC
         }.uniq
       end
 
-      # Conference detailed display, intended for record show page.
+      # Conference detailed display, intended for record show page. Retrieve conference values for record display from
+      # {https://www.loc.gov/marc/bibliographic/bd111.html 111}, {https://www.loc.gov/marc/bibliographic/bd711.html 711}
+      # , and their linked 880s. If there is no $i, we join subfield $i we join subfield values other than
+      # $0, $4, $5, $6, $8, $e, $j, and $w. to create the conference value. We then join the conference subunit value
+      # using subfields $e and $w. We append any relators, preferring those defined in $4 and using $j as a fallback.
       # @note ported from get_conference_values
       # @todo what is ǂi for?
       # @param [MARC::Record] record
@@ -126,15 +145,7 @@ module PennMARC
         conferences = record.fields(%w[111 711]).filter_map do |field|
           next unless field.indicator2.in? ['', ' ']
 
-          conf = if subfield_undefined? field, 'i'
-                   join_subfields field, &subfield_not_in?(%w[0 4 5 6 8 e j w])
-                 else
-                   ''
-                 end
-          sub_unit = join_subfields(field, &subfield_in?(%w[e w]))
-          conf = [conf, sub_unit].compact_blank.join(' ')
-
-          append_relator(field: field, joined_subfields: conf, relator_term_sf: 'j', relator_map: relator_map)
+          parse_conference_detail_show_value(field, relator_map: relator_map)
         end
         conferences += record.fields('880').filter_map do |field|
           next unless subfield_value? field, '6', /^(111|711)/
@@ -150,6 +161,24 @@ module PennMARC
         conferences.uniq
       end
 
+      # Return hash of detailed conference values mapped to their corresponding facets from fields
+      # {https://www.loc.gov/marc/bibliographic/bd111.html 111} and
+      # {https://www.loc.gov/marc/bibliographic/bd711.html 711}. Does not include linked 880s.
+      # @param [MARC::Record] record
+      # @param [Hash] relator_map
+      # @return [Hash]
+      def conference_detail_show_facet_map(record, relator_map: Mappers.relator)
+        conferences = record.fields(%w[111 711]).filter_map do |field|
+          next unless field.indicator2.in? ['', ' ']
+
+          show = parse_conference_detail_show_value(field, relator_map: relator_map)
+          facet = parse_facet_value(field, FACET_SOURCE_MAP[field.tag.to_i].chars)
+          { show: show, facet: facet }
+        end
+
+        conferences.to_h { |conf| [conf[:show], conf[:facet]] }
+      end
+
       # Conference name values for searching
       # @param [MARC::Record] record
       # @return [Array<String>]
@@ -264,6 +293,40 @@ module PennMARC
         before_comma = substring_before(name, ', ')
         "#{after_comma} #{before_comma}".squish
       end
+
+      # Parse creator facet value from given creator field and desired subfields
+      # @param [MARC::Field] field
+      # @param [Array<String>] subfields
+      # @return [String]
+      def parse_facet_value(field, subfields)
+        trim_punctuation(join_subfields(field, &subfield_in?(subfields)))
+      end
+
+      # Parse creator show value from given main creator fields (100/110).
+      # @param [MARC::Field] field
+      # @param [Hash] relator_map
+      # @return [String]
+      def parse_show_value(field, relator_map: Mappers.relator)
+        creator = join_subfields(field, &subfield_not_in?(%w[0 1 4 6 8 e w]))
+        append_relator(field: field, joined_subfields: creator, relator_term_sf: 'e', relator_map: relator_map)
+      end
+
+      # Parse detailed conference show value from given conference field (111/711). If there is no $i, we join subfield
+      # values other than $0, $4, $5, $6, $8, $e, $j, and $w to create conference value. We join subfields $e and $w to
+      # determine the subunit value. We append any relators, preferring those defined in $4 and using $j as a fallback.
+      # @param [MARC::Field] field
+      # @return [String]
+      def parse_conference_detail_show_value(field, relator_map: Mappers.relator)
+        conf = if subfield_undefined? field, 'i'
+                 join_subfields field, &subfield_not_in?(%w[0 4 5 6 8 e j w])
+               else
+                 ''
+               end
+        sub_unit = join_subfields(field, &subfield_in?(%w[e w]))
+        conf = [conf, sub_unit].compact_blank.join(' ')
+
+        append_relator(field: field, joined_subfields: conf, relator_term_sf: 'j', relator_map: relator_map)
+      end
     end
   end
 end

data/lib/pennmarc/helpers/identifier.rb

@@ -159,6 +159,18 @@ module PennMARC
         }.uniq
       end
 
+      # Gets any Host record MMS ID values from an Enriched::Pub::RELATED_RECORD_TAG field added during Alma enrichment.
+      # This aids in our handling of "bound with" records.
+      # @param [MARC::Record] record
+      # @return [Array<String>]
+      def host_record_id(record)
+        record.fields(Enriched::Pub::RELATED_RECORD_TAG).filter_map { |field|
+          next unless subfield_value?(field, 'c', /contains/i)
+
+          subfield_values field, :w
+        }.flatten.uniq
+      end
+
       private
 
       # Determine if subfield 'a' is an OCLC id.

data/lib/pennmarc/helpers/subject.rb

@@ -18,7 +18,7 @@ module PennMARC
       # - 2: MeSH
       # - 4: Source not specified (local?)
       # - 7: Source specified in ǂ2
-      SEARCH_SOURCE_INDICATORS = %w[0 1 2 4 7].freeze
+      VALID_SOURCE_INDICATORS = %w[0 1 2 4 7].freeze
 
       # Tags that serve as sources for Subject facet values
       DISPLAY_TAGS = %w[600 610 611 630 650 651].freeze
@@ -27,7 +27,7 @@ module PennMARC
       LOCAL_TAGS = %w[690 691 697].freeze
 
       # All Subjects for searching. This includes most subfield content from any field contained in {SEARCH_TAGS} or
-      # 69X, including any linked 880 fields. Fields must have an indicator2 value in {SEARCH_SOURCE_INDICATORS}.
+      # 69X, including any linked 880 fields. Fields must have an indicator2 value in {VALID_SOURCE_INDICATORS}.
       # @todo this includes subfields that may not be desired like 1 (uri) and 2 (source code) but this might be OK for
       #       a search (non-display) field?
       # @param [Hash] relator_map
@@ -179,12 +179,18 @@ module PennMARC
         end
       end
 
-      # Is a field intended for display in a general subject field
+      # Is a field intended for display in a general subject field? To be included, the field tag is in either
+      # {DISPLAY_TAGS} or {LOCAL_TAGS}, and has an indicator 2 value that is in {VALID_SOURCE_INDICATORS}. If
+      # indicator 2 is '7' - indicating "source specified", the specified source must be in our approved source code
+      # list.
+      # @see Util.valid_subject_genre_source_code?
       # @param [MARC::DataField] field
       # @return [Boolean] whether a MARC field is intended for display under general "Subjects"
       def subject_general_display_field?(field)
         return false unless field.tag.in?(DISPLAY_TAGS + LOCAL_TAGS) && field.respond_to?(:indicator2)
 
+        return false if field.indicator2.present? && !field.indicator2.in?(VALID_SOURCE_INDICATORS)
+
         return false if field.indicator2 == '7' && !valid_subject_genre_source_code?(field)
 
         true
@@ -278,7 +284,7 @@ module PennMARC
       # @param [MARC::DataField] field
       # @return [Boolean]
       def subject_search_field?(field)
-        return false unless field.respond_to?(:indicator2) && SEARCH_SOURCE_INDICATORS.include?(field.indicator2)
+        return false unless field.respond_to?(:indicator2) && VALID_SOURCE_INDICATORS.include?(field.indicator2)
 
         tag = if field.tag == '880'
                 subfield_values(field, '6').first

data/lib/pennmarc/helpers/title.rb

@@ -3,33 +3,36 @@
 module PennMARC
   # This helper contains logic for parsing out Title and Title-related fields.
   class Title < Helper
-    class << self
-      # We use these fields when retrieving auxiliary titles in the *search_aux methods:
-      # {https://www.loc.gov/marc/bibliographic/bd130.html 130},
-      # {https://www.loc.gov/marc/bibliographic/bd210.html 210},
-      # {https://www.loc.gov/marc/bibliographic/bd245.html 245},
-      # {https://www.loc.gov/marc/bibliographic/bd246.html 246},
-      # {https://www.loc.gov/marc/bibliographic/bd247.html 247},
-      # {https://www.loc.gov/marc/bibliographic/bd440.html 440},
-      # {https://www.loc.gov/marc/bibliographic/bd490.html 490},
-      # {https://www.loc.gov/marc/bibliographic/bd730.html 730},
-      # {https://www.loc.gov/marc/bibliographic/bd740.html 740},
-      # {https://www.loc.gov/marc/bibliographic/bd830.html 830},
-      # {https://www.loc.gov/marc/bibliographic/bd773.html 773},
-      # {https://www.loc.gov/marc/bibliographic/bd774.html 774},
-      # {https://www.loc.gov/marc/bibliographic/bd780.html 780},
-      # {https://www.loc.gov/marc/bibliographic/bd785.html 785},
-      # {https://www.loc.gov/marc/bibliographic/bd700.html 700},
-      # {https://www.loc.gov/marc/bibliographic/bd710.html 710},
-      # {https://www.loc.gov/marc/bibliographic/bd711.html 711},
-      # {https://www.loc.gov/marc/bibliographic/bd505.html 505}
-      AUX_TITLE_TAGS = {
-        main: %w[130 210 240 245 246 247 440 490 730 740 830],
-        related: %w[773 774 780 785],
-        entity: %w[700 710 711],
-        note: %w[505]
-      }.freeze
+    # We use these fields when retrieving auxiliary titles in the *search_aux methods:
+    # {https://www.loc.gov/marc/bibliographic/bd130.html 130},
+    # {https://www.loc.gov/marc/bibliographic/bd210.html 210},
+    # {https://www.loc.gov/marc/bibliographic/bd245.html 245},
+    # {https://www.loc.gov/marc/bibliographic/bd246.html 246},
+    # {https://www.loc.gov/marc/bibliographic/bd247.html 247},
+    # {https://www.loc.gov/marc/bibliographic/bd440.html 440},
+    # {https://www.loc.gov/marc/bibliographic/bd490.html 490},
+    # {https://www.loc.gov/marc/bibliographic/bd730.html 730},
+    # {https://www.loc.gov/marc/bibliographic/bd740.html 740},
+    # {https://www.loc.gov/marc/bibliographic/bd830.html 830},
+    # {https://www.loc.gov/marc/bibliographic/bd773.html 773},
+    # {https://www.loc.gov/marc/bibliographic/bd774.html 774},
+    # {https://www.loc.gov/marc/bibliographic/bd780.html 780},
+    # {https://www.loc.gov/marc/bibliographic/bd785.html 785},
+    # {https://www.loc.gov/marc/bibliographic/bd700.html 700},
+    # {https://www.loc.gov/marc/bibliographic/bd710.html 710},
+    # {https://www.loc.gov/marc/bibliographic/bd711.html 711},
+    # {https://www.loc.gov/marc/bibliographic/bd505.html 505}
+    AUX_TITLE_TAGS = {
+      main: %w[130 210 240 245 246 247 440 490 730 740 830],
+      related: %w[773 774 780 785],
+      entity: %w[700 710 711],
+      note: %w[505]
+    }.freeze
+
+    # This text is used in Alma to indicate a Bib record is a "Host" record for other bibs (bound-withs)
+    HOST_BIB_TITLE = 'Host bibliographic record for boundwith'
 
+    class << self
       # Main Title Search field. Takes from {https://www.loc.gov/marc/bibliographic/bd245.html 245} and linked 880.
       # @note Ported from get_title_1_search_values.
       # @param [MARC::Record] record
@@ -217,6 +220,16 @@ module PennMARC
         }.uniq
       end
 
+      # Determine if the record is a "Host" bibliographic record for other bib records ("bound-withs")
+      # @param [MARC::Record] record
+      # @return [Boolean]
+      def host_bib_record?(record)
+        record.fields('245').any? do |f|
+          title = join_subfields(f, &subfield_in?(%w[a]))
+          title.include?(HOST_BIB_TITLE)
+        end
+      end
+
       private
 
       # Create prefix/filing hash for representing a title value with filing characters removed, with special

data/lib/pennmarc/mappings/locations.yml

@@ -547,6 +547,12 @@ dent:
     - Health Sciences Libraries
     - Levy Dental Medicine Library
   display: Levy Dental Medicine Library - Stacks
+dentpcare:
+  specific_location: Levy Dental Medicine Library - PCare
+  library:
+    - Health Sciences Libraries
+    - Levy Dental Medicine Library
+  display: Levy Dental Medicine Library - PCare
 dentrare:
   specific_location: Levy Dental Medicine Library - Rare Books
   library:

data/lib/pennmarc/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PennMARC
-  VERSION = '1.0.23'
+  VERSION = '1.0.25'
 end

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

@@ -69,6 +69,17 @@ describe 'PennMARC::Access' do
           { z: 'Finding aid', u: 'http://hdl.library.upenn.edu/1017/d/pacscl/UPENN_RBML_MsColl200' }
         end
 
+        it 'does not include online access' do
+          expect(helper.facet(record)).to contain_exactly(PennMARC::Access::AT_THE_LIBRARY)
+        end
+      end
+
+      context 'with an 856 describing a resource link' do
+        let(:indicators) { { indicator1: '4', indicator2: '1' } }
+        let(:location_and_access_subfields) do
+          { z: 'Connect to resource', u: 'http://hdl.library.upenn.edu/1234' }
+        end
+
         it 'includes online access' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::ONLINE, PennMARC::Access::AT_THE_LIBRARY)
         end

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

@@ -95,6 +95,23 @@ describe 'PennMARC::Creator' do
     end
   end
 
+  describe '.show_facet_map' do
+    let(:record) do
+      marc_record fields: [
+        marc_field(tag: '100', subfields: { a: 'Surname, Name', '0': 'http://cool.uri/12345', d: '1900-2000',
+                                            e: 'author.', '4': 'http://cool.uri/vocabulary/relators/aut' }),
+        marc_field(tag: '110', subfields: { a: 'Group of People', b: 'Annual Meeting', '4': 'aut' }),
+        marc_field(tag: '880', subfields: { a: 'Ignore', '6': '100' })
+      ]
+    end
+
+    it 'returns expected hash' do
+      values = helper.show_facet_map(record, relator_map: mapping)
+      expect(values).to eq({ 'Surname, Name 1900-2000, author.' => 'Surname, Name 1900-2000',
+                             'Group of People Annual Meeting, Author.' => 'Group of People Annual Meeting' })
+    end
+  end
+
   describe '.show_aux' do
     let(:record) { marc_record fields: fields }
 
@@ -257,6 +274,22 @@ describe 'PennMARC::Creator' do
     end
   end
 
+  describe '.conference_detail_show_facet_map' do
+    let(:record) do
+      marc_record fields: [
+        marc_field(tag: '111', subfields: { a: 'Council of Trent', d: '(1545-1563 :', c: 'Trento, Italy)' }),
+        marc_field(tag: '711', subfields: { a: 'Code4Lib', n: '(18th :', d: '2024 :', c: 'Ann Arbor, MI)' }),
+        marc_field(tag: '880', subfields: { a: 'Alt Ignore', '6': '111' })
+      ]
+    end
+
+    it 'returns the expected hash' do
+      value = helper.conference_detail_show_facet_map(record)
+      expect(value).to eq({ 'Council of Trent (1545-1563 : Trento, Italy)' => 'Council of Trent Trento, Italy)',
+                            'Code4Lib (18th : 2024 : Ann Arbor, MI)' => 'Code4Lib (18th : Ann Arbor, MI)' })
+    end
+  end
+
   describe '.conference_search' do
     let(:record) do
       marc_record fields: [

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

@@ -161,4 +161,18 @@ describe 'PennMARC::Identifier' do
                                                          '10.1038/sdata.2016.18', '10.18574/9781479842865')
     end
   end
+
+  describe '.host_record_id' do
+    let(:record) do
+      marc_record fields: [
+        marc_field(tag: PennMARC::Enriched::Pub::RELATED_RECORD_TAG, subfields: { w: '123456789', c: 'Contains',
+                                                                                  a: 'Title' }),
+        marc_field(tag: PennMARC::Enriched::Pub::RELATED_RECORD_TAG, subfields: { w: '666666666', c: 'Contained In' })
+      ]
+    end
+
+    it 'returns only the desired host record MMS ID values' do
+      expect(helper.host_record_id(record)).to contain_exactly '123456789'
+    end
+  end
 end

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

@@ -288,6 +288,22 @@ describe 'PennMARC::Subject' do
                                    'Early works to 1950.')
       end
     end
+
+    context 'with a mix of allowed and disallowed heading sources' do
+      let(:fields) do
+        [marc_field(tag: '650', indicator2: '7', subfields: {
+                      '2': 'fast',
+                      a: 'Philosophy in motion pictures.',
+                      '0': 'http://id.loc.gov/authorities/subjects/sh92003501'
+                    }),
+         marc_field(tag: '650', indicator2: '7', subfields: { '2': 'gnd', a: 'Filmästhetik.' }),
+         marc_field(tag: '650', indicator2: '6', subfields: { a: 'Cinéma et arts.' })]
+      end
+
+      it 'includes only permitted headings from approved ontologies' do
+        expect(values).to contain_exactly 'Philosophy in motion pictures.'
+      end
+    end
   end
 
   describe '.childrens_show' do

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

@@ -249,4 +249,24 @@ describe 'PennMARC::Title' do
       expect(values).not_to include 'Linkage', '247'
     end
   end
+
+  describe '.host_bib_record?' do
+    let(:record) { marc_record fields: [marc_field(tag: '245', subfields: subfields)] }
+
+    context 'with a host record' do
+      let(:subfields) { { a: "#{PennMARC::Title::HOST_BIB_TITLE} for 123456789" } }
+
+      it 'returns true' do
+        expect(helper.host_bib_record?(record)).to be true
+      end
+    end
+
+    context 'with a non-host record' do
+      let(:subfields) { { a: 'Regular record' } }
+
+      it 'returns false' do
+        expect(helper.host_bib_record?(record)).to be false
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pennmarc
 version: !ruby/object:Gem::Version
-  version: 1.0.23
+  version: 1.0.25
 platform: ruby
 authors:
 - Mike Kanning
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-06-04 00:00:00.000000000 Z
+date: 2024-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport