forked from fedi/mastodon
be3b9f8151
* Fix URI of repeat follow requests not being recorded In case we receive a “repeat” or “duplicate” follow request, we automatically fast-forward the accept with the latest received Activity `id`, but we don't record it. In general, a “repeat” or “duplicate” follow request may happen if for some reason (e.g. inconsistent handling of Block or Undo Accept activities, an instance being brought back up from the dead, etc.) the local instance thought the remote actor were following them while the remote actor thought otherwise. In those cases, the remote instance does not know about the older Follow activity `id`, so keeping that record serves no purpose, but knowing the most recent one is useful if the remote implementation at some point refers to it by `id` without inlining it. * Add tests
189 lines
5.4 KiB
Ruby
189 lines
5.4 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe ActivityPub::Activity::Follow do
|
|
let(:sender) { Fabricate(:account) }
|
|
let(:recipient) { Fabricate(:account) }
|
|
|
|
let(:json) do
|
|
{
|
|
'@context': 'https://www.w3.org/ns/activitystreams',
|
|
id: 'foo',
|
|
type: 'Follow',
|
|
actor: ActivityPub::TagManager.instance.uri_for(sender),
|
|
object: ActivityPub::TagManager.instance.uri_for(recipient),
|
|
}.with_indifferent_access
|
|
end
|
|
|
|
describe '#perform' do
|
|
subject { described_class.new(json, sender) }
|
|
|
|
context 'with no prior follow' do
|
|
context 'unlocked account' do
|
|
before do
|
|
subject.perform
|
|
end
|
|
|
|
it 'creates a follow from sender to recipient' do
|
|
expect(sender.following?(recipient)).to be true
|
|
expect(sender.active_relationships.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
|
|
it 'does not create a follow request' do
|
|
expect(sender.requested?(recipient)).to be false
|
|
end
|
|
end
|
|
|
|
context 'silenced account following an unlocked account' do
|
|
before do
|
|
sender.touch(:silenced_at)
|
|
subject.perform
|
|
end
|
|
|
|
it 'does not create a follow from sender to recipient' do
|
|
expect(sender.following?(recipient)).to be false
|
|
end
|
|
|
|
it 'creates a follow request' do
|
|
expect(sender.requested?(recipient)).to be true
|
|
expect(sender.follow_requests.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
end
|
|
|
|
context 'unlocked account muting the sender' do
|
|
before do
|
|
recipient.mute!(sender)
|
|
subject.perform
|
|
end
|
|
|
|
it 'creates a follow from sender to recipient' do
|
|
expect(sender.following?(recipient)).to be true
|
|
expect(sender.active_relationships.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
|
|
it 'does not create a follow request' do
|
|
expect(sender.requested?(recipient)).to be false
|
|
end
|
|
end
|
|
|
|
context 'locked account' do
|
|
before do
|
|
recipient.update(locked: true)
|
|
subject.perform
|
|
end
|
|
|
|
it 'does not create a follow from sender to recipient' do
|
|
expect(sender.following?(recipient)).to be false
|
|
end
|
|
|
|
it 'creates a follow request' do
|
|
expect(sender.requested?(recipient)).to be true
|
|
expect(sender.follow_requests.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'when a follow relationship already exists' do
|
|
before do
|
|
sender.active_relationships.create!(target_account: recipient, uri: 'bar')
|
|
end
|
|
|
|
context 'unlocked account' do
|
|
before do
|
|
subject.perform
|
|
end
|
|
|
|
it 'correctly sets the new URI' do
|
|
expect(sender.active_relationships.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
|
|
it 'does not create a follow request' do
|
|
expect(sender.requested?(recipient)).to be false
|
|
end
|
|
end
|
|
|
|
context 'silenced account following an unlocked account' do
|
|
before do
|
|
sender.touch(:silenced_at)
|
|
subject.perform
|
|
end
|
|
|
|
it 'correctly sets the new URI' do
|
|
expect(sender.active_relationships.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
|
|
it 'does not create a follow request' do
|
|
expect(sender.requested?(recipient)).to be false
|
|
end
|
|
end
|
|
|
|
context 'unlocked account muting the sender' do
|
|
before do
|
|
recipient.mute!(sender)
|
|
subject.perform
|
|
end
|
|
|
|
it 'correctly sets the new URI' do
|
|
expect(sender.active_relationships.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
|
|
it 'does not create a follow request' do
|
|
expect(sender.requested?(recipient)).to be false
|
|
end
|
|
end
|
|
|
|
context 'locked account' do
|
|
before do
|
|
recipient.update(locked: true)
|
|
subject.perform
|
|
end
|
|
|
|
it 'correctly sets the new URI' do
|
|
expect(sender.active_relationships.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
|
|
it 'does not create a follow request' do
|
|
expect(sender.requested?(recipient)).to be false
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'when a follow request already exists' do
|
|
before do
|
|
sender.follow_requests.create!(target_account: recipient, uri: 'bar')
|
|
end
|
|
|
|
context 'silenced account following an unlocked account' do
|
|
before do
|
|
sender.touch(:silenced_at)
|
|
subject.perform
|
|
end
|
|
|
|
it 'does not create a follow from sender to recipient' do
|
|
expect(sender.following?(recipient)).to be false
|
|
end
|
|
|
|
it 'correctly sets the new URI' do
|
|
expect(sender.requested?(recipient)).to be true
|
|
expect(sender.follow_requests.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
end
|
|
|
|
context 'locked account' do
|
|
before do
|
|
recipient.update(locked: true)
|
|
subject.perform
|
|
end
|
|
|
|
it 'does not create a follow from sender to recipient' do
|
|
expect(sender.following?(recipient)).to be false
|
|
end
|
|
|
|
it 'correctly sets the new URI' do
|
|
expect(sender.requested?(recipient)).to be true
|
|
expect(sender.follow_requests.find_by(target_account: recipient).uri).to eq 'foo'
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|