diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2017-10-17 13:53:42 +0000 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2017-10-17 13:53:42 +0000 |
commit | 65a1cbc600cf36ff5db9644072e4bf1e2b8e5cde (patch) | |
tree | 0354292dccf0debc90d55a00937d4033453603e0 /spec | |
parent | 526c47618e446bfec776b6e17462298f17fb24ee (diff) | |
parent | c909b6aa6d62a4c556a866166d0a98c952d2ef62 (diff) | |
download | gitlab-ce-65a1cbc600cf36ff5db9644072e4bf1e2b8e5cde.tar.gz |
Merge branch 'prevent-creating-multiple-application-settings' into 'master'
Prevent creating multiple ApplicationSetting by forcing it to always have id=1
Closes #34728
See merge request gitlab-org/gitlab-ce!14718
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/application_setting_spec.rb | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/spec/models/application_setting_spec.rb b/spec/models/application_setting_spec.rb index cf192691507..6945c90cb9b 100644 --- a/spec/models/application_setting_spec.rb +++ b/spec/models/application_setting_spec.rb @@ -222,6 +222,16 @@ describe ApplicationSetting do end end + context 'restrict creating duplicates' do + before do + described_class.create_from_defaults + end + + it 'raises an record creation violation if already created' do + expect { described_class.create_from_defaults }.to raise_error(ActiveRecord::RecordNotUnique) + end + end + context 'restricted signup domains' do it 'sets single domain' do setting.domain_whitelist_raw = 'example.com' |