Commit cf628631 authored by James Lopez's avatar James Lopez

added bang option to spec

parent 11044ab1
......@@ -7,6 +7,7 @@ describe Users::UpdateService, services: true do
describe '#execute' do
it 'updates the name' do
result = update_user(user, user, name: 'New Name')
expect(result).to eq({ status: :success })
expect(user.name).to eq('New Name')
end
......@@ -14,6 +15,7 @@ describe Users::UpdateService, services: true do
context 'when updated by an admin' do
it 'updates the name' do
result = update_user(admin, user, name: 'New Name')
expect(result).to eq({ status: :success })
expect(user.name).to eq('New Name')
end
......@@ -29,4 +31,32 @@ describe Users::UpdateService, services: true do
described_class.new(current_user, user, opts).execute
end
end
describe '#execute!' do
it 'updates the name' do
result = update_user(user, user, name: 'New Name')
expect(result).to be true
expect(user.name).to eq('New Name')
end
context 'when updated by an admin' do
it 'updates the name' do
result = update_user(admin, user, name: 'New Name')
expect(result).to be true
expect(user.name).to eq('New Name')
end
end
it 'returns an error result when record cannot be updated' do
expect do
update_user(user, create(:user), { name: 'New Name' })
end.to raise_error Gitlab::Access::AccessDeniedError
end
def update_user(current_user, user, opts)
described_class.new(current_user, user, opts).execute!
end
end
end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment