mirror of
https://github.com/kikobar/mastodon.git
synced 2024-12-30 17:54:13 +00:00
da2143b308
* Revert "Fix ignoring whole status because of one invalid hashtag (#11621)"
This reverts commit dff46b260b
.
* Fix statuses being rejected because of invalid hashtag names
* Add spec for invalid hashtag names in statuses
* Add test for featured tags controller
680 lines
18 KiB
Ruby
680 lines
18 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe ActivityPub::Activity::Create do
|
|
let(:sender) { Fabricate(:account, followers_url: 'http://example.com/followers', domain: 'example.com', uri: 'https://example.com/actor') }
|
|
|
|
let(:json) do
|
|
{
|
|
'@context': 'https://www.w3.org/ns/activitystreams',
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#foo'].join,
|
|
type: 'Create',
|
|
actor: ActivityPub::TagManager.instance.uri_for(sender),
|
|
object: object_json,
|
|
}.with_indifferent_access
|
|
end
|
|
|
|
before do
|
|
sender.update(uri: ActivityPub::TagManager.instance.uri_for(sender))
|
|
|
|
stub_request(:get, 'http://example.com/attachment.png').to_return(request_fixture('avatar.txt'))
|
|
stub_request(:get, 'http://example.com/emoji.png').to_return(body: attachment_fixture('emojo.png'))
|
|
end
|
|
|
|
describe '#perform' do
|
|
context 'when fetching' do
|
|
subject { described_class.new(json, sender) }
|
|
|
|
before do
|
|
subject.perform
|
|
end
|
|
|
|
context 'unknown object type' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Banana',
|
|
content: 'Lorem ipsum',
|
|
}
|
|
end
|
|
|
|
it 'does not create a status' do
|
|
expect(sender.statuses.count).to be_zero
|
|
end
|
|
end
|
|
|
|
context 'standalone' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.text).to eq 'Lorem ipsum'
|
|
end
|
|
|
|
it 'missing to/cc defaults to direct privacy' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.visibility).to eq 'direct'
|
|
end
|
|
end
|
|
|
|
context 'public' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
to: 'https://www.w3.org/ns/activitystreams#Public',
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.visibility).to eq 'public'
|
|
end
|
|
end
|
|
|
|
context 'unlisted' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
cc: 'https://www.w3.org/ns/activitystreams#Public',
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.visibility).to eq 'unlisted'
|
|
end
|
|
end
|
|
|
|
context 'private' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
to: 'http://example.com/followers',
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.visibility).to eq 'private'
|
|
end
|
|
end
|
|
|
|
context 'limited' do
|
|
let(:recipient) { Fabricate(:account) }
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
to: ActivityPub::TagManager.instance.uri_for(recipient),
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.visibility).to eq 'limited'
|
|
end
|
|
|
|
it 'creates silent mention' do
|
|
status = sender.statuses.first
|
|
expect(status.mentions.first).to be_silent
|
|
end
|
|
end
|
|
|
|
context 'direct' do
|
|
let(:recipient) { Fabricate(:account) }
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
to: ActivityPub::TagManager.instance.uri_for(recipient),
|
|
tag: {
|
|
type: 'Mention',
|
|
href: ActivityPub::TagManager.instance.uri_for(recipient),
|
|
},
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.visibility).to eq 'direct'
|
|
end
|
|
end
|
|
|
|
context 'as a reply' do
|
|
let(:original_status) { Fabricate(:status) }
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
inReplyTo: ActivityPub::TagManager.instance.uri_for(original_status),
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.thread).to eq original_status
|
|
expect(status.reply?).to be true
|
|
expect(status.in_reply_to_account).to eq original_status.account
|
|
expect(status.conversation).to eq original_status.conversation
|
|
end
|
|
end
|
|
|
|
context 'with mentions' do
|
|
let(:recipient) { Fabricate(:account) }
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
tag: [
|
|
{
|
|
type: 'Mention',
|
|
href: ActivityPub::TagManager.instance.uri_for(recipient),
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.mentions.map(&:account)).to include(recipient)
|
|
end
|
|
end
|
|
|
|
context 'with mentions missing href' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
tag: [
|
|
{
|
|
type: 'Mention',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
expect(status).to_not be_nil
|
|
end
|
|
end
|
|
|
|
context 'with media attachments' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
attachment: [
|
|
{
|
|
type: 'Document',
|
|
mediaType: 'image/png',
|
|
url: 'http://example.com/attachment.png',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.media_attachments.map(&:remote_url)).to include('http://example.com/attachment.png')
|
|
end
|
|
end
|
|
|
|
|
|
context 'with media attachments with long description' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
attachment: [
|
|
{
|
|
type: 'Document',
|
|
mediaType: 'image/png',
|
|
url: 'http://example.com/attachment.png',
|
|
name: '*' * 1500,
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.media_attachments.map(&:description)).to include('*' * 1500)
|
|
end
|
|
end
|
|
|
|
context 'with media attachments with focal points' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
attachment: [
|
|
{
|
|
type: 'Document',
|
|
mediaType: 'image/png',
|
|
url: 'http://example.com/attachment.png',
|
|
focalPoint: [0.5, -0.7],
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.media_attachments.map(&:focus)).to include('0.5,-0.7')
|
|
end
|
|
end
|
|
|
|
context 'with media attachments missing url' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
attachment: [
|
|
{
|
|
type: 'Document',
|
|
mediaType: 'image/png',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
expect(status).to_not be_nil
|
|
end
|
|
end
|
|
|
|
context 'with hashtags' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
tag: [
|
|
{
|
|
type: 'Hashtag',
|
|
href: 'http://example.com/blah',
|
|
name: '#test',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.tags.map(&:name)).to include('test')
|
|
end
|
|
end
|
|
|
|
context 'with hashtags missing name' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
tag: [
|
|
{
|
|
type: 'Hashtag',
|
|
href: 'http://example.com/blah',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
expect(status).to_not be_nil
|
|
end
|
|
end
|
|
|
|
context 'with hashtags invalid name' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
tag: [
|
|
{
|
|
type: 'Hashtag',
|
|
href: 'http://example.com/blah',
|
|
name: 'foo, #eh !',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
expect(status).to_not be_nil
|
|
end
|
|
end
|
|
|
|
context 'with emojis' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum :tinking:',
|
|
tag: [
|
|
{
|
|
type: 'Emoji',
|
|
icon: {
|
|
url: 'http://example.com/emoji.png',
|
|
},
|
|
name: 'tinking',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.emojis.map(&:shortcode)).to include('tinking')
|
|
end
|
|
end
|
|
|
|
context 'with emojis missing name' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum :tinking:',
|
|
tag: [
|
|
{
|
|
type: 'Emoji',
|
|
icon: {
|
|
url: 'http://example.com/emoji.png',
|
|
},
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
expect(status).to_not be_nil
|
|
end
|
|
end
|
|
|
|
context 'with emojis missing icon' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum :tinking:',
|
|
tag: [
|
|
{
|
|
type: 'Emoji',
|
|
name: 'tinking',
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
expect(status).to_not be_nil
|
|
end
|
|
end
|
|
|
|
context 'with poll' do
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Question',
|
|
content: 'Which color was the submarine?',
|
|
oneOf: [
|
|
{
|
|
name: 'Yellow',
|
|
replies: {
|
|
type: 'Collection',
|
|
totalItems: 10,
|
|
},
|
|
},
|
|
{
|
|
name: 'Blue',
|
|
replies: {
|
|
type: 'Collection',
|
|
totalItems: 3,
|
|
}
|
|
},
|
|
],
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
expect(status).to_not be_nil
|
|
expect(status.poll).to_not be_nil
|
|
end
|
|
|
|
it 'creates a poll' do
|
|
poll = sender.polls.first
|
|
expect(poll).to_not be_nil
|
|
expect(poll.status).to_not be_nil
|
|
expect(poll.options).to eq %w(Yellow Blue)
|
|
expect(poll.cached_tallies).to eq [10, 3]
|
|
end
|
|
end
|
|
|
|
context 'when a vote to a local poll' do
|
|
let(:poll) { Fabricate(:poll, options: %w(Yellow Blue)) }
|
|
let!(:local_status) { Fabricate(:status, poll: poll) }
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
name: 'Yellow',
|
|
inReplyTo: ActivityPub::TagManager.instance.uri_for(local_status)
|
|
}
|
|
end
|
|
|
|
it 'adds a vote to the poll with correct uri' do
|
|
vote = poll.votes.first
|
|
expect(vote).to_not be_nil
|
|
expect(vote.uri).to eq object_json[:id]
|
|
expect(poll.reload.cached_tallies).to eq [1, 0]
|
|
end
|
|
end
|
|
|
|
context 'when a vote to an expired local poll' do
|
|
let(:poll) do
|
|
poll = Fabricate.build(:poll, options: %w(Yellow Blue), expires_at: 1.day.ago)
|
|
poll.save(validate: false)
|
|
poll
|
|
end
|
|
let!(:local_status) { Fabricate(:status, poll: poll) }
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
name: 'Yellow',
|
|
inReplyTo: ActivityPub::TagManager.instance.uri_for(local_status)
|
|
}
|
|
end
|
|
|
|
it 'does not add a vote to the poll' do
|
|
expect(poll.votes.first).to be_nil
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'when sender is followed by local users' do
|
|
subject { described_class.new(json, sender, delivery: true) }
|
|
|
|
before do
|
|
Fabricate(:account).follow!(sender)
|
|
subject.perform
|
|
end
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.text).to eq 'Lorem ipsum'
|
|
end
|
|
end
|
|
|
|
context 'when sender replies to local status' do
|
|
let!(:local_status) { Fabricate(:status) }
|
|
|
|
subject { described_class.new(json, sender, delivery: true) }
|
|
|
|
before do
|
|
subject.perform
|
|
end
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
inReplyTo: ActivityPub::TagManager.instance.uri_for(local_status),
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.text).to eq 'Lorem ipsum'
|
|
end
|
|
end
|
|
|
|
context 'when sender targets a local user' do
|
|
let!(:local_account) { Fabricate(:account) }
|
|
|
|
subject { described_class.new(json, sender, delivery: true) }
|
|
|
|
before do
|
|
subject.perform
|
|
end
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
to: ActivityPub::TagManager.instance.uri_for(local_account),
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.text).to eq 'Lorem ipsum'
|
|
end
|
|
end
|
|
|
|
context 'when sender cc\'s a local user' do
|
|
let!(:local_account) { Fabricate(:account) }
|
|
|
|
subject { described_class.new(json, sender, delivery: true) }
|
|
|
|
before do
|
|
subject.perform
|
|
end
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
cc: ActivityPub::TagManager.instance.uri_for(local_account),
|
|
}
|
|
end
|
|
|
|
it 'creates status' do
|
|
status = sender.statuses.first
|
|
|
|
expect(status).to_not be_nil
|
|
expect(status.text).to eq 'Lorem ipsum'
|
|
end
|
|
end
|
|
|
|
context 'when the sender has no relevance to local activity' do
|
|
subject { described_class.new(json, sender, delivery: true) }
|
|
|
|
before do
|
|
subject.perform
|
|
end
|
|
|
|
let(:object_json) do
|
|
{
|
|
id: [ActivityPub::TagManager.instance.uri_for(sender), '#bar'].join,
|
|
type: 'Note',
|
|
content: 'Lorem ipsum',
|
|
}
|
|
end
|
|
|
|
it 'does not create anything' do
|
|
expect(sender.statuses.count).to eq 0
|
|
end
|
|
end
|
|
end
|
|
end
|