123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053 |
- # frozen_string_literal: true
- require 'rails_helper'
- RSpec.describe Account do
- context 'with an account record' do
- subject { Fabricate(:account) }
- let(:bob) { Fabricate(:account, username: 'bob') }
- describe '#suspend!' do
- it 'marks the account as suspended and creates a deletion request' do
- expect { subject.suspend! }
- .to change(subject, :suspended?).from(false).to(true)
- .and(change { AccountDeletionRequest.exists?(account: subject) }.from(false).to(true))
- end
- context 'when the account is of a local user' do
- subject { local_user_account }
- let!(:local_user_account) { Fabricate(:user, email: 'foo+bar@domain.org').account }
- it 'creates a canonical domain block' do
- subject.suspend!
- expect(CanonicalEmailBlock.block?(subject.user_email)).to be true
- end
- context 'when a canonical domain block already exists for that email' do
- before do
- Fabricate(:canonical_email_block, email: subject.user_email)
- end
- it 'does not raise an error' do
- expect { subject.suspend! }.to_not raise_error
- end
- end
- end
- end
- describe '#follow!' do
- it 'creates a follow' do
- follow = subject.follow!(bob)
- expect(follow).to be_instance_of Follow
- expect(follow.account).to eq subject
- expect(follow.target_account).to eq bob
- end
- end
- describe '#unfollow!' do
- before do
- subject.follow!(bob)
- end
- it 'destroys a follow' do
- unfollow = subject.unfollow!(bob)
- expect(unfollow).to be_instance_of Follow
- expect(unfollow.account).to eq subject
- expect(unfollow.target_account).to eq bob
- expect(unfollow.destroyed?).to be true
- end
- end
- describe '#following?' do
- it 'returns true when the target is followed' do
- subject.follow!(bob)
- expect(subject.following?(bob)).to be true
- end
- it 'returns false if the target is not followed' do
- expect(subject.following?(bob)).to be false
- end
- end
- end
- describe '#local?' do
- it 'returns true when the account is local' do
- account = Fabricate(:account, domain: nil)
- expect(account.local?).to be true
- end
- it 'returns false when the account is on a different domain' do
- account = Fabricate(:account, domain: 'foreign.tld')
- expect(account.local?).to be false
- end
- end
- describe 'Local domain user methods' do
- subject { Fabricate(:account, domain: nil, username: 'alice') }
- around do |example|
- before = Rails.configuration.x.local_domain
- example.run
- Rails.configuration.x.local_domain = before
- end
- describe '#to_webfinger_s' do
- it 'returns a webfinger string for the account' do
- Rails.configuration.x.local_domain = 'example.com'
- expect(subject.to_webfinger_s).to eq 'acct:alice@example.com'
- end
- end
- describe '#local_username_and_domain' do
- it 'returns the username and local domain for the account' do
- Rails.configuration.x.local_domain = 'example.com'
- expect(subject.local_username_and_domain).to eq 'alice@example.com'
- end
- end
- end
- describe '#acct' do
- it 'returns username for local users' do
- account = Fabricate(:account, domain: nil, username: 'alice')
- expect(account.acct).to eql 'alice'
- end
- it 'returns username@domain for foreign users' do
- account = Fabricate(:account, domain: 'foreign.tld', username: 'alice')
- expect(account.acct).to eql 'alice@foreign.tld'
- end
- end
- describe '#save_with_optional_media!' do
- before do
- stub_request(:get, 'https://remote.test/valid_avatar').to_return(request_fixture('avatar.txt'))
- stub_request(:get, 'https://remote.test/invalid_avatar').to_return(request_fixture('feed.txt'))
- end
- let(:account) do
- Fabricate(:account,
- avatar_remote_url: 'https://remote.test/valid_avatar',
- header_remote_url: 'https://remote.test/valid_avatar')
- end
- let!(:expectation) { account.dup }
- context 'with valid properties' do
- before do
- account.save_with_optional_media!
- end
- it 'unchanges avatar, header, avatar_remote_url, and header_remote_url' do
- expect(account.avatar_remote_url).to eq expectation.avatar_remote_url
- expect(account.header_remote_url).to eq expectation.header_remote_url
- expect(account.avatar_file_name).to eq expectation.avatar_file_name
- expect(account.header_file_name).to eq expectation.header_file_name
- end
- end
- context 'with invalid properties' do
- before do
- account.avatar_remote_url = 'https://remote.test/invalid_avatar'
- account.save_with_optional_media!
- end
- it 'sets default avatar, header, avatar_remote_url, and header_remote_url' do
- expect(account.avatar_remote_url).to eq 'https://remote.test/invalid_avatar'
- expect(account.header_remote_url).to eq expectation.header_remote_url
- expect(account.avatar_file_name).to be_nil
- expect(account.header_file_name).to eq expectation.header_file_name
- end
- end
- end
- describe '#possibly_stale?' do
- let(:account) { Fabricate(:account, last_webfingered_at: last_webfingered_at) }
- context 'when last_webfingered_at is nil' do
- let(:last_webfingered_at) { nil }
- it 'returns true' do
- expect(account.possibly_stale?).to be true
- end
- end
- context 'when last_webfingered_at is more than 24 hours before' do
- let(:last_webfingered_at) { 25.hours.ago }
- it 'returns true' do
- expect(account.possibly_stale?).to be true
- end
- end
- context 'when last_webfingered_at is less than 24 hours before' do
- let(:last_webfingered_at) { 23.hours.ago }
- it 'returns false' do
- expect(account.possibly_stale?).to be false
- end
- end
- end
- describe '#refresh!' do
- let(:account) { Fabricate(:account, domain: domain) }
- let(:acct) { account.acct }
- context 'when domain is nil' do
- let(:domain) { nil }
- it 'returns nil' do
- expect(account.refresh!).to be_nil
- end
- it 'does not call ResolveAccountService#call' do
- service = instance_double(ResolveAccountService, call: nil)
- allow(ResolveAccountService).to receive(:new).and_return(service)
- account.refresh!
- expect(service).to_not have_received(:call).with(acct)
- end
- end
- context 'when domain is present' do
- let(:domain) { 'example.com' }
- it 'calls ResolveAccountService#call' do
- service = instance_double(ResolveAccountService, call: nil)
- allow(ResolveAccountService).to receive(:new).and_return(service)
- account.refresh!
- expect(service).to have_received(:call).with(acct).once
- end
- end
- end
- describe '#to_param' do
- it 'returns username' do
- account = Fabricate(:account, username: 'alice')
- expect(account.to_param).to eq 'alice'
- end
- end
- describe '#keypair' do
- it 'returns an RSA key pair' do
- account = Fabricate(:account)
- expect(account.keypair).to be_instance_of OpenSSL::PKey::RSA
- end
- end
- describe '#object_type' do
- it 'is always a person' do
- account = Fabricate(:account)
- expect(account.object_type).to be :person
- end
- end
- describe '#favourited?' do
- subject { Fabricate(:account) }
- let(:original_status) do
- author = Fabricate(:account, username: 'original')
- Fabricate(:status, account: author)
- end
- context 'when the status is a reblog of another status' do
- let(:original_reblog) do
- author = Fabricate(:account, username: 'original_reblogger')
- Fabricate(:status, reblog: original_status, account: author)
- end
- it 'is true when this account has favourited it' do
- Fabricate(:favourite, status: original_reblog, account: subject)
- expect(subject.favourited?(original_status)).to be true
- end
- it 'is false when this account has not favourited it' do
- expect(subject.favourited?(original_status)).to be false
- end
- end
- context 'when the status is an original status' do
- it 'is true when this account has favourited it' do
- Fabricate(:favourite, status: original_status, account: subject)
- expect(subject.favourited?(original_status)).to be true
- end
- it 'is false when this account has not favourited it' do
- expect(subject.favourited?(original_status)).to be false
- end
- end
- end
- describe '#reblogged?' do
- subject { Fabricate(:account) }
- let(:original_status) do
- author = Fabricate(:account, username: 'original')
- Fabricate(:status, account: author)
- end
- context 'when the status is a reblog of another status' do
- let(:original_reblog) do
- author = Fabricate(:account, username: 'original_reblogger')
- Fabricate(:status, reblog: original_status, account: author)
- end
- it 'is true when this account has reblogged it' do
- Fabricate(:status, reblog: original_reblog, account: subject)
- expect(subject.reblogged?(original_reblog)).to be true
- end
- it 'is false when this account has not reblogged it' do
- expect(subject.reblogged?(original_reblog)).to be false
- end
- end
- context 'when the status is an original status' do
- it 'is true when this account has reblogged it' do
- Fabricate(:status, reblog: original_status, account: subject)
- expect(subject.reblogged?(original_status)).to be true
- end
- it 'is false when this account has not reblogged it' do
- expect(subject.reblogged?(original_status)).to be false
- end
- end
- end
- describe '#excluded_from_timeline_account_ids' do
- it 'includes account ids of blockings, blocked_bys and mutes' do
- account = Fabricate(:account)
- block = Fabricate(:block, account: account)
- mute = Fabricate(:mute, account: account)
- block_by = Fabricate(:block, target_account: account)
- results = account.excluded_from_timeline_account_ids
- expect(results.size).to eq 3
- expect(results).to include(
- block.target_account.id,
- mute.target_account.id,
- block_by.account.id
- )
- end
- end
- describe '#excluded_from_timeline_domains' do
- it 'returns the domains blocked by the account' do
- account = Fabricate(:account)
- account.block_domain!('domain')
- expect(account.excluded_from_timeline_domains).to contain_exactly('domain')
- end
- end
- describe '.search_for' do
- before do
- _missing = Fabricate(
- :account,
- display_name: 'Missing',
- username: 'missing',
- domain: 'missing.com'
- )
- end
- it 'does not return suspended users' do
- Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username',
- domain: 'example.com',
- suspended: true
- )
- results = described_class.search_for('username')
- expect(results).to eq []
- end
- it 'does not return unapproved users' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username'
- )
- match.user.update(approved: false)
- results = described_class.search_for('username')
- expect(results).to eq []
- end
- it 'does not return unconfirmed users' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username'
- )
- match.user.update(confirmed_at: nil)
- results = described_class.search_for('username')
- expect(results).to eq []
- end
- it 'accepts ?, \, : and space as delimiter' do
- match = Fabricate(
- :account,
- display_name: 'A & l & i & c & e',
- username: 'username',
- domain: 'example.com'
- )
- results = described_class.search_for('A?l\i:c e')
- expect(results).to eq [match]
- end
- it 'finds accounts with matching display_name' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username',
- domain: 'example.com'
- )
- results = described_class.search_for('display')
- expect(results).to eq [match]
- end
- it 'finds accounts with matching username' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username',
- domain: 'example.com'
- )
- results = described_class.search_for('username')
- expect(results).to eq [match]
- end
- it 'finds accounts with matching domain' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username',
- domain: 'example.com'
- )
- results = described_class.search_for('example')
- expect(results).to eq [match]
- end
- it 'limits via constant by default' do
- stub_const('Account::Search::DEFAULT_LIMIT', 1)
- 2.times.each { Fabricate(:account, display_name: 'Display Name') }
- results = described_class.search_for('display')
- expect(results.size).to eq 1
- end
- it 'accepts arbitrary limits' do
- 2.times.each { Fabricate(:account, display_name: 'Display Name') }
- results = described_class.search_for('display', limit: 1)
- expect(results.size).to eq 1
- end
- it 'ranks multiple matches higher' do
- matches = [
- { username: 'username', display_name: 'username' },
- { display_name: 'Display Name', username: 'username', domain: 'example.com' },
- ].map(&method(:Fabricate).curry(2).call(:account))
- results = described_class.search_for('username')
- expect(results).to eq matches
- end
- end
- describe '.advanced_search_for' do
- let(:account) { Fabricate(:account) }
- context 'when limiting search to followed accounts' do
- it 'accepts ?, \, : and space as delimiter' do
- match = Fabricate(
- :account,
- display_name: 'A & l & i & c & e',
- username: 'username',
- domain: 'example.com'
- )
- account.follow!(match)
- results = described_class.advanced_search_for('A?l\i:c e', account, limit: 10, following: true)
- expect(results).to eq [match]
- end
- it 'does not return non-followed accounts' do
- Fabricate(
- :account,
- display_name: 'A & l & i & c & e',
- username: 'username',
- domain: 'example.com'
- )
- results = described_class.advanced_search_for('A?l\i:c e', account, limit: 10, following: true)
- expect(results).to eq []
- end
- it 'does not return suspended users' do
- Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username',
- domain: 'example.com',
- suspended: true
- )
- results = described_class.advanced_search_for('username', account, limit: 10, following: true)
- expect(results).to eq []
- end
- it 'does not return unapproved users' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username'
- )
- match.user.update(approved: false)
- results = described_class.advanced_search_for('username', account, limit: 10, following: true)
- expect(results).to eq []
- end
- it 'does not return unconfirmed users' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username'
- )
- match.user.update(confirmed_at: nil)
- results = described_class.advanced_search_for('username', account, limit: 10, following: true)
- expect(results).to eq []
- end
- end
- it 'does not return suspended users' do
- Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username',
- domain: 'example.com',
- suspended: true
- )
- results = described_class.advanced_search_for('username', account)
- expect(results).to eq []
- end
- it 'does not return unapproved users' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username'
- )
- match.user.update(approved: false)
- results = described_class.advanced_search_for('username', account)
- expect(results).to eq []
- end
- it 'does not return unconfirmed users' do
- match = Fabricate(
- :account,
- display_name: 'Display Name',
- username: 'username'
- )
- match.user.update(confirmed_at: nil)
- results = described_class.advanced_search_for('username', account)
- expect(results).to eq []
- end
- it 'accepts ?, \, : and space as delimiter' do
- match = Fabricate(
- :account,
- display_name: 'A & l & i & c & e',
- username: 'username',
- domain: 'example.com'
- )
- results = described_class.advanced_search_for('A?l\i:c e', account)
- expect(results).to eq [match]
- end
- it 'limits by 10 by default' do
- stub_const('Account::Search::DEFAULT_LIMIT', 1)
- 2.times { Fabricate(:account, display_name: 'Display Name') }
- results = described_class.advanced_search_for('display', account)
- expect(results.size).to eq 1
- end
- it 'accepts arbitrary limits' do
- 2.times { Fabricate(:account, display_name: 'Display Name') }
- results = described_class.advanced_search_for('display', account, limit: 1)
- expect(results.size).to eq 1
- end
- it 'ranks followed accounts higher' do
- match = Fabricate(:account, username: 'Matching')
- followed_match = Fabricate(:account, username: 'Matcher')
- Fabricate(:follow, account: account, target_account: followed_match)
- results = described_class.advanced_search_for('match', account)
- expect(results).to eq [followed_match, match]
- expect(results.first.rank).to be > results.last.rank
- end
- end
- describe '#statuses_count' do
- subject { Fabricate(:account) }
- it 'counts statuses' do
- Fabricate(:status, account: subject)
- Fabricate(:status, account: subject)
- expect(subject.statuses_count).to eq 2
- end
- it 'does not count direct statuses' do
- Fabricate(:status, account: subject, visibility: :direct)
- expect(subject.statuses_count).to eq 0
- end
- it 'is decremented when status is removed' do
- status = Fabricate(:status, account: subject)
- expect(subject.statuses_count).to eq 1
- status.destroy
- expect(subject.statuses_count).to eq 0
- end
- it 'is decremented when status is removed when account is not preloaded' do
- status = Fabricate(:status, account: subject)
- expect(subject.reload.statuses_count).to eq 1
- clean_status = Status.find(status.id)
- expect(clean_status.association(:account).loaded?).to be false
- clean_status.destroy
- expect(subject.reload.statuses_count).to eq 0
- end
- end
- describe '.following_map' do
- it 'returns an hash' do
- expect(described_class.following_map([], 1)).to be_a Hash
- end
- end
- describe '.followed_by_map' do
- it 'returns an hash' do
- expect(described_class.followed_by_map([], 1)).to be_a Hash
- end
- end
- describe '.blocking_map' do
- it 'returns an hash' do
- expect(described_class.blocking_map([], 1)).to be_a Hash
- end
- end
- describe '.requested_map' do
- it 'returns an hash' do
- expect(described_class.requested_map([], 1)).to be_a Hash
- end
- end
- describe '.requested_by_map' do
- it 'returns an hash' do
- expect(described_class.requested_by_map([], 1)).to be_a Hash
- end
- end
- describe 'MENTION_RE' do
- subject { described_class::MENTION_RE }
- it 'matches usernames in the middle of a sentence' do
- expect(subject.match('Hello to @alice from me')[1]).to eq 'alice'
- end
- it 'matches usernames in the beginning of status' do
- expect(subject.match('@alice Hey how are you?')[1]).to eq 'alice'
- end
- it 'matches full usernames' do
- expect(subject.match('@alice@example.com')[1]).to eq 'alice@example.com'
- end
- it 'matches full usernames with a dot at the end' do
- expect(subject.match('Hello @alice@example.com.')[1]).to eq 'alice@example.com'
- end
- it 'matches dot-prepended usernames' do
- expect(subject.match('.@alice I want everybody to see this')[1]).to eq 'alice'
- end
- it 'does not match e-mails' do
- expect(subject.match('Drop me an e-mail at alice@example.com')).to be_nil
- end
- it 'does not match URLs' do
- expect(subject.match('Check this out https://medium.com/@alice/some-article#.abcdef123')).to be_nil
- end
- it 'does not match URL query string' do
- expect(subject.match('https://example.com/?x=@alice')).to be_nil
- end
- end
- describe 'validations' do
- it 'is invalid without a username' do
- account = Fabricate.build(:account, username: nil)
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'squishes the username before validation' do
- account = Fabricate(:account, domain: nil, username: " \u3000bob \t \u00a0 \n ")
- expect(account.username).to eq 'bob'
- end
- context 'when is local' do
- it 'is invalid if the username is not unique in case-insensitive comparison among local accounts' do
- _account = Fabricate(:account, username: 'the_doctor')
- non_unique_account = Fabricate.build(:account, username: 'the_Doctor')
- non_unique_account.valid?
- expect(non_unique_account).to model_have_error_on_field(:username)
- end
- it 'is invalid if the username is reserved' do
- account = Fabricate.build(:account, username: 'support')
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'is valid when username is reserved but record has already been created' do
- account = Fabricate.build(:account, username: 'support')
- account.save(validate: false)
- expect(account.valid?).to be true
- end
- it 'is valid if we are creating an instance actor account with a period' do
- account = Fabricate.build(:account, id: described_class::INSTANCE_ACTOR_ID, actor_type: 'Application', locked: true, username: 'example.com')
- expect(account.valid?).to be true
- end
- it 'is valid if we are creating a possibly-conflicting instance actor account' do
- _account = Fabricate(:account, username: 'examplecom')
- instance_account = Fabricate.build(:account, id: described_class::INSTANCE_ACTOR_ID, actor_type: 'Application', locked: true, username: 'example.com')
- expect(instance_account.valid?).to be true
- end
- it 'is invalid if the username doesn\'t only contains letters, numbers and underscores' do
- account = Fabricate.build(:account, username: 'the-doctor')
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'is invalid if the username contains a period' do
- account = Fabricate.build(:account, username: 'the.doctor')
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'is invalid if the username is longer than the character limit' do
- account = Fabricate.build(:account, username: username_over_limit)
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'is invalid if the display name is longer than the character limit' do
- account = Fabricate.build(:account, display_name: username_over_limit)
- account.valid?
- expect(account).to model_have_error_on_field(:display_name)
- end
- it 'is invalid if the note is longer than the character limit' do
- account = Fabricate.build(:account, note: account_note_over_limit)
- account.valid?
- expect(account).to model_have_error_on_field(:note)
- end
- end
- context 'when is remote' do
- it 'is invalid if the username is same among accounts in the same normalized domain' do
- Fabricate(:account, domain: 'にゃん', username: 'username')
- account = Fabricate.build(:account, domain: 'xn--r9j5b5b', username: 'username')
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'is invalid if the username is not unique in case-insensitive comparison among accounts in the same normalized domain' do
- Fabricate(:account, domain: 'にゃん', username: 'username')
- account = Fabricate.build(:account, domain: 'xn--r9j5b5b', username: 'Username')
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'is valid even if the username contains hyphens' do
- account = Fabricate.build(:account, domain: 'domain', username: 'the-doctor')
- account.valid?
- expect(account).to_not model_have_error_on_field(:username)
- end
- it 'is invalid if the username doesn\'t only contains letters, numbers, underscores and hyphens' do
- account = Fabricate.build(:account, domain: 'domain', username: 'the doctor')
- account.valid?
- expect(account).to model_have_error_on_field(:username)
- end
- it 'is valid even if the username is longer than the character limit' do
- account = Fabricate.build(:account, domain: 'domain', username: username_over_limit)
- account.valid?
- expect(account).to_not model_have_error_on_field(:username)
- end
- it 'is valid even if the display name is longer than the character limit' do
- account = Fabricate.build(:account, domain: 'domain', display_name: username_over_limit)
- account.valid?
- expect(account).to_not model_have_error_on_field(:display_name)
- end
- it 'is valid even if the note is longer than the character limit' do
- account = Fabricate.build(:account, domain: 'domain', note: account_note_over_limit)
- account.valid?
- expect(account).to_not model_have_error_on_field(:note)
- end
- end
- def username_over_limit
- 'a' * described_class::USERNAME_LENGTH_LIMIT * 2
- end
- def account_note_over_limit
- 'a' * described_class::NOTE_LENGTH_LIMIT * 2
- end
- end
- describe 'scopes' do
- describe 'matches_uri_prefix' do
- let!(:alice) { Fabricate :account, domain: 'host.example', uri: 'https://host.example/user/a' }
- let!(:bob) { Fabricate :account, domain: 'top-level.example', uri: 'https://top-level.example' }
- it 'returns accounts which start with the value' do
- results = described_class.matches_uri_prefix('https://host.example')
- expect(results.size)
- .to eq(1)
- expect(results)
- .to include(alice)
- .and not_include(bob)
- end
- it 'returns accounts which equal the value' do
- results = described_class.matches_uri_prefix('https://top-level.example')
- expect(results.size)
- .to eq(1)
- expect(results)
- .to include(bob)
- .and not_include(alice)
- end
- end
- describe 'auditable' do
- let!(:alice) { Fabricate :account }
- let!(:bob) { Fabricate :account }
- before do
- 2.times { Fabricate :action_log, account: alice }
- end
- it 'returns distinct accounts with action log records' do
- results = described_class.auditable
- expect(results.size)
- .to eq(1)
- expect(results)
- .to include(alice)
- .and not_include(bob)
- end
- end
- describe 'alphabetic' do
- it 'sorts by alphabetic order of domain and username' do
- matches = [
- { username: 'a', domain: 'a' },
- { username: 'b', domain: 'a' },
- { username: 'a', domain: 'b' },
- { username: 'b', domain: 'b' },
- ].map(&method(:Fabricate).curry(2).call(:account))
- expect(described_class.without_internal.alphabetic).to eq matches
- end
- end
- describe 'matches_display_name' do
- it 'matches display name which starts with the given string' do
- match = Fabricate(:account, display_name: 'pattern and suffix')
- Fabricate(:account, display_name: 'prefix and pattern')
- expect(described_class.matches_display_name('pattern')).to eq [match]
- end
- end
- describe 'matches_username' do
- it 'matches display name which starts with the given string' do
- match = Fabricate(:account, username: 'pattern_and_suffix')
- Fabricate(:account, username: 'prefix_and_pattern')
- expect(described_class.matches_username('pattern')).to eq [match]
- end
- end
- describe 'by_domain_and_subdomains' do
- it 'returns exact domain matches' do
- account = Fabricate(:account, domain: 'example.com')
- expect(described_class.by_domain_and_subdomains('example.com')).to eq [account]
- end
- it 'returns subdomains' do
- account = Fabricate(:account, domain: 'foo.example.com')
- expect(described_class.by_domain_and_subdomains('example.com')).to eq [account]
- end
- it 'does not return partially matching domains' do
- account = Fabricate(:account, domain: 'grexample.com')
- expect(described_class.by_domain_and_subdomains('example.com')).to_not eq [account]
- end
- end
- describe 'remote' do
- it 'returns an array of accounts who have a domain' do
- _account = Fabricate(:account, domain: nil)
- account_with_domain = Fabricate(:account, domain: 'example.com')
- expect(described_class.remote).to contain_exactly(account_with_domain)
- end
- end
- describe 'local' do
- it 'returns an array of accounts who do not have a domain' do
- local_account = Fabricate(:account, domain: nil)
- _account_with_domain = Fabricate(:account, domain: 'example.com')
- expect(described_class.without_internal.local).to contain_exactly(local_account)
- end
- end
- describe 'partitioned' do
- it 'returns a relation of accounts partitioned by domain' do
- matches = %w(a b a b)
- matches.size.times.to_a.shuffle.each do |index|
- matches[index] = Fabricate(:account, domain: matches[index])
- end
- expect(described_class.without_internal.partitioned).to match_array(matches)
- end
- end
- describe 'recent' do
- it 'returns a relation of accounts sorted by recent creation' do
- matches = Array.new(2) { Fabricate(:account) }
- expect(described_class.without_internal.recent).to match_array(matches)
- end
- end
- describe 'silenced' do
- it 'returns an array of accounts who are silenced' do
- silenced_account = Fabricate(:account, silenced: true)
- _account = Fabricate(:account, silenced: false)
- expect(described_class.silenced).to contain_exactly(silenced_account)
- end
- end
- describe 'suspended' do
- it 'returns an array of accounts who are suspended' do
- suspended_account = Fabricate(:account, suspended: true)
- _account = Fabricate(:account, suspended: false)
- expect(described_class.suspended).to contain_exactly(suspended_account)
- end
- end
- describe 'searchable' do
- let!(:suspended_local) { Fabricate(:account, suspended: true, username: 'suspended_local') }
- let!(:suspended_remote) { Fabricate(:account, suspended: true, domain: 'example.org', username: 'suspended_remote') }
- let!(:silenced_local) { Fabricate(:account, silenced: true, username: 'silenced_local') }
- let!(:silenced_remote) { Fabricate(:account, silenced: true, domain: 'example.org', username: 'silenced_remote') }
- let!(:unconfirmed) { Fabricate(:user, confirmed_at: nil).account }
- let!(:unapproved) { Fabricate(:user, approved: false).account }
- let!(:unconfirmed_unapproved) { Fabricate(:user, confirmed_at: nil, approved: false).account }
- let!(:local_account) { Fabricate(:account, username: 'local_account') }
- let!(:remote_account) { Fabricate(:account, domain: 'example.org', username: 'remote_account') }
- before do
- # Accounts get automatically-approved depending on settings, so ensure they aren't approved
- unapproved.user.update(approved: false)
- unconfirmed_unapproved.user.update(approved: false)
- end
- it 'returns every usable non-suspended account' do
- expect(described_class.searchable).to contain_exactly(silenced_local, silenced_remote, local_account, remote_account)
- expect(described_class.searchable).to_not include(suspended_local, suspended_remote, unconfirmed, unapproved)
- end
- it 'does not mess with previously-applied scopes' do
- expect(described_class.where.not(id: remote_account.id).searchable).to contain_exactly(silenced_local, silenced_remote, local_account)
- end
- end
- end
- context 'when is local' do
- it 'generates keys' do
- account = described_class.create!(domain: nil, username: Faker::Internet.user_name(separators: ['_']))
- expect(account.keypair).to be_private
- expect(account.keypair).to be_public
- end
- end
- context 'when is remote' do
- it 'does not generate keys' do
- key = OpenSSL::PKey::RSA.new(1024).public_key
- account = described_class.create!(domain: 'remote', uri: 'https://remote/actor', username: Faker::Internet.user_name(separators: ['_']), public_key: key.to_pem)
- expect(account.keypair.params).to eq key.params
- end
- it 'normalizes domain' do
- account = described_class.create!(domain: 'にゃん', uri: 'https://xn--r9j5b5b/actor', username: Faker::Internet.user_name(separators: ['_']))
- expect(account.domain).to eq 'xn--r9j5b5b'
- end
- end
- include_examples 'AccountAvatar', :account
- include_examples 'AccountHeader', :account
- describe '#increment_count!' do
- subject { Fabricate(:account) }
- it 'increments the count in multi-threaded an environment when account_stat is not yet initialized' do
- subject
- multi_threaded_execution(15) do
- described_class.find(subject.id).increment_count!(:followers_count)
- end
- expect(subject.reload.followers_count).to eq 15
- end
- end
- end
|