Fix holdingType typo

sul-dlss · Jul 25, 2023 · dd169fc · dd169fc
1 parent 7b29ad5
commit dd169fc
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/lib/folio/eresource_holdings_builder.rb b/lib/folio/eresource_holdings_builder.rb
@@ -84,7 +84,7 @@ def mapped_location_codes
     # This approach works fine unless there are records with multiple
     # e-resource holdings associated with different locations.
     def electronic_holding_location
-      @electronic_holding_location ||= holdings&.find { |h| (h.dig('holdingType', 'name') || h.dig('location', 'effectiveLocation', 'details', 'holdingTypeName')) == 'Electronic' }
+      @electronic_holding_location ||= holdings&.find { |h| (h.dig('holdingsType', 'name') || h.dig('location', 'effectiveLocation', 'details', 'holdingsTypeName')) == 'Electronic' }
     end
   end
 end
diff --git a/spec/integration/folio_config_spec.rb b/spec/integration/folio_config_spec.rb
@@ -113,7 +113,7 @@
     let(:items_and_holdings) do
       { 'items' => [],
         'holdings' =>
-         [{ 'holdingType' => { 'name' => 'Electronic' },
+         [{ 'holdingsType' => { 'name' => 'Electronic' },
             'location' =>
           { 'permanentLocation' =>
             { 'code' => 'SUL-ELECTRONIC' },
@@ -147,7 +147,7 @@
       let(:items_and_holdings) do
         { 'items' => [],
           'holdings' =>
-           [{ 'holdingType' => { 'name' => 'Electronic' },
+           [{ 'holdingsType' => { 'name' => 'Electronic' },
               'location' =>
             { 'permanentLocation' =>
               { 'code' => 'LAW-ELECTRONIC' },

diff --git a/spec/lib/folio/eresource_holdings_builder_spec.rb b/spec/lib/folio/eresource_holdings_builder_spec.rb
@@ -24,7 +24,7 @@
   let(:items_and_holdings) do
     { 'items' => [],
       'holdings' =>
-       [{ 'holdingType' => { 'name' => 'Electronic' },
+       [{ 'holdingsType' => { 'name' => 'Electronic' },
           'location' =>
           { 'permanentLocation' =>
             { 'code' => 'SUL-ELECTRONIC' },
@@ -64,7 +64,7 @@
     let(:items_and_holdings) do
       { 'items' => [],
         'holdings' =>
-         [{ 'holdingType' => { 'name' => 'Electronic' },
+         [{ 'holdingsType' => { 'name' => 'Electronic' },
             'location' =>
           { 'permanentLocation' =>
             { 'code' => 'LAW-ELECTRONIC' },
@@ -82,7 +82,7 @@
     let(:items_and_holdings) do
       { 'items' => [],
         'holdings' =>
-         [{ 'holdingType' => { 'name' => 'Whatever' } }] }
+         [{ 'holdingsType' => { 'name' => 'Whatever' } }] }
     end
 
     it { expect(holdings).to be_empty }
@@ -92,7 +92,7 @@
     let(:items_and_holdings) do
       { 'items' => [],
         'holdings' =>
-         [{ 'holdingType' => { 'name' => 'Electronic' }, 'location' =>
+         [{ 'holdingsType' => { 'name' => 'Electronic' }, 'location' =>
           { 'permanentLocation' =>
             { 'code' => 'SUL-SDR' },
             'effectiveLocation' =>
@@ -104,15 +104,15 @@
     it { expect(holdings.first.home_location).to eq 'INTERNET' }
   end
 
-  context 'the holding library is Lane (without a explicit holdingType)' do
+  context 'the holding library is Lane (without a explicit holdingsType)' do
     let(:items_and_holdings) do
       { 'items' => [],
         'holdings' =>
          [{ 'location' =>
           { 'permanentLocation' =>
             { 'code' => 'LANE-EDATA' },
             'effectiveLocation' =>
-            { 'code' => 'LANE-EDATA',  'details' => { 'holdingTypeName' => 'Electronic' } } },
+            { 'code' => 'LANE-EDATA',  'details' => { 'holdingsTypeName' => 'Electronic' } } },
             'suppressFromDiscovery' => false,
             'id' => '81a56270-e8dd-5759-8083-5cc96cdf0045',
             'holdingsStatements' => [] }] }