2023-02-21 16:55:31 -08:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2016-02-24 15:17:01 -08:00
|
|
|
require 'rails_helper'
|
|
|
|
|
2024-03-13 01:39:26 -07:00
|
|
|
RSpec.describe PostStatusService do
|
2023-06-06 04:58:33 -07:00
|
|
|
subject { described_class.new }
|
2016-03-05 03:50:59 -08:00
|
|
|
|
2017-04-07 10:48:38 -07:00
|
|
|
it 'creates a new status' do
|
|
|
|
account = Fabricate(:account)
|
2023-02-18 14:38:14 -08:00
|
|
|
text = 'test status update'
|
2017-04-07 09:50:43 -07:00
|
|
|
|
2019-01-05 03:43:28 -08:00
|
|
|
status = subject.call(account, text: text)
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status.text).to eq text
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'creates a new response status' do
|
|
|
|
in_reply_to_status = Fabricate(:status)
|
2017-04-07 09:50:43 -07:00
|
|
|
account = Fabricate(:account)
|
2023-02-18 14:38:14 -08:00
|
|
|
text = 'test status update'
|
2017-04-07 09:50:43 -07:00
|
|
|
|
2019-01-05 03:43:28 -08:00
|
|
|
status = subject.call(account, text: text, thread: in_reply_to_status)
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status.text).to eq text
|
|
|
|
expect(status.thread).to eq in_reply_to_status
|
|
|
|
end
|
|
|
|
|
2021-10-14 10:59:21 -07:00
|
|
|
context 'when scheduling a status' do
|
|
|
|
let!(:account) { Fabricate(:account) }
|
|
|
|
let!(:future) { Time.now.utc + 2.hours }
|
|
|
|
let!(:previous_status) { Fabricate(:status, account: account) }
|
|
|
|
|
2024-02-15 23:52:29 -08:00
|
|
|
it 'schedules a status for future creation and does not create one immediately' do
|
2021-10-14 10:59:21 -07:00
|
|
|
media = Fabricate(:media_attachment, account: account)
|
|
|
|
status = subject.call(account, text: 'Hi future!', media_ids: [media.id], scheduled_at: future)
|
|
|
|
|
2024-02-15 23:52:29 -08:00
|
|
|
expect(status)
|
|
|
|
.to be_a(ScheduledStatus)
|
|
|
|
.and have_attributes(
|
|
|
|
scheduled_at: eq(future),
|
|
|
|
params: include(
|
|
|
|
'text' => eq('Hi future!'),
|
|
|
|
'media_ids' => contain_exactly(media.id)
|
|
|
|
)
|
|
|
|
)
|
2021-10-14 10:59:21 -07:00
|
|
|
expect(media.reload.status).to be_nil
|
2023-05-23 07:49:11 -07:00
|
|
|
expect(Status.where(text: 'Hi future!')).to_not exist
|
2021-10-14 10:59:21 -07:00
|
|
|
end
|
|
|
|
|
2024-02-15 23:52:29 -08:00
|
|
|
it 'does not change statuses_count of account or replies_count of thread previous status' do
|
|
|
|
expect { subject.call(account, text: 'Hi future!', scheduled_at: future, thread: previous_status) }
|
|
|
|
.to not_change { account.statuses_count }
|
|
|
|
.and(not_change { previous_status.replies_count })
|
2021-10-14 10:59:21 -07:00
|
|
|
end
|
2024-04-26 06:19:02 -07:00
|
|
|
|
|
|
|
it 'returns existing status when used twice with idempotency key' do
|
|
|
|
account = Fabricate(:account)
|
|
|
|
status1 = subject.call(account, text: 'test', idempotency: 'meepmeep', scheduled_at: future)
|
|
|
|
status2 = subject.call(account, text: 'test', idempotency: 'meepmeep', scheduled_at: future)
|
|
|
|
expect(status2.id).to eq status1.id
|
|
|
|
end
|
2024-06-10 06:33:48 -07:00
|
|
|
|
|
|
|
context 'when scheduled_at is less than min offset' do
|
|
|
|
let(:invalid_scheduled_time) { 4.minutes.from_now }
|
|
|
|
|
|
|
|
it 'raises invalid record error' do
|
|
|
|
expect do
|
|
|
|
subject.call(account, text: 'Hi future!', scheduled_at: invalid_scheduled_time)
|
|
|
|
end.to raise_error(ActiveRecord::RecordInvalid)
|
|
|
|
end
|
|
|
|
end
|
2019-01-21 11:03:04 -08:00
|
|
|
end
|
|
|
|
|
2018-11-25 07:35:21 -08:00
|
|
|
it 'creates response to the original status of boost' do
|
|
|
|
boosted_status = Fabricate(:status)
|
|
|
|
in_reply_to_status = Fabricate(:status, reblog: boosted_status)
|
|
|
|
account = Fabricate(:account)
|
2023-02-18 14:38:14 -08:00
|
|
|
text = 'test status update'
|
2018-11-25 07:35:21 -08:00
|
|
|
|
2019-01-05 03:43:28 -08:00
|
|
|
status = subject.call(account, text: text, thread: in_reply_to_status)
|
2018-11-25 07:35:21 -08:00
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status.text).to eq text
|
|
|
|
expect(status.thread).to eq boosted_status
|
|
|
|
end
|
|
|
|
|
2017-04-07 10:48:38 -07:00
|
|
|
it 'creates a sensitive status' do
|
|
|
|
status = create_status_with_options(sensitive: true)
|
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status).to be_sensitive
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'creates a status with spoiler text' do
|
2023-02-18 14:38:14 -08:00
|
|
|
spoiler_text = 'spoiler text'
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
status = create_status_with_options(spoiler_text: spoiler_text)
|
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status.spoiler_text).to eq spoiler_text
|
|
|
|
end
|
|
|
|
|
2020-03-25 14:40:58 -07:00
|
|
|
it 'creates a sensitive status when there is a CW but no text' do
|
|
|
|
status = subject.call(Fabricate(:account), text: '', spoiler_text: 'foo')
|
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status).to be_sensitive
|
|
|
|
end
|
|
|
|
|
2017-04-07 10:48:38 -07:00
|
|
|
it 'creates a status with empty default spoiler text' do
|
|
|
|
status = create_status_with_options(spoiler_text: nil)
|
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status.spoiler_text).to eq ''
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'creates a status with the given visibility' do
|
|
|
|
status = create_status_with_options(visibility: :private)
|
|
|
|
|
|
|
|
expect(status).to be_persisted
|
2023-02-18 14:38:14 -08:00
|
|
|
expect(status.visibility).to eq 'private'
|
2017-04-07 10:48:38 -07:00
|
|
|
end
|
|
|
|
|
2018-12-24 10:06:14 -08:00
|
|
|
it 'creates a status with limited visibility for silenced users' do
|
2019-01-05 03:43:28 -08:00
|
|
|
status = subject.call(Fabricate(:account, silenced: true), text: 'test', visibility: :public)
|
2018-12-24 10:06:14 -08:00
|
|
|
|
|
|
|
expect(status).to be_persisted
|
2023-02-18 14:38:14 -08:00
|
|
|
expect(status.visibility).to eq 'unlisted'
|
2018-12-24 10:06:14 -08:00
|
|
|
end
|
|
|
|
|
2017-04-07 10:48:38 -07:00
|
|
|
it 'creates a status for the given application' do
|
|
|
|
application = Fabricate(:application)
|
|
|
|
|
|
|
|
status = create_status_with_options(application: application)
|
|
|
|
|
|
|
|
expect(status).to be_persisted
|
|
|
|
expect(status.application).to eq application
|
|
|
|
end
|
|
|
|
|
2017-04-18 13:20:12 -07:00
|
|
|
it 'creates a status with a language set' do
|
|
|
|
account = Fabricate(:account)
|
2017-09-16 05:59:41 -07:00
|
|
|
text = 'This is an English text.'
|
2017-04-18 13:20:12 -07:00
|
|
|
|
2019-01-05 03:43:28 -08:00
|
|
|
status = subject.call(account, text: text)
|
2017-04-18 13:20:12 -07:00
|
|
|
|
2017-09-16 05:59:41 -07:00
|
|
|
expect(status.language).to eq 'en'
|
2017-04-18 13:20:12 -07:00
|
|
|
end
|
|
|
|
|
2017-04-07 10:48:38 -07:00
|
|
|
it 'processes mentions' do
|
2023-06-22 05:55:22 -07:00
|
|
|
mention_service = instance_double(ProcessMentionsService)
|
2017-04-07 10:48:38 -07:00
|
|
|
allow(mention_service).to receive(:call)
|
|
|
|
allow(ProcessMentionsService).to receive(:new).and_return(mention_service)
|
|
|
|
account = Fabricate(:account)
|
|
|
|
|
2023-02-18 14:38:14 -08:00
|
|
|
status = subject.call(account, text: 'test status update')
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
expect(ProcessMentionsService).to have_received(:new)
|
2023-02-13 07:36:29 -08:00
|
|
|
expect(mention_service).to have_received(:call).with(status, save_records: false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'safeguards mentions' do
|
|
|
|
account = Fabricate(:account)
|
|
|
|
mentioned_account = Fabricate(:account, username: 'alice')
|
|
|
|
unexpected_mentioned_account = Fabricate(:account, username: 'bob')
|
|
|
|
|
|
|
|
expect do
|
|
|
|
subject.call(account, text: '@alice hm, @bob is really annoying lately', allowed_mentions: [mentioned_account.id])
|
2024-03-13 08:42:39 -07:00
|
|
|
end.to raise_error(an_instance_of(described_class::UnexpectedMentionsError).and(having_attributes(accounts: [unexpected_mentioned_account])))
|
2023-02-13 07:36:29 -08:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'processes duplicate mentions correctly' do
|
|
|
|
account = Fabricate(:account)
|
2023-10-19 07:55:06 -07:00
|
|
|
Fabricate(:account, username: 'alice')
|
2023-02-13 07:36:29 -08:00
|
|
|
|
|
|
|
expect do
|
|
|
|
subject.call(account, text: '@alice @alice @alice hey @alice')
|
2023-02-19 17:33:27 -08:00
|
|
|
end.to_not raise_error
|
2017-04-07 10:48:38 -07:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'processes hashtags' do
|
2023-06-22 05:55:22 -07:00
|
|
|
hashtags_service = instance_double(ProcessHashtagsService)
|
2017-04-07 10:48:38 -07:00
|
|
|
allow(hashtags_service).to receive(:call)
|
|
|
|
allow(ProcessHashtagsService).to receive(:new).and_return(hashtags_service)
|
|
|
|
account = Fabricate(:account)
|
|
|
|
|
2023-02-18 14:38:14 -08:00
|
|
|
status = subject.call(account, text: 'test status update')
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
expect(ProcessHashtagsService).to have_received(:new)
|
|
|
|
expect(hashtags_service).to have_received(:call).with(status)
|
|
|
|
end
|
|
|
|
|
2017-08-12 15:44:41 -07:00
|
|
|
it 'gets distributed' do
|
2017-04-07 10:48:38 -07:00
|
|
|
allow(DistributionWorker).to receive(:perform_async)
|
2017-08-12 15:44:41 -07:00
|
|
|
allow(ActivityPub::DistributionWorker).to receive(:perform_async)
|
|
|
|
|
2017-04-07 10:48:38 -07:00
|
|
|
account = Fabricate(:account)
|
|
|
|
|
2023-02-18 14:38:14 -08:00
|
|
|
status = subject.call(account, text: 'test status update')
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
expect(DistributionWorker).to have_received(:perform_async).with(status.id)
|
2017-08-12 15:44:41 -07:00
|
|
|
expect(ActivityPub::DistributionWorker).to have_received(:perform_async).with(status.id)
|
2017-04-07 10:48:38 -07:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'crawls links' do
|
|
|
|
allow(LinkCrawlWorker).to receive(:perform_async)
|
|
|
|
account = Fabricate(:account)
|
|
|
|
|
2023-02-18 14:38:14 -08:00
|
|
|
status = subject.call(account, text: 'test status update')
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
expect(LinkCrawlWorker).to have_received(:perform_async).with(status.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'attaches the given media to the created status' do
|
|
|
|
account = Fabricate(:account)
|
2019-01-26 14:59:39 -08:00
|
|
|
media = Fabricate(:media_attachment, account: account)
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
status = subject.call(
|
2017-04-07 09:50:43 -07:00
|
|
|
account,
|
2023-02-18 14:38:14 -08:00
|
|
|
text: 'test status update',
|
2023-02-18 03:39:58 -08:00
|
|
|
media_ids: [media.id]
|
2017-04-07 09:50:43 -07:00
|
|
|
)
|
2017-04-07 10:48:38 -07:00
|
|
|
|
|
|
|
expect(media.reload.status).to eq status
|
|
|
|
end
|
|
|
|
|
2019-01-26 14:59:39 -08:00
|
|
|
it 'does not attach media from another account to the created status' do
|
|
|
|
account = Fabricate(:account)
|
|
|
|
media = Fabricate(:media_attachment, account: Fabricate(:account))
|
|
|
|
|
2023-10-19 07:55:06 -07:00
|
|
|
subject.call(
|
2019-01-26 14:59:39 -08:00
|
|
|
account,
|
2023-02-18 14:38:14 -08:00
|
|
|
text: 'test status update',
|
2023-02-18 03:39:58 -08:00
|
|
|
media_ids: [media.id]
|
2019-01-26 14:59:39 -08:00
|
|
|
)
|
|
|
|
|
2023-02-19 21:14:50 -08:00
|
|
|
expect(media.reload.status).to be_nil
|
2019-01-26 14:59:39 -08:00
|
|
|
end
|
|
|
|
|
2024-05-27 02:49:44 -07:00
|
|
|
it 'does not allow attaching more files than configured limit' do
|
|
|
|
stub_const('Status::MEDIA_ATTACHMENTS_LIMIT', 1)
|
2017-04-07 10:48:38 -07:00
|
|
|
account = Fabricate(:account)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
subject.call(
|
|
|
|
account,
|
2023-02-18 14:38:14 -08:00
|
|
|
text: 'test status update',
|
2024-05-27 02:49:44 -07:00
|
|
|
media_ids: Array.new(2) { Fabricate(:media_attachment, account: account) }.map(&:id)
|
2017-04-07 10:48:38 -07:00
|
|
|
)
|
2017-04-07 09:50:43 -07:00
|
|
|
end.to raise_error(
|
|
|
|
Mastodon::ValidationError,
|
2023-02-18 03:39:58 -08:00
|
|
|
I18n.t('media_attachments.validations.too_many')
|
2017-04-07 09:50:43 -07:00
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not allow attaching both videos and images' do
|
|
|
|
account = Fabricate(:account)
|
2020-01-23 12:40:03 -08:00
|
|
|
video = Fabricate(:media_attachment, type: :video, account: account)
|
|
|
|
image = Fabricate(:media_attachment, type: :image, account: account)
|
|
|
|
|
|
|
|
video.update(type: :video)
|
2017-04-07 09:50:43 -07:00
|
|
|
|
|
|
|
expect do
|
2017-04-07 10:48:38 -07:00
|
|
|
subject.call(
|
|
|
|
account,
|
2023-02-18 14:38:14 -08:00
|
|
|
text: 'test status update',
|
2017-04-07 10:48:38 -07:00
|
|
|
media_ids: [
|
2020-01-23 12:40:03 -08:00
|
|
|
video,
|
|
|
|
image,
|
2023-02-18 03:39:58 -08:00
|
|
|
].map(&:id)
|
2017-04-07 10:48:38 -07:00
|
|
|
)
|
2017-04-07 09:50:43 -07:00
|
|
|
end.to raise_error(
|
|
|
|
Mastodon::ValidationError,
|
2023-02-18 03:39:58 -08:00
|
|
|
I18n.t('media_attachments.validations.images_and_video')
|
2017-04-07 09:50:43 -07:00
|
|
|
)
|
|
|
|
end
|
2017-04-07 10:48:38 -07:00
|
|
|
|
2017-04-25 06:04:49 -07:00
|
|
|
it 'returns existing status when used twice with idempotency key' do
|
|
|
|
account = Fabricate(:account)
|
2019-01-05 03:43:28 -08:00
|
|
|
status1 = subject.call(account, text: 'test', idempotency: 'meepmeep')
|
|
|
|
status2 = subject.call(account, text: 'test', idempotency: 'meepmeep')
|
2017-04-25 06:04:49 -07:00
|
|
|
expect(status2.id).to eq status1.id
|
|
|
|
end
|
|
|
|
|
2017-12-06 02:41:57 -08:00
|
|
|
def create_status_with_options(**options)
|
2019-01-05 03:43:28 -08:00
|
|
|
subject.call(Fabricate(:account), options.merge(text: 'test'))
|
2017-04-07 10:48:38 -07:00
|
|
|
end
|
2016-02-24 15:17:01 -08:00
|
|
|
end
|