Commit b32b6d6c authored by Rémy Coutable's avatar Rémy Coutable

Merge branch 'rails5-fix-47805' into 'master'

Rails5 ActionController::ParameterMissing: param is missing or the value is…

Closes #47805

See merge request gitlab-org/gitlab-ce!19763
parents 001602b8 71e0d33e
---
title: 'Rails5 ActionController::ParameterMissing: param is missing or the value is
empty: application_setting'
merge_request: 19763
author: Jasper Maes
type: fixed
...@@ -73,7 +73,7 @@ describe Admin::ApplicationSettingsController do ...@@ -73,7 +73,7 @@ describe Admin::ApplicationSettingsController do
end end
it 'updates the restricted_visibility_levels when empty array is passed' do it 'updates the restricted_visibility_levels when empty array is passed' do
put :update, application_setting: { restricted_visibility_levels: [] } put :update, application_setting: { restricted_visibility_levels: [""] }
expect(response).to redirect_to(admin_application_settings_path) expect(response).to redirect_to(admin_application_settings_path)
expect(ApplicationSetting.current.restricted_visibility_levels).to be_empty expect(ApplicationSetting.current.restricted_visibility_levels).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