diff options
author | Toon Claes <toon@gitlab.com> | 2017-03-20 22:54:34 +0100 |
---|---|---|
committer | Toon Claes <toon@gitlab.com> | 2017-03-20 22:57:45 +0100 |
commit | 4e1cebabc7906f46cfab15602a5b9ae15448598d (patch) | |
tree | 274fc0aaff418edd5c2b1f064da8417646a5c5a4 /spec/controllers/admin/application_settings_controller_spec.rb | |
parent | 71306f14f69b9798a21905cfa8160c4990ca5d5d (diff) | |
download | gitlab-ce-4e1cebabc7906f46cfab15602a5b9ae15448598d.tar.gz |
Undo explicit conversion to Integer
With the changes in Gitlab::VisibilityLevel it is no longer needed to
explicit conversion to Integer in the controller itself.
Diffstat (limited to 'spec/controllers/admin/application_settings_controller_spec.rb')
-rw-r--r-- | spec/controllers/admin/application_settings_controller_spec.rb | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/spec/controllers/admin/application_settings_controller_spec.rb b/spec/controllers/admin/application_settings_controller_spec.rb new file mode 100644 index 00000000000..84a1ce773a1 --- /dev/null +++ b/spec/controllers/admin/application_settings_controller_spec.rb @@ -0,0 +1,28 @@ +require 'spec_helper' + +describe Admin::ApplicationSettingsController do + include StubENV + + let(:admin) { create(:admin) } + + before do + sign_in(admin) + stub_env('IN_MEMORY_APPLICATION_SETTINGS', 'false') + end + + describe 'PATCH #update' do + it 'updates the default_project_visibility for string value' do + patch :update, application_setting: { default_project_visibility: "20" } + + expect(response).to redirect_to(admin_application_settings_path) + expect(ApplicationSetting.current.default_project_visibility).to eq Gitlab::VisibilityLevel::PUBLIC + end + + it 'falls back to default with default_project_visibility setting is omitted' do + patch :update, application_setting: {} + + expect(response).to redirect_to(admin_application_settings_path) + expect(ApplicationSetting.current.default_project_visibility).to eq Gitlab::VisibilityLevel::PRIVATE + end + end +end |