lib/dor/services/client/marcxml.rb in dor-services-client-4.9.0 vs lib/dor/services/client/marcxml.rb in dor-services-client-4.10.0

@@ -1,26 +1,63 @@
 # frozen_string_literal: true
 
+require 'nokogiri'
+
 module Dor
   module Services
     class Client
       # API calls around MARCXML-based operations from dor-services-app
       class Marcxml < VersionedService
         # Get a catkey corresponding to a barcode
         # @param barcode [String] required string representing a barcode
         # @raise [NotFoundResponse] when the response is a 404 (object not found)
         # @raise [UnexpectedResponse] on an unsuccessful response from the server
-        # @return [String] result of background job
+        # @return [String] catkey
         def catkey(barcode:)
           resp = connection.get do |req|
             req.url "#{api_version}/catalog/catkey"
             req.params['barcode'] = barcode
           end
 
           return resp.body if resp.success? && resp.body.present?
           raise NotFoundResponse if resp.success? && resp.body.blank?
 
           raise_exception_based_on_response!(resp)
+        end
+
+        # Gets MARCXML corresponding to a barcode or catkey
+        # @param barcode [String] required string representing a barcode
+        # @param catkey [String] required string representing a catkey
+        # @raise [NotFoundResponse] when the response is a 500 with "Record not found in Symphony"
+        # @raise [UnexpectedResponse] on an unsuccessful response from the server
+        # @return [String] MARCXML
+        def marcxml(barcode: nil, catkey: nil)
+          check_args(barcode, catkey)
+
+          resp = connection.get do |req|
+            req.url "#{api_version}/catalog/marcxml"
+            req.params['barcode'] = barcode unless barcode.nil?
+            req.params['catkey'] = catkey unless catkey.nil?
+          end
+
+          raise_exception_based_on_marcxml_response!(resp)
+
+          resp.body
+        end
+
+        private
+
+        def check_args(barcode, catkey)
+          raise ArgumentError, 'Barcode or catkey must be provided' if barcode.nil? && catkey.nil?
+          raise ArgumentError, 'Both barcode and catkey may not be provided' if !barcode.nil? && !catkey.nil?
+        end
+
+        def raise_exception_based_on_marcxml_response!(resp)
+          # DOR Services App does not respond with a 404 when no match in Symphony.
+          # Rather, it responds with a 500 containing "Record not found in Symphony" in the body.
+          raise NotFoundResponse if !resp.success? && resp.body.match?(/Record not found in Symphony/)
+
+          raise UnexpectedResponse, ResponseErrorFormatter.format(response: resp) unless resp.success?
         end
       end
     end
   end
 end