mods_display 0.7.0 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b98625a861414ce3c06cd365c9850454a50ccd83719124820a49f4fcb7ff107d
-  data.tar.gz: e0c7ded47f3f3f38b821294f4bb9252e0fd7ffdb3fae837dc4da340bc225ab44
+  metadata.gz: 586a8299be19e795bc90b434d9902084ca8fa10bb8cbe9690fc108471e1f5495
+  data.tar.gz: 00574a48dc3c3ba050529bf2d3fcd5e38df5de4c0e5ed4023ad07904dea29c99
 SHA512:
-  metadata.gz: efd399144bc193f4db0bfd4b3185346dce1e8cc523fad133af38f61a19d2f41b42e8d05a4fad6ea77dab744eeba2b7727b2dfe911f34c49844ca94fd433320fa
-  data.tar.gz: 121a7fa3ba9a0af4887fbc5fac992c95be7c19b888cb632fede718bb2796b1ceb783ce98f16e06de415e149b344584f6c4046dfa2f25c1bcc01823ade826ee46
+  metadata.gz: 9430fa89a1efb94a17ec8c83f5fc96e55f96221f962480ff6a71a9283548d385e1afabe1c8653215155a8d7804648075aae93d217ff2d25ec29357c5b2e64544
+  data.tar.gz: 0d7b2e673cae7f336e65824c528b78abd59e8b730430570faa98910cf715e875a707dde7787a48cbf158f15bc596b057181c443d253387136604b307036812ea

data/.github/workflows/ruby.yml

@@ -0,0 +1,24 @@
+name: CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: [2.6, 2.7, 3.0]
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+    - name: Install dependencies
+      run: bundle install
+    - name: Run tests
+      run: bundle exec rake

data/lib/mods_display.rb

@@ -49,3 +49,14 @@ require 'i18n/backend/fallbacks'
 I18n::Backend::Simple.send(:include, I18n::Backend::Fallbacks)
 I18n.load_path += Dir["#{File.expand_path('../..', __FILE__)}/config/locales/*.yml"]
 I18n.backend.load_translations
+
+# load Rails/Railtie
+begin
+  require 'rails'
+rescue LoadError
+  #do nothing
+end
+
+if defined? ::Rails::Railtie
+  require 'mods_display/railtie'
+end

data/lib/mods_display/fields/access_condition.rb

@@ -77,6 +77,9 @@ module ModsDisplay
 
     def license_statement(element)
       matches = element.text.match(/^(?<code>.*) (?<type>.*):(?<description>.*)$/)
+
+      return "<div>#{element.text}</div>" unless matches
+
       code = matches[:code].downcase
       type = matches[:type].downcase
       description = license_description(code, type) || matches[:description]

data/lib/mods_display/fields/imprint.rb

@@ -68,6 +68,8 @@ module ModsDisplay
       date_fields.map do |date_field|
         if date_is_w3cdtf?(date_field)
           process_w3cdtf_date(date_field)
+        elsif date_is_iso8601?(date_field)
+          process_iso8601_date(date_field)
         else
           date_field
         end
@@ -158,6 +160,10 @@ module ModsDisplay
       field_is_encoded?(date_field, 'w3cdtf')
     end
 
+    def date_is_iso8601?(date_field)
+      field_is_encoded?(date_field, 'iso8601')
+    end
+
     def process_w3cdtf_date(date_field)
       date_field = date_field.clone
       date_field.content = begin
@@ -174,6 +180,16 @@ module ModsDisplay
       date_field
     end
 
+    def process_iso8601_date(date_field)
+      date_field = date_field.clone
+      date_field.content = begin
+        Date.iso8601(date_field.text).strftime(@config.full_date_format)
+      rescue
+        date_field.content
+      end
+      date_field
+    end
+
     def dedup_dates(date_fields)
       date_text = date_fields.map { |d| normalize_date(d.text) }
       if date_text != date_text.uniq

data/lib/mods_display/fields/location.rb

@@ -24,7 +24,7 @@ module ModsDisplay
     private
 
     def location_field_keys
-      [:physicalLocation, :url, :shelfLocation, :holdingSimple, :holdingExternal]
+      [:physicalLocation, :url, :shelfLocator, :holdingSimple, :holdingExternal]
     end
 
     def location_label(element)

data/lib/mods_display/fields/title.rb

@@ -4,31 +4,10 @@ module ModsDisplay
       return_values = []
       if @values
         @values.each do |value|
-          if displayForm(value)
-            return_values << ModsDisplay::Values.new(
-              label: displayLabel(value) || title_label(value),
-              values: [displayForm(value)]
-            )
-          else
-            nonSort = nil
-            title = nil
-            subTitle = nil
-            nonSort = value.nonSort.text.strip unless value.nonSort.text.strip.empty?
-            title = value.title.text.strip unless value.title.text.strip.empty?
-            subTitle = value.subTitle.text unless value.subTitle.text.strip.empty?
-            preSubTitle = [nonSort, title].compact.join(' ')
-            preSubTitle = nil if preSubTitle.strip.empty?
-            preParts = compact_and_join_with_delimiter([preSubTitle, subTitle], ' : ')
-            preParts = nil if preParts.strip.empty?
-            parts = value.children.select do |child|
-              %w(partName partNumber).include?(child.name)
-            end.map(&:text).compact.join(parts_delimiter(value))
-            parts = nil if parts.strip.empty?
-            return_values << ModsDisplay::Values.new(
-              label: displayLabel(value) || title_label(value),
-              values: [compact_and_join_with_delimiter([preParts, parts], '. ')]
-            )
-          end
+          return_values << ModsDisplay::Values.new(
+            label: displayLabel(value) || title_label(value),
+            values: [assemble_title(value)]
+          )
         end
       end
       collapse_fields(return_values)
@@ -36,14 +15,42 @@ module ModsDisplay
 
     private
 
-    def parts_delimiter(element)
-      children = element.children.to_a
-      # index will retun nil which is not comparable so we call 100
-      # if the element isn't present (thus meaning it's at the end of the list)
-      if (children.index { |c| c.name == 'partNumber' } || 100) < (children.index { |c| c.name == 'partName' } || 100)
-        return ', '
+    def assemble_title(element)
+      return displayForm(element) if displayForm(element)
+
+      title = ''
+      previous_element = nil
+
+      element.children.select { |value| title_parts.include? value.name }.each do |value|
+        str = value.text.strip
+        next if str.empty?
+
+        delimiter = case
+        when title.empty?, title.end_with?(' ')
+          nil
+        when title.end_with?('.', ',', ':', ';')
+          ' '
+        when value.name == 'subTitle'
+          ' : '
+        when value.name == 'partName' && previous_element.name == 'partNumber'
+          ', '
+        when value.name == 'partNumber', value.name == 'partName'
+          '. '
+        else
+          ' '
+        end
+
+        title += delimiter if delimiter
+        title += str
+
+        previous_element = value
       end
-      '. '
+
+      title
+    end
+
+    def title_parts
+      %w[nonSort title subTitle partName partNumber]
     end
 
     def title_label(element)

data/lib/mods_display/helpers/record_helper.rb

@@ -0,0 +1,131 @@
+# frozen_string_literal: true
+
+module ModsDisplay
+  module Helpers
+    module RecordHelper
+      def display_content_field(field)
+        return unless field.respond_to?(:label, :values) && field.values.any?(&:present?)
+
+        display_content_label(field.label) + display_content_values(field.values)
+      end
+
+      def display_content_label(label)
+        content_tag :dt, label
+      end
+
+      def display_content_values(values)
+        values.map do |value|
+          content_tag :dd, value
+        end.join('').html_safe
+      end
+
+      def mods_display_label(label)
+        content_tag(:dt, label.delete(':')) + "\n".html_safe
+      end
+
+      def mods_display_content(values, delimiter = nil)
+        if delimiter
+          content_tag(:dd, values.map do |value|
+            link_urls_and_email(value) if value.present?
+          end.compact.join(delimiter).html_safe)
+        else
+          Array[values].flatten.map do |value|
+            content_tag(:dd, link_urls_and_email(value.to_s).html_safe) if value.present?
+          end.join.html_safe
+        end
+      end
+
+      def mods_record_field(field, delimiter = nil)
+        return unless field.respond_to?(:label, :values) && field.values.any?(&:present?)
+
+        mods_display_label(field.label) + mods_display_content(field.values, delimiter)
+      end
+
+      def mods_name_field(field, &block)
+        return unless field.respond_to?(:label, :values) && field.values.any?(&:present?)
+
+        mods_display_label(field.label) + mods_display_name(field.values, &block)
+      end
+
+      def mods_display_name(names, &block)
+        names.map do |name|
+          content_tag(:dd) do
+            block_given? ? yield(name.name) : name.name
+          end
+        end.join.html_safe
+      end
+
+      # We need this to remove the ending ":" from the role labels only in data from
+      # mods_display
+      def sanitize_mods_name_label(label)
+        label.sub(/:$/, '')
+      end
+
+      def mods_subject_field(subject, &block)
+        return unless subject.values.any?(&:present?)
+
+        fields = subject.values.map do |subject_line|
+          content_tag :dd, safe_join(link_mods_subjects(subject_line, &block), ' > ')
+        end
+
+        (mods_display_label(subject.label) + safe_join(fields))
+      end
+
+      def mods_genre_field(genre, &block)
+        return unless genre.values.any?(&:present?)
+
+        fields = genre.values.map do |genre_line|
+          content_tag :dd, link_mods_genres(genre_line, &block)
+        end
+
+        mods_display_label(genre.label) + safe_join(fields)
+      end
+
+      def link_mods_genres(genre, &block)
+        link_buffer = []
+        link_to_mods_subject(genre, link_buffer, &block)
+      end
+
+      def link_mods_subjects(subjects, &block)
+        link_buffer = []
+        linked_subjects = []
+        subjects.each do |subject|
+          if subject.present?
+            linked_subjects << link_to_mods_subject(subject, link_buffer, &block)
+          end
+        end
+        linked_subjects
+      end
+
+      def link_to_mods_subject(subject, buffer, &block)
+        subject_text = subject.respond_to?(:name) ? subject.name : subject
+        link = block_given? ? yield(subject_text, buffer) : subject_text
+        buffer << subject_text.strip
+        link << " (#{subject.roles.join(', ')})" if subject.respond_to?(:roles) && subject.roles.present?
+        link
+      end
+
+      # rubocop:disable Layout/LineLength
+      def link_urls_and_email(val)
+        val = val.dup
+        # http://daringfireball.net/2010/07/improved_regex_for_matching_urls
+        url = %r{(?i)\b(?:https?://|www\d{0,3}[.]|[a-z0-9.\-]+[.][a-z]{2,4}/)(?:[^\s()<>]+|\([^\s()<>]+|\([^\s()<>]+\)*\))+(?:\([^\s()<>]+|\([^\s()<>]+\)*\)|[^\s`!()\[\]{};:'".,<>?«»“”‘’])}i
+        # http://www.regular-expressions.info/email.html
+        email = %r{[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+(?:[A-Z]{2}|com|org|net|edu|gov|mil|biz|info|mobi|name|aero|asia|jobs|museum)\b}i
+        matches = [val.scan(url), val.scan(email)].flatten.uniq
+        unless val =~ /<a/ # we'll assume that linking has alraedy occured and we don't want to double link
+          matches.each do |match|
+            if match =~ email
+              val.gsub!(match, "<a href='mailto:#{match}'>#{match}</a>")
+            else
+              val.gsub!(match, "<a href='#{match}'>#{match}</a>")
+            end
+          end
+        end
+        val
+      end
+      # rubocop:enable Layout/LineLength
+
+    end
+  end
+end

data/lib/mods_display/html.rb

@@ -41,7 +41,9 @@ module ModsDisplay
       if to_s.respond_to?(method_name)
         to_html.send(method_name, *args, &block)
       elsif method_name == :subTitle || mods_display_fields.include?(method_name)
-        mods_field(@xml, method_name).fields
+        field = mods_field(@xml, method_name)
+        return field if (args.dig(0, :raw))
+        field.fields
       else
         super
       end

data/lib/mods_display/railtie.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+module ModsDisplay
+  class Railtie < ::Rails::Railtie
+    ActiveSupport.on_load :action_view do
+      require 'mods_display/helpers/record_helper'
+      ::ActionView::Base.send :include, ModsDisplay::Helpers::RecordHelper
+    end
+  end
+end

data/lib/mods_display/version.rb

@@ -1,3 +1,3 @@
 module ModsDisplay
-  VERSION = '0.7.0'
+  VERSION = '0.10.0'
 end

data/mods_display.gemspec

@@ -22,7 +22,9 @@ Gem::Specification.new do |gem|
 
   gem.add_development_dependency 'rake'
   gem.add_development_dependency 'rspec', '~> 3.0'
+  gem.add_development_dependency 'rspec-rails'
   gem.add_development_dependency 'rubocop'
   gem.add_development_dependency 'capybara'
   gem.add_development_dependency 'byebug'
+  gem.add_development_dependency 'rails', '~> 6.0'
 end

data/spec/fake_app.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require 'action_controller/railtie'
+require 'action_view/railtie'
+
+class ModsDisplayTestApp < Rails::Application
+end
+
+Rails.application.routes.draw do
+  resources 'searches', only: :index
+end
+
+class ApplicationController < ActionController::Base; end
+class SearchesController < ApplicationController
+  def index; end
+end
+
+Object.const_set(:ApplicationHelper, Module.new)

data/spec/fields/access_condition_spec.rb

@@ -25,6 +25,7 @@ describe ModsDisplay::AccessCondition do
     @cc_license_note = Stanford::Mods::Record.new.from_str(cc_license_fixture, false).accessCondition
     @odc_license_note = Stanford::Mods::Record.new.from_str(odc_license_fixture, false).accessCondition
     @no_link_license_note = Stanford::Mods::Record.new.from_str(no_license_fixture, false).accessCondition
+    @garbage_license_fixture = Stanford::Mods::Record.new.from_str(garbage_license_fixture, false).accessCondition
   end
   describe 'labels' do
     it 'should normalize types and assign proper labels' do
@@ -54,7 +55,7 @@ describe ModsDisplay::AccessCondition do
         expect(fields.first.values.length).to eq(1)
         expect(fields.first.values.first).to match(%r{^<div class='unknown-something'>.*</div>$})
       end
-      it 'should itentify and link CreativeCommons licenses properly' do
+      it 'should identify and link CreativeCommons licenses properly' do
         fields = mods_display_access_condition(@cc_license_note).fields
         expect(fields.length).to eq(1)
         expect(fields.first.values.length).to eq(1)
@@ -63,7 +64,7 @@ describe ModsDisplay::AccessCondition do
           'This work is licensed under a Creative Commons Attribution-Share Alike 3.0 Unported License'
         )
       end
-      it 'should itentify and link OpenDataCommons licenses properly' do
+      it 'should identify and link OpenDataCommons licenses properly' do
         fields = mods_display_access_condition(@odc_license_note).fields
         expect(fields.length).to eq(1)
         expect(fields.first.values.length).to eq(1)
@@ -82,6 +83,16 @@ describe ModsDisplay::AccessCondition do
         )
         expect(fields.first.values.first).not_to include('<a.*>')
       end
+
+      it 'returns the license text if it does not look like our expected format' do
+        fields = mods_display_access_condition(@garbage_license_fixture).fields
+        expect(fields.length).to eq(1)
+        expect(fields.first.values.length).to eq(1)
+        expect(fields.first.values.first).to include(
+          'Unknown garbage that does not look like a license'
+        )
+        expect(fields.first.values.first).not_to include('<a.*>')
+      end
     end
   end
   describe 'to_html' do

data/spec/fields/imprint_spec.rb

@@ -43,6 +43,7 @@ describe ModsDisplay::Imprint do
     @imprint_date_range = Stanford::Mods::Record.new.from_str(imprint_date_range, false).origin_info
     @encoded_place = Stanford::Mods::Record.new.from_str(encoded_place, false).origin_info
     @encoded_dates = Stanford::Mods::Record.new.from_str(encoded_dates, false).origin_info
+    @iso8601_encoded_dates = Stanford::Mods::Record.new.from_str(iso8601_encoded_dates, false).origin_info
     @bad_dates = Stanford::Mods::Record.new.from_str(bad_dates, false).origin_info
     @invalid_dates = Stanford::Mods::Record.new.from_str(invalid_dates, false).origin_info
     @punctuation_imprint = Stanford::Mods::Record.new.from_str(punctuation_imprint_fixture, false).origin_info
@@ -209,6 +210,23 @@ describe ModsDisplay::Imprint do
           end.values).to eq(['July (2013)'])
         end
       end
+
+      describe 'iso8601' do
+        it 'handles full dates properly' do
+          fields = mods_display_imprint(@iso8601_encoded_dates).fields
+          expect(fields.length).to eq(2)
+          expect(fields.find do |field|
+            field.label == 'Date created:'
+          end.values).to eq(['November 14, 2013'])
+        end
+        it "should not try to handle dates we can't parse" do
+          fields = mods_display_imprint(@iso8601_encoded_dates).fields
+          expect(fields.length).to eq(2)
+          expect(fields.find do |field|
+            field.label == 'Date modified:'
+          end.values).to eq(['Jul. 22, 2013'])
+        end
+      end
     end
     describe 'bad dates' do
       it 'should ignore date values' do

data/spec/fields/location_spec.rb

@@ -5,7 +5,7 @@ describe ModsDisplay::Location do
     <<-XML
       <mods>
         <location>
-          <shelfLocation>On Shelf A</shelfLocation>
+          <shelfLocator>On Shelf A</shelfLocator>
         </location>
       </mods>
     XML

data/spec/fields/title_spec.rb

@@ -15,6 +15,7 @@ describe ModsDisplay::Title do
     @multi_label = Stanford::Mods::Record.new.from_str(multi_label_fixture, false).title_info
     @alt_title = Stanford::Mods::Record.new.from_str(alt_title_fixture, false).title_info
     @title_punctuation = Stanford::Mods::Record.new.from_str(title_puncutation_fixture, false).title_info
+    @ordered_title_fixture = Stanford::Mods::Record.new.from_str(ordered_title_fixture, false).title_info
   end
   describe 'labels' do
     it 'should return a default label of Title if nothing else is available' do
@@ -70,5 +71,12 @@ describe ModsDisplay::Title do
       expect(values.first).not_to include '..'
       expect(values.first).to eq 'A title that ends in punctuation. 2015'
     end
+
+    it 'combines the title parts in the order from the record' do
+      values = mods_display_title(@ordered_title_fixture).fields.first.values
+      expect(values.length).to eq 1
+
+      expect(values.first).to eq 'The medium term expenditure framework (MTEF) for ... and the annual estimates for ... 016, Ministry of Tourism : expenditure to be met out of moneys granted and drawn from the consolidated fund, central government budget'
+    end
   end
 end

data/spec/fixtures/access_condition_fixtures.rb

@@ -47,4 +47,12 @@ module AccessConditionFixtures
       </mods>
     XML
   end
+
+  def garbage_license_fixture
+    <<-XML
+      <mods>
+        <accessCondition type='license'>Unknown garbage that does not look like a license</accessCondition>
+      </mods>
+    XML
+  end
 end

data/spec/fixtures/imprint_fixtures.rb

@@ -280,6 +280,17 @@ module ImprintFixtures
     MODS
   end
 
+  def iso8601_encoded_dates
+    <<-MODS
+      <mods>
+        <originInfo>
+          <dateCreated encoding="iso8601">20131114161429</dateCreated>
+          <dateModified encoding="iso8601">Jul. 22, 2013</dateModified>
+        </originInfo>
+      </mods>
+    MODS
+  end
+
   def bad_dates
     <<-MODS
       <mods>

data/spec/fixtures/title_fixtures.rb

@@ -98,4 +98,18 @@ module TitleFixtures
       </mods>
     XML
   end
+
+  def ordered_title_fixture
+    <<-XML
+    <mods>
+      <titleInfo>
+        <nonSort>The</nonSort>
+        <title>medium term expenditure framework (MTEF) for ... and the annual estimates for ...</title>
+        <partNumber>016</partNumber>
+        <partName>Ministry of Tourism</partName>
+        <subTitle>expenditure to be met out of moneys granted and drawn from the consolidated fund, central government budget</subTitle>
+      </titleInfo>
+    </mods>
+    XML
+  end
 end

data/spec/helpers/record_helper_spec.rb

@@ -0,0 +1,241 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'byebug'
+
+describe ModsDisplay::Helpers::RecordHelper, type: :helper do
+  let(:empty_field) { OpenStruct.new(label: 'test', values: ['']) }
+
+  describe 'display_content_field' do
+    let(:values) { ['guitar (1)', 'solo cowbell, trombone (2)'] }
+    let(:content) { OpenStruct.new(label: 'Instrumentation', values: values) }
+
+    it 'should return dt with label and dd with values' do
+      expect(helper.display_content_field(content)).to have_css('dt', text: 'Instrumentation')
+      expect(helper.display_content_field(content)).to have_css('dd', count: 2)
+    end
+  end
+
+  describe 'display_content_label' do
+    it 'should return correct dt' do
+      expect(helper.display_content_label('test')).to have_css('dt', text: 'test')
+    end
+  end
+
+  describe 'display_content_values' do
+    let(:values) { ['guitar (1)', 'solo cowbell, trombone (2)'] }
+
+    it 'should return dds of values' do
+      expect(helper.display_content_values(values)).to have_css('dd', count: 2)
+      expect(helper.display_content_values(values)).to have_css('dd', text: 'guitar (1)')
+      expect(helper.display_content_values(values)).to have_css('dd', text: 'solo cowbell, trombone (2)')
+    end
+  end
+
+  describe 'mods_display_label' do
+    it 'should return correct label' do
+      expect(helper.mods_display_label('test:')).not_to have_content ':'
+      expect(helper.mods_display_label('test:')).to have_css('dt', text: 'test')
+    end
+  end
+
+  describe 'mods_display_content' do
+    it 'should return correct content' do
+      expect(helper.mods_display_content('hello, there')).to have_css('dd', text: 'hello, there')
+    end
+    it 'should return multiple dd elements when a multi-element array is passed' do
+      expect(helper.mods_display_content(%w(hello there))).to have_css('dd', count: 2)
+    end
+    it 'should handle nil values correctly' do
+      expect(helper.mods_display_content(['something', nil])).to have_css('dd', count: 1)
+    end
+  end
+
+  describe 'mods_record_field' do
+    let(:mods_field) { OpenStruct.new(label: 'test', values: ['hello, there']) }
+    let(:url_field) { OpenStruct.new(label: 'test', values: ['https://library.stanford.edu']) }
+    let(:multi_values) { double(label: 'test', values: %w(123 321)) }
+
+    it 'should return correct content' do
+      expect(helper.mods_record_field(mods_field)).to have_css('dt', text: 'test')
+      expect(helper.mods_record_field(mods_field)).to have_css('dd', text: 'hello, there')
+    end
+    it 'should link fields with URLs' do
+      expect(mods_record_field(url_field)).to have_css("a[href='https://library.stanford.edu']", text: 'https://library.stanford.edu')
+    end
+    it 'should not print empty labels' do
+      expect(helper.mods_record_field(empty_field)).not_to be_present
+    end
+    it 'should join values with a <dd> by default' do
+      expect(helper.mods_record_field(multi_values)).to have_css('dd', count: 2)
+    end
+    it 'should join values with a supplied delimiter' do
+      expect(helper.mods_record_field(multi_values, 'DELIM')).to have_css('dd', count: 1)
+      expect(helper.mods_record_field(multi_values, 'DELIM')).to have_css('dd', text: '123DELIM321')
+    end
+  end
+
+  describe 'names' do
+    let(:name_field) do
+      OpenStruct.new(
+        label: 'Contributor',
+        values: [
+          OpenStruct.new(name: 'Winefrey, Oprah', roles: %w(Host Producer)),
+          OpenStruct.new(name: 'Kittenz, Emergency')
+        ]
+      )
+    end
+
+    describe '#mods_name_field' do
+      it 'should join the label and values' do
+        name = mods_name_field(name_field) do |name|
+          link_to(name, searches_path(q: "\"#{name}\"", search_field: 'search_author'))
+        end
+        expect(name).to match /<dt>Contributor<\/dt>/
+        expect(name).to match /<dd><a href.*<\/dd>/
+      end
+      it 'should not print empty labels' do
+        expect(mods_name_field(empty_field)).not_to be_present
+      end
+    end
+
+    describe '#mods_display_name' do
+      let(:name) do
+        mods_display_name(name_field.values) do |name|
+          link_to(name, searches_path(q: "\"#{name}\"", search_field: 'search_author'))
+        end
+      end
+
+      it 'should link to the name' do
+        expect(name).to match /<a href=.*%22Winefrey%2C\+Oprah%22.*>Winefrey, Oprah<\/a>/
+        expect(name).to match /<a href=.*%22Kittenz%2C\+Emergency%22.*>Kittenz, Emergency<\/a>/
+      end
+      it 'should link to an author search' do
+        expect(name).to match /<a href.*search_field=search_author.*>/
+      end
+    end
+
+    describe '#sanitize_mods_name_label' do
+      it 'removes a ":" at the end of label if present' do
+        expect(sanitize_mods_name_label('Test String:')).to eq 'Test String'
+        expect(sanitize_mods_name_label('Test String')).to eq 'Test String'
+      end
+    end
+  end
+
+  describe 'subjects' do
+    let(:subjects) { [OpenStruct.new(label: 'Subjects', values: [%w(Subject1a Subject1b), %w(Subject2a Subject2b Subject2c)])] }
+    let(:name_subjects) { [OpenStruct.new(label: 'Subjects', values: [OpenStruct.new(name: 'Person Name', roles: %w(Role1 Role2))])] }
+    let(:genres) { [OpenStruct.new(label: 'Genres', values: %w(Genre1 Genre2 Genre3))] }
+
+    describe '#mods_subject_field' do
+      let(:subject) do
+        mods_subject_field(subjects.first) do |subject_text|
+          link_to(
+            subject_text,
+            searches_path(
+              q: "\"#{[[], subject_text.strip].flatten.join(' ')}\"",
+              search_field: 'subject_terms'
+            )
+          )
+        end
+      end
+      it 'should join the subject fields in a dd' do
+        expect(subject).to match /<dd><a href=*.*\">Subject1a*.*Subject1b<\/a><\/dd><dd><a/
+      end
+      it "should join the individual subjects with a '>'" do
+        expect(subject).to match /Subject2b<\/a> &gt; <a href/
+      end
+      it 'should not print empty labels' do
+        expect(mods_subject_field(empty_field)).not_to be_present
+      end
+    end
+
+    describe '#mods_genre_field' do
+      it 'should join the genre fields with a dd' do
+        expect(mods_genre_field(genres.first) do |text|
+          link_to(text, searches_path(q: text))
+        end).to match /<dd><a href=*.*>Genre1*.*<\/a><\/dd><dd><a*.*Genre2<\/a><\/dd>/
+      end
+      it 'should not print empty labels' do
+        expect(mods_genre_field(empty_field)).not_to be_present
+      end
+    end
+
+    describe '#link_mods_subjects' do
+      let(:linked_subjects) do
+        link_mods_subjects(subjects.first.values.last) do |subject_text, buffer|
+          link_to(
+            subject_text,
+            searches_path(
+              q: "\"#{[buffer, subject_text.strip].flatten.join(' ')}\"",
+              search_field: 'subject_terms'
+            )
+          )
+        end
+      end
+
+      it 'should return all subjects' do
+        expect(linked_subjects.length).to eq 3
+      end
+      it 'should link to the subject hierarchically' do
+        expect(linked_subjects[0]).to match /^<a href=.*q=%22Subject2a%22.*>Subject2a<\/a>$/
+        expect(linked_subjects[1]).to match /^<a href=.*q=%22Subject2a\+Subject2b%22.*>Subject2b<\/a>$/
+        expect(linked_subjects[2]).to match /^<a href=.*q=%22Subject2a\+Subject2b\+Subject2c%22.*>Subject2c<\/a>$/
+      end
+      it 'should link to subject terms search field' do
+        linked_subjects.each do |subject|
+          expect(subject).to match /search_field=subject_terms/
+        end
+      end
+    end
+
+    describe '#link_mods_genres' do
+      let(:linked_genres) do
+        link_mods_genres(genres.first.values.last) do |text|
+          link_to(
+            text,
+            searches_path(
+              q: "\"#{[[], text.strip].flatten.join(' ')}\"",
+              search_field: 'subject_terms'
+            )
+          )
+        end
+      end
+
+      it 'should return correct link' do
+        expect(linked_genres).to match /<a href=*.*Genre3*.*<\/a>/
+      end
+      it 'should link to subject terms search field' do
+        expect(linked_genres).to match /search_field=subject_terms/
+      end
+    end
+
+    describe '#link_to_mods_subject' do
+      it 'should handle subjects that behave like names' do
+        name_subject = link_to_mods_subject(name_subjects.first.values.first, []) do |subject_text|
+          link_to(
+            subject_text,
+            searches_path(
+              q: "\"#{[[], subject_text.strip].flatten.join(' ')}\"",
+              search_field: 'subject_terms'
+            )
+          )
+        end
+        expect(name_subject).to match /<a href=.*%22Person\+Name%22.*>Person Name<\/a> \(Role1, Role2\)/
+      end
+    end
+  end
+
+  describe '#link_urls_and_email' do
+    let(:url) { 'This is a field that contains an https://library.stanford.edu URL' }
+    let(:email) { 'This is a field that contains an email@email.com address' }
+
+    it 'should link URLs' do
+      expect(link_urls_and_email(url)).to eq "This is a field that contains an <a href='https://library.stanford.edu'>https://library.stanford.edu</a> URL"
+    end
+    it 'should link email addresses' do
+      expect(link_urls_and_email(email)).to eq "This is a field that contains an <a href='mailto:email@email.com'>email@email.com</a> address"
+    end
+  end
+end

data/spec/integration/html_spec.rb

@@ -79,4 +79,9 @@ describe 'HTML Output' do
       expect { @abstract.not_a_real_field }.to raise_error NoMethodError
     end
   end
+  describe 'individual fields' do
+    it 'should return ModsDispaly::Class when raw is specified' do
+      expect(@abstract.abstract(raw: true)).to be_a ModsDisplay::Abstract
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -1,6 +1,11 @@
+ENV['RAILS_ENV'] ||= 'test'
 require 'mods_display'
 require 'stanford-mods'
 require 'capybara'
+require 'rails'
+require 'fake_app'
+require 'rspec/rails'
+require 'mods_display/helpers/record_helper'
 
 Dir["#{File.expand_path('..', __FILE__)}/fixtures/*.rb"].each { |file| require file }
 # Load i18n test file.
@@ -24,6 +29,7 @@ RSpec.configure do |config|
   # the seed, which is printed after each run.
   #     --seed 1234
   config.order = 'random'
+  config.include Rails.application.routes.url_helpers
 end
 class TestModel
   attr_accessor :modsxml

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mods_display
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.10.0
 platform: ruby
 authors:
 - Jessie Keck
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-08-12 00:00:00.000000000 Z
+date: 2021-05-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: stanford-mods
@@ -66,6 +66,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -108,6 +122,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '6.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '6.0'
 description: MODS Display is a gem to centralize the display logic of MODS medadata.
 email:
 - jessie.keck@gmail.com
@@ -115,10 +143,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
-- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -163,14 +191,17 @@ files:
 - lib/mods_display/fields/subject.rb
 - lib/mods_display/fields/title.rb
 - lib/mods_display/fields/values.rb
+- lib/mods_display/helpers/record_helper.rb
 - lib/mods_display/html.rb
 - lib/mods_display/model_extension.rb
+- lib/mods_display/railtie.rb
 - lib/mods_display/related_item_concerns.rb
 - lib/mods_display/relator_codes.rb
 - lib/mods_display/version.rb
 - mods_display.gemspec
 - spec/configuration/access_condition_spec.rb
 - spec/configuration/base_spec.rb
+- spec/fake_app.rb
 - spec/fields/abstract_spec.rb
 - spec/fields/access_condition_spec.rb
 - spec/fields/audience_spec.rb
@@ -203,6 +234,7 @@ files:
 - spec/fixtures/related_item_fixtures.rb
 - spec/fixtures/subjects_fixtures.rb
 - spec/fixtures/title_fixtures.rb
+- spec/helpers/record_helper_spec.rb
 - spec/integration/configuration_spec.rb
 - spec/integration/html_spec.rb
 - spec/integration/installation_spec.rb
@@ -211,7 +243,7 @@ files:
 homepage: https://github.com/sul-dlss/mods_display
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -226,8 +258,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: The MODS Display gem allows implementers to configure a customized display
   of MODS metadata.  This display implements the specifications defined at Stanford
@@ -235,6 +267,7 @@ summary: The MODS Display gem allows implementers to configure a customized disp
 test_files:
 - spec/configuration/access_condition_spec.rb
 - spec/configuration/base_spec.rb
+- spec/fake_app.rb
 - spec/fields/abstract_spec.rb
 - spec/fields/access_condition_spec.rb
 - spec/fields/audience_spec.rb
@@ -267,6 +300,7 @@ test_files:
 - spec/fixtures/related_item_fixtures.rb
 - spec/fixtures/subjects_fixtures.rb
 - spec/fixtures/title_fixtures.rb
+- spec/helpers/record_helper_spec.rb
 - spec/integration/configuration_spec.rb
 - spec/integration/html_spec.rb
 - spec/integration/installation_spec.rb

data/.travis.yml

@@ -1,4 +0,0 @@
-notifications:
-  email: false
-rvm:
-  - 2.5.3