Merge pull request #676 from MITLibraries/timx-197-record-not-found

Provide message when record not found

Author: JPrevost

app/graphql/types/query_type.rb

@@ -38,7 +38,7 @@ def info
       def record_id(id:, index:)
         result = Retrieve.new.fetch(id, Timdex::OSClient, index)
         result['hits']['hits'].first['_source']
-      rescue Elasticsearch::Transport::Transport::Errors::NotFound
+      rescue OpenSearch::Transport::Transport::Errors::NotFound
         raise GraphQL::ExecutionError, "Record '#{id}' not found"
       end
 

test/controllers/graphql_controller_v2_test.rb

@@ -331,6 +331,22 @@ def setup
     end
   end
 
+    test 'graphqlv2 retrieve with not found recordid' do
+    VCR.use_cassette('graphql v2 retrieve not found') do
+      post '/graphql', params: { query: '{
+                                  recordId(id: "totallylegitrecordid") {
+                                    timdexRecordId
+                                    title
+                                  }
+                                }' }
+      assert_equal(200, response.status)
+      json = JSON.parse(response.body)
+      assert_nil(json['data'])
+      assert_equal("Record 'totallylegitrecordid' not found", json['errors'].first['message'])
+    end
+  end
+
+
   test 'graphqlv2 holding location is not required' do
     VCR.use_cassette('graphql v2 location') do
       post '/graphql', params: { query: '{