89255db0d8
* Add coverage for domain block service with silence * Get rid of warning about find_each and order * Move domain_block to attr_reader * Move optional clear_media into silence_accounts method * Use blocked_domain method to reduce passed vars * Extract blocked_domain_accounts method to find accounts on the domain * Extract media_from_blocked_domain method to find relevant attachments * Separate destruction of account images and account attachments
58 lines
2 KiB
Ruby
58 lines
2 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe BlockDomainService do
|
|
let(:bad_account) { Fabricate(:account, username: 'badguy666', domain: 'evil.org') }
|
|
let(:bad_status1) { Fabricate(:status, account: bad_account, text: 'You suck') }
|
|
let(:bad_status2) { Fabricate(:status, account: bad_account, text: 'Hahaha') }
|
|
let(:bad_attachment) { Fabricate(:media_attachment, account: bad_account, status: bad_status2, file: attachment_fixture('attachment.jpg')) }
|
|
|
|
subject { BlockDomainService.new }
|
|
|
|
before do
|
|
bad_account
|
|
bad_status1
|
|
bad_status2
|
|
bad_attachment
|
|
end
|
|
|
|
describe 'for a suspension' do
|
|
before do
|
|
subject.call(DomainBlock.create!(domain: 'evil.org', severity: :suspend))
|
|
end
|
|
|
|
it 'creates a domain block' do
|
|
expect(DomainBlock.blocked?('evil.org')).to be true
|
|
end
|
|
|
|
it 'removes remote accounts from that domain' do
|
|
expect(Account.find_remote('badguy666', 'evil.org').suspended?).to be true
|
|
end
|
|
|
|
it 'removes the remote accounts\'s statuses and media attachments' do
|
|
expect { bad_status1.reload }.to raise_exception ActiveRecord::RecordNotFound
|
|
expect { bad_status2.reload }.to raise_exception ActiveRecord::RecordNotFound
|
|
expect { bad_attachment.reload }.to raise_exception ActiveRecord::RecordNotFound
|
|
end
|
|
end
|
|
|
|
describe 'for a silence with reject media' do
|
|
before do
|
|
subject.call(DomainBlock.create!(domain: 'evil.org', severity: :silence, reject_media: true))
|
|
end
|
|
|
|
it 'does not create a domain block' do
|
|
expect(DomainBlock.blocked?('evil.org')).to be false
|
|
end
|
|
|
|
it 'silences remote accounts from that domain' do
|
|
expect(Account.find_remote('badguy666', 'evil.org').silenced?).to be true
|
|
end
|
|
|
|
it 'leaves the domains status and attachements, but clears media' do
|
|
expect { bad_status1.reload }.not_to raise_error
|
|
expect { bad_status2.reload }.not_to raise_error
|
|
expect { bad_attachment.reload }.not_to raise_error
|
|
expect(bad_attachment.file.exists?).to be false
|
|
end
|
|
end
|
|
end
|