diff options
author | Alexis Reigel <alexis.reigel.ext@siemens.com> | 2018-09-26 17:25:31 +0200 |
---|---|---|
committer | Alexis Reigel <alexis.reigel.ext@siemens.com> | 2018-10-01 13:24:16 +0200 |
commit | f38aa2bf3043fbfa49cafad3834f3d4aada75afd (patch) | |
tree | 1a70d3177b4313e34654c92f7f3c4e468239c9ed /spec/services/users | |
parent | 921c96ab0c9fe0fb532a3261f2664268b48e3bce (diff) | |
download | gitlab-ce-f38aa2bf3043fbfa49cafad3834f3d4aada75afd.tar.gz |
add missing allowed attributes
Diffstat (limited to 'spec/services/users')
-rw-r--r-- | spec/services/users/build_service_spec.rb | 42 |
1 files changed, 42 insertions, 0 deletions
diff --git a/spec/services/users/build_service_spec.rb b/spec/services/users/build_service_spec.rb index b987fe45138..7b2444489cb 100644 --- a/spec/services/users/build_service_spec.rb +++ b/spec/services/users/build_service_spec.rb @@ -14,6 +14,48 @@ describe Users::BuildService do expect(service.execute).to be_valid end + context 'allowed params' do + let(:params) do + { + access_level: 1, + admin: 1, + avatar: anything, + bio: 1, + can_create_group: 1, + color_scheme_id: 1, + email: 1, + external: 1, + force_random_password: 1, + hide_no_password: 1, + hide_no_ssh_key: 1, + linkedin: 1, + name: 1, + password: 1, + password_automatically_set: 1, + password_expires_at: 1, + projects_limit: 1, + remember_me: 1, + skip_confirmation: 1, + skype: 1, + theme_id: 1, + twitter: 1, + username: 1, + website_url: 1, + private_profile: 1, + organization: 1, + location: 1 + } + end + + it 'sets all allowed attributes' do + admin_user # call first so the admin gets created before setting `expect` + + expect(User).to receive(:new).with(hash_including(params)).and_call_original + + service.execute + end + end + context 'with "user_default_external" application setting' do using RSpec::Parameterized::TableSyntax |