mirror of
https://github.com/lunaisnotaboy/mastodon.git
synced 2024-12-21 20:56:50 +00:00
aeb124491d
Mastodon expects remote servers to remove follow relationships upon receiving a Block. However, the spec only evokes Block activities in a C2S context, never in a S2S context. This PR, in addition to federating the Block, explicitly sends a Reject for any affected follow relationship, which makes a bit more sense with regards to the spec.
78 lines
2.5 KiB
Ruby
78 lines
2.5 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe UnfollowService, type: :service do
|
|
let(:sender) { Fabricate(:account, username: 'alice') }
|
|
|
|
subject { UnfollowService.new }
|
|
|
|
describe 'local' do
|
|
let(:bob) { Fabricate(:user, email: 'bob@example.com', account: Fabricate(:account, username: 'bob')).account }
|
|
|
|
before do
|
|
sender.follow!(bob)
|
|
subject.call(sender, bob)
|
|
end
|
|
|
|
it 'destroys the following relation' do
|
|
expect(sender.following?(bob)).to be false
|
|
end
|
|
end
|
|
|
|
describe 'remote OStatus' do
|
|
let(:bob) { Fabricate(:user, email: 'bob@example.com', account: Fabricate(:account, username: 'bob', protocol: :ostatus, domain: 'example.com', salmon_url: 'http://salmon.example.com')).account }
|
|
|
|
before do
|
|
sender.follow!(bob)
|
|
stub_request(:post, "http://salmon.example.com/").to_return(:status => 200, :body => "", :headers => {})
|
|
subject.call(sender, bob)
|
|
end
|
|
|
|
it 'destroys the following relation' do
|
|
expect(sender.following?(bob)).to be false
|
|
end
|
|
|
|
it 'sends an unfollow salmon slap' do
|
|
expect(a_request(:post, "http://salmon.example.com/").with { |req|
|
|
xml = OStatus2::Salmon.new.unpack(req.body)
|
|
xml.match(OStatus::TagManager::VERBS[:unfollow])
|
|
}).to have_been_made.once
|
|
end
|
|
end
|
|
|
|
describe 'remote ActivityPub' do
|
|
let(:bob) { Fabricate(:user, email: 'bob@example.com', account: Fabricate(:account, username: 'bob', protocol: :activitypub, domain: 'example.com', inbox_url: 'http://example.com/inbox')).account }
|
|
|
|
before do
|
|
sender.follow!(bob)
|
|
stub_request(:post, 'http://example.com/inbox').to_return(status: 200)
|
|
subject.call(sender, bob)
|
|
end
|
|
|
|
it 'destroys the following relation' do
|
|
expect(sender.following?(bob)).to be false
|
|
end
|
|
|
|
it 'sends an unfollow activity' do
|
|
expect(a_request(:post, 'http://example.com/inbox')).to have_been_made.once
|
|
end
|
|
end
|
|
|
|
describe 'remote ActivityPub (reverse)' do
|
|
let(:bob) { Fabricate(:user, email: 'bob@example.com', account: Fabricate(:account, username: 'bob', protocol: :activitypub, domain: 'example.com', inbox_url: 'http://example.com/inbox')).account }
|
|
|
|
before do
|
|
bob.follow!(sender)
|
|
stub_request(:post, 'http://example.com/inbox').to_return(status: 200)
|
|
subject.call(bob, sender)
|
|
end
|
|
|
|
it 'destroys the following relation' do
|
|
expect(bob.following?(sender)).to be false
|
|
end
|
|
|
|
it 'sends a reject activity' do
|
|
expect(a_request(:post, 'http://example.com/inbox')).to have_been_made.once
|
|
end
|
|
end
|
|
end
|