Remove delegations

This commit is contained in:
Artur Beljajev 2019-03-28 00:17:48 +02:00
parent c85ebf542d
commit 0f4a67d696
18 changed files with 30 additions and 46 deletions

View file

@ -81,9 +81,8 @@ RSpec.describe DomainPresenter do
domain_delegatable_attributes = %i(
name
transfer_code
registrant_name
registrant
registrant_id
registrant_code
)
domain_delegatable_attributes.each do |attribute_name|

View file

@ -46,7 +46,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'keeps same registrant' do
expect { request; domain.reload }.to_not change { domain.registrant_code }
expect { request; domain.reload }.to_not change { domain.registrant.code }
end
it 'does not ask for confirmation' do
@ -91,7 +91,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'does not change registrant without confirmation' do
expect { request; domain.reload }.to_not change { domain.registrant_code }
expect { request; domain.reload }.to_not change { domain.registrant.code }
end
it 'asks for confirmation' do
@ -147,7 +147,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'keeps same registrant' do
expect { request; domain.reload }.to_not change { domain.registrant_code }
expect { request; domain.reload }.to_not change { domain.registrant.code }
end
it 'does not ask for confirmation' do
@ -192,7 +192,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'changes registrant without confirmation' do
expect { request; domain.reload }.to change { domain.registrant_code }.from('OLD-CODE').to('NEW-CODE')
expect { request; domain.reload }.to change { domain.registrant.code }.from('OLD-CODE').to('NEW-CODE')
end
it 'does not ask for confirmation' do

View file

@ -47,7 +47,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'changes registrant' do
expect { request; domain.reload }.to change { domain.registrant_code }.from('old-code').to('new-code')
expect { request; domain.reload }.to change { domain.registrant.code }.from('old-code').to('new-code')
end
it 'does not send confirmation email' do
@ -90,7 +90,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'does not change registrant' do
expect { request; domain.reload }.to_not change { domain.registrant_code }
expect { request; domain.reload }.to_not change { domain.registrant.code }
end
it 'sends confirmation and notice emails' do
@ -139,7 +139,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'changes registrant' do
expect { request; domain.reload }.to change { domain.registrant_code }.from('old-code').to('new-code')
expect { request; domain.reload }.to change { domain.registrant.code }.from('old-code').to('new-code')
end
it 'does not send confirmation email' do
@ -182,7 +182,7 @@ RSpec.describe 'EPP domain:update' do
end
it 'changes registrant' do
expect { request; domain.reload }.to change { domain.registrant_code }.from('old-code').to('new-code')
expect { request; domain.reload }.to change { domain.registrant.code }.from('old-code').to('new-code')
end
it 'does not send confirmation email' do