Updates for RLF to SLF name change

Author: steve-sullivan

CHANGES.md

@@ -1,3 +1,8 @@
+# 4.2.0 (2026-01-06)
+
+- Rename RLF to SLF (NRLF -> SLFN, SRLF -> SLFS)
+- Added bypass to skip SSL verification ONLY when recording new VCR cassettes
+
 # 4.1.0 (2025-01-10)
 
 - Fixed token refresh function and test

lib/berkeley_library/location/location_result.rb

@@ -13,12 +13,12 @@ def initialize(oclc_number, wc_symbols: [], wc_error: nil, ht_record_url: nil, h
         @ht_error = ht_error
       end
 
-      def nrlf?
-        @has_nrlf ||= wc_symbols.intersection(WorldCat::Symbols::NRLF).any?
+      def slfn?
+        @has_slfn ||= wc_symbols.intersection(WorldCat::Symbols::SLFN).any?
       end
 
-      def srlf?
-        @has_srlf ||= wc_symbols.intersection(WorldCat::Symbols::SRLF).any?
+      def slfs?
+        @has_slfs ||= wc_symbols.intersection(WorldCat::Symbols::SLFS).any?
       end
 
       def uc_symbols

lib/berkeley_library/location/world_cat/oclc_auth.rb

@@ -22,6 +22,9 @@ def fetch_token
           http = Net::HTTP.new(url.host, url.port)
           http.use_ssl = url.scheme == 'https'
 
+          # Skip SSL verification ONLY when recording new VCR cassettes
+          http.verify_mode = OpenSSL::SSL::VERIFY_NONE if ENV['RE_RECORD_VCR'] == 'true'
+
           request = Net::HTTP::Post.new(url.request_uri)
           request.basic_auth(Config.api_key, Config.api_secret)
           request['Accept'] = 'application/json'

lib/berkeley_library/location/world_cat/symbols.rb

@@ -2,12 +2,12 @@ module BerkeleyLibrary
   module Location
     module WorldCat
       module Symbols
-        NRLF = %w[ZAP ZAPSP].freeze
-        SRLF = %w[HH0 ZAS ZASSP].freeze
-        RLF = (NRLF + SRLF).freeze
+        SLFN = %w[ZAP ZAPSP].freeze
+        SLFS = %w[HH0 ZAS ZASSP].freeze
+        SLF = (SLFN + SLFS).freeze
 
         UC = %w[CLU CRU CUI CUN CUS CUT CUV CUX CUY CUZ MERUC].freeze
-        ALL = (RLF + UC).freeze
+        ALL = (SLF + UC).freeze
 
         class << self
           include Symbols

lib/berkeley_library/location/xlsx_writer.rb

@@ -7,22 +7,22 @@ class XLSXWriter
       include Constants
       include BerkeleyLibrary::Logging
 
-      COL_NRLF = 'NRLF'.freeze
-      COL_SRLF = 'SRLF'.freeze
+      COL_SLFN = 'SLFN'.freeze
+      COL_SLFS = 'SLFS'.freeze
       COL_OTHER_UC = 'Other UC'.freeze
       COL_WC_ERROR = 'WorldCat Error'.freeze
 
       COL_HATHI_TRUST = 'Hathi Trust'.freeze
       COL_HATHI_TRUST_ERROR = "#{COL_HATHI_TRUST} Error".freeze
 
-      V_NRLF = 'nrlf'.freeze
-      V_SRLF = 'srlf'.freeze
+      V_SLFN = 'slfn'.freeze
+      V_SLFS = 'slfs'.freeze
 
-      attr_reader :ss, :rlf, :uc, :hathi_trust
+      attr_reader :ss, :slf, :uc, :hathi_trust
 
-      def initialize(ss, rlf: true, uc: true, hathi_trust: true)
+      def initialize(ss, slf: true, uc: true, hathi_trust: true)
         @ss = ss
-        @rlf = rlf
+        @slf = slf
         @uc = uc
         @hathi_trust = hathi_trust
 
@@ -32,7 +32,7 @@ def initialize(ss, rlf: true, uc: true, hathi_trust: true)
       def <<(result)
         r_indices = row_indices_for(result.oclc_number)
         r_indices.each do |idx|
-          write_wc_cols(idx, result) if rlf || uc
+          write_wc_cols(idx, result) if slf || uc
           write_ht_cols(idx, result) if hathi_trust
         end
       end
@@ -41,7 +41,7 @@ def <<(result)
 
       def write_wc_cols(r_index, result)
         write_wc_error(r_index, result)
-        write_rlf(r_index, result) if rlf
+        write_slf(r_index, result) if slf
         write_uc(r_index, result) if uc
       end
 
@@ -51,9 +51,9 @@ def write_ht_cols(r_index, result)
       end
 
       def ensure_columns!
-        if rlf
-          nrlf_col_index
-          srlf_col_index
+        if slf
+          slfn_col_index
+          slfs_col_index
         end
         uc_col_index if uc
         ht_col_index if hathi_trust
@@ -66,9 +66,9 @@ def row_indices_for(oclc_number)
         raise ArgumentError, "Unknown OCLC number: #{oclc_number}"
       end
 
-      def write_rlf(r_index, result)
-        ss.set_value_at(r_index, nrlf_col_index, V_NRLF) if result.nrlf?
-        ss.set_value_at(r_index, srlf_col_index, V_SRLF) if result.srlf?
+      def write_slf(r_index, result)
+        ss.set_value_at(r_index, slfn_col_index, V_SLFN) if result.slfn?
+        ss.set_value_at(r_index, slfs_col_index, V_SLFS) if result.slfs?
       end
 
       def write_uc(r_index, result)
@@ -99,12 +99,12 @@ def oclc_col_index
         @oclc_col_index ||= ss.find_column_index_by_header!(OCLC_COL_HEADER)
       end
 
-      def nrlf_col_index
-        @nrlf_col_index ||= ss.ensure_column!(COL_NRLF)
+      def slfn_col_index
+        @slfn_col_index ||= ss.ensure_column!(COL_SLFN)
       end
 
-      def srlf_col_index
-        @srlf_col_index ||= ss.ensure_column!(COL_SRLF)
+      def slfs_col_index
+        @slfs_col_index ||= ss.ensure_column!(COL_SLFS)
       end
 
       def uc_col_index

spec/berkeley_library/location/world_cat/libraries_request_spec.rb

@@ -3,22 +3,28 @@ module Location
     module WorldCat
       describe LibrariesRequest do
         let(:oclc_number) { '85833285' }
-        # let(:wc_base_url) { 'https://www.example.test/webservices/' }
         let(:wc_base_url) { 'https://americas.discovery.api.oclc.org/worldcat/search/v2/' }
         let(:wc_api_key) { '2lo55pdh7moyfodeo4gwgms0on65x31ghv0g6yg87ffwaljsdw' }
         let(:wc_api_secret) { 'totallyfakesecret' }
 
-        # before do
-        #   # Config.base_uri = wc_base_url
-        #   # Config.api_key = wc_api_key
-        #   # Config.api_secret = wc_api_secret
-        # end
-
         after do
-          Config.send(:reset!)
+          BerkeleyLibrary::Location::WorldCat::OCLCAuth
+            .instance_variable_set(:@singleton__instance__, nil)
         end
 
+
         describe :new do
+          before do
+            auth = instance_double(
+              BerkeleyLibrary::Location::WorldCat::OCLCAuth,
+              access_token: 'fake-access-token'
+            )
+
+            allow(BerkeleyLibrary::Location::WorldCat::OCLCAuth)
+              .to receive(:new)
+              .and_return(auth)
+          end
+
           describe :oclc_number do
             it 'accepts a valid OCLC number' do
               q = LibrariesRequest.new(oclc_number)
@@ -65,13 +71,24 @@ module WorldCat
             end
 
             it 'rejects an array containing nonexistent symbols' do
-              bad_symbols = [Symbols::NRLF, ['not a WorldCat institution symbol'], Symbols::SRLF].flatten
+              bad_symbols = [Symbols::SLFN, ['not a WorldCat institution symbol'], Symbols::SLFS].flatten
               expect { LibrariesRequest.new(oclc_number, symbols: bad_symbols) }.to raise_error(ArgumentError)
             end
           end
         end
 
         describe :uri do
+          before do
+            auth = instance_double(
+              BerkeleyLibrary::Location::WorldCat::OCLCAuth,
+              access_token: 'fake-access-token'
+            )
+
+            allow(BerkeleyLibrary::Location::WorldCat::OCLCAuth)
+              .to receive(:new)
+              .and_return(auth)
+          end
+
           it 'returns the URI for the specified OCLC number' do
             uri_expected = URI.parse("#{wc_base_url}bibs-holdings")
             uri_actual = LibrariesRequest.new(oclc_number).uri
@@ -92,7 +109,7 @@ module WorldCat
 
           it 'returns a specified subset of holdings' do
             holdings_expected = %w[ZAP]
-            symbols = Symbols::RLF
+            symbols = Symbols::SLF
             req = LibrariesRequest.new(oclc_number, symbols:)
 
             VCR.use_cassette('libraries_request/execute_holdings_2') do
@@ -116,7 +133,7 @@ module WorldCat
 
           it 'returns an empty list when no holdings are found' do
             oclc_number = '10045193'
-            symbols = Symbols::RLF
+            symbols = Symbols::SLF
             req = LibrariesRequest.new(oclc_number, symbols:)
 
             VCR.use_cassette('libraries_request/execute_holdings_4') do