fedi/mastodon
1
0
Fork 1
mirror of https://github.com/mastodon/mastodon.git synced 2024-12-12 05:24:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add tests for broken mention processing (#33171)

Browse source
This commit is contained in:
Claire 2024-12-04 10:45:35 +01:00 committed by GitHub
parent 1cb303ca75
commit c521a62b1e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
spec/services/activitypub/process_status_update_service_spec.rb
View file

@ -6,6 +6,7 @@ RSpec.describe ActivityPub::ProcessStatusUpdateService do
subject { described_class.new } subject { described_class.new }
let!(:status) { Fabricate(:status, text: 'Hello world', account: Fabricate(:account, domain: 'example.com')) } let!(:status) { Fabricate(:status, text: 'Hello world', account: Fabricate(:account, domain: 'example.com')) }
let(:bogus_mention) { 'https://example.com/users/erroringuser' }
let(:payload) do let(:payload) do
{ {
'@context': 'https://www.w3.org/ns/activitystreams', '@context': 'https://www.w3.org/ns/activitystreams',
@ -17,6 +18,7 @@ RSpec.describe ActivityPub::ProcessStatusUpdateService do
tag: [ tag: [
{ type: 'Hashtag', name: 'hoge' }, { type: 'Hashtag', name: 'hoge' },
{ type: 'Mention', href: ActivityPub::TagManager.instance.uri_for(alice) }, { type: 'Mention', href: ActivityPub::TagManager.instance.uri_for(alice) },
{ type: 'Mention', href: bogus_mention },
], ],
} }
end end
@ -33,16 +35,18 @@ RSpec.describe ActivityPub::ProcessStatusUpdateService do
mentions.each { |a| Fabricate(:mention, status: status, account: a) } mentions.each { |a| Fabricate(:mention, status: status, account: a) }
tags.each { |t| status.tags << t } tags.each { |t| status.tags << t }
media_attachments.each { |m| status.media_attachments << m } media_attachments.each { |m| status.media_attachments << m }
stub_request(:get, bogus_mention).to_raise(HTTP::ConnectionError)
end end
describe '#call' do describe '#call' do
it 'updates text and content warning' do it 'updates text and content warning, and schedules re-fetching broken mention' do
subject.call(status, json, json) subject.call(status, json, json)
expect(status.reload) expect(status.reload)
.to have_attributes( .to have_attributes(
text: eq('Hello universe'), text: eq('Hello universe'),
spoiler_text: eq('Show more') spoiler_text: eq('Show more')
) )
expect(MentionResolveWorker).to have_enqueued_sidekiq_job(status.id, bogus_mention, anything)
end end
context 'when the changes are only in sanitized-out HTML' do context 'when the changes are only in sanitized-out HTML' do