2013-11-01 17:06:20 -04:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe UserUpdater do
|
2013-12-10 12:46:35 -05:00
|
|
|
|
|
|
|
let(:acting_user) { Fabricate.build(:user) }
|
|
|
|
|
2013-11-01 17:06:20 -04:00
|
|
|
describe '#update' do
|
|
|
|
it 'saves user' do
|
|
|
|
user = Fabricate(:user, name: 'Billy Bob')
|
2013-12-10 12:46:35 -05:00
|
|
|
updater = described_class.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(name: 'Jim Tom')
|
|
|
|
|
|
|
|
expect(user.reload.name).to eq 'Jim Tom'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when update succeeds' do
|
|
|
|
it 'returns true' do
|
|
|
|
user = Fabricate(:user)
|
2013-12-10 12:46:35 -05:00
|
|
|
updater = described_class.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
expect(updater.update).to be_true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when update fails' do
|
|
|
|
it 'returns false' do
|
|
|
|
user = Fabricate(:user)
|
|
|
|
user.stubs(save: false)
|
2013-12-10 12:46:35 -05:00
|
|
|
updater = described_class.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
expect(updater.update).to be_false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with permission to update title' do
|
|
|
|
it 'allows user to change title' do
|
|
|
|
user = Fabricate(:user, title: 'Emperor')
|
|
|
|
guardian = stub
|
|
|
|
guardian.stubs(:can_grant_title?).with(user).returns(true)
|
2013-12-10 12:46:35 -05:00
|
|
|
Guardian.stubs(:new).with(acting_user).returns(guardian)
|
|
|
|
updater = described_class.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(title: 'Minion')
|
|
|
|
|
|
|
|
expect(user.reload.title).to eq 'Minion'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'without permission to update title' do
|
|
|
|
it 'does not allow user to change title' do
|
|
|
|
user = Fabricate(:user, title: 'Emperor')
|
|
|
|
guardian = stub
|
|
|
|
guardian.stubs(:can_grant_title?).with(user).returns(false)
|
2013-12-10 12:46:35 -05:00
|
|
|
Guardian.stubs(:new).with(acting_user).returns(guardian)
|
|
|
|
updater = described_class.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(title: 'Minion')
|
|
|
|
|
|
|
|
expect(user.reload.title).not_to eq 'Minion'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when website includes http' do
|
|
|
|
it 'does not add http before updating' do
|
|
|
|
user = Fabricate(:user)
|
2013-12-10 12:46:35 -05:00
|
|
|
updater = described_class.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(website: 'http://example.com')
|
|
|
|
|
2014-06-07 00:54:32 -04:00
|
|
|
expect(user.reload.user_profile.website).to eq 'http://example.com'
|
2013-11-01 17:06:20 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when website does not include http' do
|
|
|
|
it 'adds http before updating' do
|
|
|
|
user = Fabricate(:user)
|
2013-12-10 12:46:35 -05:00
|
|
|
updater = described_class.new(acting_user, user)
|
2013-11-01 17:06:20 -04:00
|
|
|
|
|
|
|
updater.update(website: 'example.com')
|
|
|
|
|
2014-06-07 00:54:32 -04:00
|
|
|
expect(user.reload.user_profile.website).to eq 'http://example.com'
|
2013-11-01 17:06:20 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|