diff options
| author | Matt Jankowski <mjankowski@thoughtbot.com> | 2017-05-07 21:32:52 -0400 |
|---|---|---|
| committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-05-08 03:32:52 +0200 |
| commit | bba537a7bebe11be4b1e4a7e126a34cd27b73678 (patch) | |
| tree | 940ae93f632bfa1cd99d525d0fde9bb3523fe5c7 /spec/controllers/settings | |
| parent | 0291b73de7e2fca7f5c137de4427800c02d7a233 (diff) | |
Improve allowed language handling (#2897)
* Dont allow empty value in user allowed languages * Sanitize language input to reject blank values in array
Diffstat (limited to 'spec/controllers/settings')
| -rw-r--r-- | spec/controllers/settings/preferences_controller_spec.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/controllers/settings/preferences_controller_spec.rb b/spec/controllers/settings/preferences_controller_spec.rb index 432e35cd4..6805a2ce0 100644 --- a/spec/controllers/settings/preferences_controller_spec.rb +++ b/spec/controllers/settings/preferences_controller_spec.rb @@ -18,7 +18,7 @@ describe Settings::PreferencesController do describe 'PUT #update' do it 'updates the user record' do - put :update, params: { user: { locale: 'en', allowed_languages: ['es', 'fr'] } } + put :update, params: { user: { locale: 'en', allowed_languages: ['es', 'fr', ''] } } expect(response).to redirect_to(settings_preferences_path) user.reload |