diff options
author | Claire <claire.github-309c@sitedethib.com> | 2022-12-07 09:03:42 +0100 |
---|---|---|
committer | Claire <claire.github-309c@sitedethib.com> | 2022-12-07 09:03:42 +0100 |
commit | 9a3d91f629ae0e8f722d56b8634bfad299ab9f04 (patch) | |
tree | 324a2b04d34049d7f31a041d6a44b23d45a46e6b /spec/models | |
parent | fe523a304520a09f6371f45bd63b9e8988776c03 (diff) | |
parent | b59fb28e90bc21d6fd1a6bafd13cfbd81ab5be54 (diff) |
Merge branch 'main' into glitch-soc/merge-upstream
Conflicts: - `app/models/concerns/domain_materializable.rb`: Fixed a code style issue upstream in a PR that got merged in glitch-soc earlier. Changed the code to match upstream's.
Diffstat (limited to 'spec/models')
-rw-r--r-- | spec/models/account_migration_spec.rb | 43 |
1 files changed, 43 insertions, 0 deletions
diff --git a/spec/models/account_migration_spec.rb b/spec/models/account_migration_spec.rb index 8461b4b28..5f66fe8da 100644 --- a/spec/models/account_migration_spec.rb +++ b/spec/models/account_migration_spec.rb @@ -1,5 +1,48 @@ require 'rails_helper' RSpec.describe AccountMigration, type: :model do + describe 'validations' do + let(:source_account) { Fabricate(:account) } + let(:target_acct) { target_account.acct } + let(:subject) { AccountMigration.new(account: source_account, acct: target_acct) } + + context 'with valid properties' do + let(:target_account) { Fabricate(:account, username: 'target', domain: 'remote.org') } + + before do + target_account.aliases.create!(acct: source_account.acct) + + service_double = double + allow(ResolveAccountService).to receive(:new).and_return(service_double) + allow(service_double).to receive(:call).with(target_acct, anything).and_return(target_account) + end + + it 'passes validations' do + expect(subject).to be_valid + end + end + + context 'with unresolveable account' do + let(:target_acct) { 'target@remote' } + + before do + service_double = double + allow(ResolveAccountService).to receive(:new).and_return(service_double) + allow(service_double).to receive(:call).with(target_acct, anything).and_return(nil) + end + + it 'has errors on acct field' do + expect(subject).to model_have_error_on_field(:acct) + end + end + + context 'with a space in the domain part' do + let(:target_acct) { 'target@remote. org' } + + it 'has errors on acct field' do + expect(subject).to model_have_error_on_field(:acct) + end + end + end end |