Add coverage for sanitize failure path in api/web/embeds spec (#29851)
This commit is contained in:
parent
1c87cb8019
commit
191bf5876e
|
@ -137,6 +137,18 @@ RSpec.describe '/api/web/embed' do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
context 'when sanitizing the fragment fails' do
|
||||||
|
let(:call_result) { { html: 'ok' } }
|
||||||
|
|
||||||
|
before { allow(Sanitize).to receive(:fragment).and_raise(ArgumentError) }
|
||||||
|
|
||||||
|
it 'returns http not found' do
|
||||||
|
subject
|
||||||
|
|
||||||
|
expect(response).to have_http_status(404)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
context 'when failing to fetch OEmbed' do
|
context 'when failing to fetch OEmbed' do
|
||||||
let(:call_result) { nil }
|
let(:call_result) { nil }
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue