diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2017-03-30 14:53:48 +0300 |
---|---|---|
committer | Sean McGivern <sean@mcgivern.me.uk> | 2017-03-30 14:53:48 +0300 |
commit | c7403521a8d3f0410a2929784339334af4c74059 (patch) | |
tree | 572780c719f986d0ff270c68c9ba6bc563519dfe /spec/services | |
parent | 5920b4a52f7458172610e29150f84f13a617e37f (diff) | |
parent | 53ef1de4fcdc7ea0f94ae8fe73f9a2b46e376223 (diff) |
Merge branch '30248-fix-production-admin-fixture' into 'master'
Fix production admin fixture to use the new `Users::CreateService`
Closes #30248
See merge request !10326
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/users/create_service_spec.rb | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/spec/services/users/create_service_spec.rb b/spec/services/users/create_service_spec.rb index 5f79203701a..66f68650f81 100644 --- a/spec/services/users/create_service_spec.rb +++ b/spec/services/users/create_service_spec.rb @@ -61,6 +61,23 @@ describe Users::CreateService, services: true do ) end + context 'when the current_user is not persisted' do + let(:admin_user) { build(:admin) } + + it 'persists the given attributes and sets created_by_id to nil' do + user = service.execute + user.reload + + expect(user).to have_attributes( + name: params[:name], + username: params[:username], + email: params[:email], + password: params[:password], + created_by_id: nil + ) + end + end + it 'user is not confirmed if skip_confirmation param is not present' do expect(service.execute).not_to be_confirmed end |