2020-05-03 16:30:36 +02:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'rails_helper'
|
|
|
|
|
2023-05-04 05:49:53 +02:00
|
|
|
RSpec.describe ActivityPub::RepliesController do
|
2020-05-03 16:30:36 +02:00
|
|
|
let(:status) { Fabricate(:status, visibility: parent_visibility) }
|
2022-02-07 17:06:43 +01:00
|
|
|
let(:remote_account) { Fabricate(:account, domain: 'foobar.com') }
|
|
|
|
let(:remote_reply_id) { 'https://foobar.com/statuses/1234' }
|
|
|
|
let(:remote_querier) { nil }
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
shared_examples 'common behavior' do
|
|
|
|
context 'when status is private' do
|
|
|
|
let(:parent_visibility) { :private }
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'returns http not found' do
|
|
|
|
expect(response).to have_http_status(404)
|
|
|
|
end
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when status is direct' do
|
|
|
|
let(:parent_visibility) { :direct }
|
2020-11-08 00:28:39 +01:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'returns http not found' do
|
|
|
|
expect(response).to have_http_status(404)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2020-11-08 00:28:39 +01:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
shared_examples 'disallowed access' do
|
|
|
|
context 'when status is public' do
|
|
|
|
let(:parent_visibility) { :public }
|
2020-11-08 00:28:39 +01:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'returns http not found' do
|
|
|
|
expect(response).to have_http_status(404)
|
2020-11-08 00:28:39 +01:00
|
|
|
end
|
2022-02-07 17:06:43 +01:00
|
|
|
end
|
2020-11-08 00:28:39 +01:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it_behaves_like 'common behavior'
|
|
|
|
end
|
2020-11-08 00:28:39 +01:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
shared_examples 'allowed access' do
|
|
|
|
context 'when account is permanently suspended' do
|
|
|
|
let(:parent_visibility) { :public }
|
2020-11-08 00:28:39 +01:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
before do
|
|
|
|
status.account.suspend!
|
|
|
|
status.account.deletion_request.destroy
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'returns http gone' do
|
|
|
|
expect(response).to have_http_status(410)
|
|
|
|
end
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when account is temporarily suspended' do
|
|
|
|
let(:parent_visibility) { :public }
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
before do
|
|
|
|
status.account.suspend!
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'returns http forbidden' do
|
|
|
|
expect(response).to have_http_status(403)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
2022-02-07 17:06:43 +01:00
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when status is public' do
|
|
|
|
let(:parent_visibility) { :public }
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2024-07-17 10:09:34 +02:00
|
|
|
it 'returns http success and correct media type' do
|
2024-09-03 17:35:19 +02:00
|
|
|
expect(response)
|
|
|
|
.to have_http_status(200)
|
|
|
|
.and have_cacheable_headers
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
expect(response.media_type).to eq 'application/activity+json'
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'without only_other_accounts' do
|
|
|
|
it "returns items with thread author's replies" do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(response.parsed_body)
|
|
|
|
.to include(
|
|
|
|
first: be_a(Hash).and(
|
|
|
|
include(
|
|
|
|
items: be_an(Array)
|
|
|
|
.and(have_attributes(size: 1))
|
|
|
|
.and(all(satisfy { |item| targets_public_collection?(item) }))
|
|
|
|
)
|
|
|
|
)
|
|
|
|
)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when there are few self-replies' do
|
|
|
|
it 'points next to replies from other people' do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(response.parsed_body)
|
|
|
|
.to include(
|
|
|
|
first: be_a(Hash).and(
|
|
|
|
include(
|
|
|
|
next: satisfy { |value| (parsed_uri_query_values(value) & %w(only_other_accounts=true page=true)).any? }
|
|
|
|
)
|
|
|
|
)
|
|
|
|
)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
2022-02-07 17:06:43 +01:00
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when there are many self-replies' do
|
|
|
|
before do
|
|
|
|
10.times { Fabricate(:status, account: status.account, thread: status, visibility: :public) }
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'points next to other self-replies' do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(response.parsed_body)
|
|
|
|
.to include(
|
|
|
|
first: be_a(Hash).and(
|
|
|
|
include(
|
|
|
|
next: satisfy { |value| (parsed_uri_query_values(value) & %w(only_other_accounts=false page=true)).any? }
|
|
|
|
)
|
|
|
|
)
|
|
|
|
)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
2022-02-07 17:06:43 +01:00
|
|
|
end
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'with only_other_accounts' do
|
|
|
|
let(:only_other_accounts) { 'true' }
|
2020-06-04 19:03:31 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'returns items with other public or unlisted replies' do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(response.parsed_body)
|
|
|
|
.to include(
|
|
|
|
first: be_a(Hash).and(
|
|
|
|
include(items: be_an(Array).and(have_attributes(size: 3)))
|
|
|
|
)
|
|
|
|
)
|
2022-02-07 17:06:43 +01:00
|
|
|
end
|
2020-06-04 19:03:31 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'only inlines items that are local and public or unlisted replies' do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(inlined_replies)
|
|
|
|
.to all(satisfy { |item| targets_public_collection?(item) })
|
|
|
|
.and all(satisfy { |item| ActivityPub::TagManager.instance.local_uri?(item[:id]) })
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'uses ids for remote toots' do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(remote_replies)
|
|
|
|
.to all(satisfy { |item| item.is_a?(String) && !ActivityPub::TagManager.instance.local_uri?(item) })
|
2022-02-07 17:06:43 +01:00
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when there are few replies' do
|
|
|
|
it 'does not have a next page' do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(response.parsed_body)
|
|
|
|
.to include(
|
|
|
|
first: be_a(Hash).and(not_include(next: be_present))
|
|
|
|
)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when there are many replies' do
|
|
|
|
before do
|
|
|
|
10.times { Fabricate(:status, thread: status, visibility: :public) }
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it 'points next to other replies' do
|
2024-09-26 10:54:01 +02:00
|
|
|
expect(response.parsed_body)
|
|
|
|
.to include(
|
|
|
|
first: be_a(Hash).and(
|
|
|
|
include(
|
|
|
|
next: satisfy { |value| (parsed_uri_query_values(value) & %w(only_other_accounts=true page=true)).any? }
|
|
|
|
)
|
|
|
|
)
|
|
|
|
)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2022-02-07 17:06:43 +01:00
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it_behaves_like 'common behavior'
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
before do
|
|
|
|
stub_const 'ActivityPub::RepliesController::DESCENDANTS_LIMIT', 5
|
2022-09-21 22:45:57 +02:00
|
|
|
allow(controller).to receive(:signed_request_actor).and_return(remote_querier)
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
Fabricate(:status, thread: status, visibility: :public)
|
|
|
|
Fabricate(:status, thread: status, visibility: :public)
|
|
|
|
Fabricate(:status, thread: status, visibility: :private)
|
|
|
|
Fabricate(:status, account: status.account, thread: status, visibility: :public)
|
|
|
|
Fabricate(:status, account: status.account, thread: status, visibility: :private)
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
Fabricate(:status, account: remote_account, thread: status, visibility: :public, uri: remote_reply_id)
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
describe 'GET #index' do
|
|
|
|
subject(:response) { get :index, params: { account_username: status.account.username, status_id: status.id, only_other_accounts: only_other_accounts } }
|
2023-02-20 02:46:00 +01:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
let(:only_other_accounts) { nil }
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'with no signature' do
|
|
|
|
it_behaves_like 'allowed access'
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'with signature' do
|
|
|
|
let(:remote_querier) { Fabricate(:account, domain: 'example.com') }
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it_behaves_like 'allowed access'
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when signed request account is blocked' do
|
|
|
|
before do
|
|
|
|
status.account.block!(remote_querier)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it_behaves_like 'disallowed access'
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
context 'when signed request account is domain blocked' do
|
|
|
|
before do
|
|
|
|
status.account.block_domain!(remote_querier.domain)
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
|
2022-02-07 17:06:43 +01:00
|
|
|
it_behaves_like 'disallowed access'
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2024-01-11 17:17:21 +01:00
|
|
|
|
|
|
|
private
|
|
|
|
|
2024-09-26 10:54:01 +02:00
|
|
|
def inlined_replies
|
|
|
|
response
|
|
|
|
.parsed_body[:first][:items]
|
|
|
|
.select { |x| x.is_a?(Hash) }
|
|
|
|
end
|
|
|
|
|
|
|
|
def remote_replies
|
|
|
|
response
|
|
|
|
.parsed_body[:first][:items]
|
|
|
|
.reject { |x| x.is_a?(Hash) }
|
|
|
|
end
|
|
|
|
|
2024-02-29 14:47:38 +01:00
|
|
|
def parsed_uri_query_values(uri)
|
|
|
|
Addressable::URI
|
|
|
|
.parse(uri)
|
|
|
|
.query
|
|
|
|
.split('&')
|
|
|
|
end
|
|
|
|
|
2024-01-11 17:17:21 +01:00
|
|
|
def ap_public_collection
|
|
|
|
ActivityPub::TagManager::COLLECTIONS[:public]
|
|
|
|
end
|
|
|
|
|
|
|
|
def targets_public_collection?(item)
|
|
|
|
item[:to].include?(ap_public_collection) || item[:cc].include?(ap_public_collection)
|
|
|
|
end
|
2020-05-03 16:30:36 +02:00
|
|
|
end
|