Commit 351a563c authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Fix conflicts in application_settings_controller.rb and spec/features/admin/admin_settings_spec.rb

Signed-off-by: default avatarDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
parent 33967fb7
...@@ -59,19 +59,11 @@ class Admin::ApplicationSettingsController < Admin::ApplicationController ...@@ -59,19 +59,11 @@ class Admin::ApplicationSettingsController < Admin::ApplicationController
def application_setting_params def application_setting_params
params[:application_setting] ||= {} params[:application_setting] ||= {}
<<<<<<< HEAD
if params[:application_setting].key?(:enabled_oauth_sign_in_sources) if params[:application_setting].key?(:enabled_oauth_sign_in_sources)
enabled_oauth_sign_in_sources = params[:application_setting].delete(:enabled_oauth_sign_in_sources) enabled_oauth_sign_in_sources = params[:application_setting].delete(:enabled_oauth_sign_in_sources)
enabled_oauth_sign_in_sources&.delete("") enabled_oauth_sign_in_sources&.delete("")
=======
if params[:application_setting].key?(:enabled_oauth_sign_in_sources)
enabled_oauth_sign_in_sources = params[:application_setting].delete(:enabled_oauth_sign_in_sources)
enabled_oauth_sign_in_sources&.delete("")
>>>>>>> upstream/master
params[:application_setting][:disabled_oauth_sign_in_sources] = params[:application_setting][:disabled_oauth_sign_in_sources] =
AuthHelper.button_based_providers.map(&:to_s) - AuthHelper.button_based_providers.map(&:to_s) -
Array(enabled_oauth_sign_in_sources) Array(enabled_oauth_sign_in_sources)
......
...@@ -32,7 +32,6 @@ feature 'Admin updates settings' do ...@@ -32,7 +32,6 @@ feature 'Admin updates settings' do
expect(find('#application_setting_visibility_level_20')).not_to be_checked expect(find('#application_setting_visibility_level_20')).not_to be_checked
end end
<<<<<<< HEAD
describe 'LDAP settings' do describe 'LDAP settings' do
context 'with LDAP enabled' do context 'with LDAP enabled' do
scenario 'Change allow group owners to manage ldap' do scenario 'Change allow group owners to manage ldap' do
...@@ -58,8 +57,6 @@ feature 'Admin updates settings' do ...@@ -58,8 +57,6 @@ feature 'Admin updates settings' do
end end
end end
=======
>>>>>>> upstream/master
scenario 'Modify import sources' do scenario 'Modify import sources' do
expect(Gitlab::CurrentSettings.import_sources).not_to be_empty expect(Gitlab::CurrentSettings.import_sources).not_to be_empty
......
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