Skip to content

Commit

Permalink
alert honeybadger for incomplete response from Symphony
Browse files Browse the repository at this point in the history
  • Loading branch information
ndushay committed Sep 6, 2019
1 parent 070e438 commit a12c41d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
1 change: 1 addition & 0 deletions app/models/symphony_reader.rb
Expand Up @@ -50,6 +50,7 @@ def validate_response(resp)
return resp if actual_content_length == exp_content_length

errmsg = "Incomplete response received from Symphony for #{@catkey} - expected #{exp_content_length} bytes but got #{actual_content_length}"
Honeybadger.notify(errmsg)
raise RecordIncompleteError, errmsg
end

Expand Down
8 changes: 6 additions & 2 deletions spec/models/symphony_reader_spec.rb
Expand Up @@ -63,8 +63,12 @@

context 'when wrong number of bytes received from Symphony' do
let(:headers) { { 'Content-Length': 268 } }
it 'raises RecordIncompleteError' do
expect { reader.to_marc }.to raise_error(SymphonyReader::RecordIncompleteError, 'Incomplete response received from Symphony for catkey - expected 268 bytes but got 394')

it 'raises RecordIncompleteError and notifies Honeybadger' do
msg = 'Incomplete response received from Symphony for catkey - expected 268 bytes but got 394'
allow(Honeybadger).to receive(:notify)
expect { reader.to_marc }.to raise_error(SymphonyReader::RecordIncompleteError, msg)
expect(Honeybadger).to have_received(:notify).with(msg)
end
end
end
Expand Down

0 comments on commit a12c41d

Please sign in to comment.