From bba537a7bebe11be4b1e4a7e126a34cd27b73678 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Sun, 7 May 2017 21:32:52 -0400 Subject: Improve allowed language handling (#2897) * Dont allow empty value in user allowed languages * Sanitize language input to reject blank values in array --- spec/models/user_spec.rb | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'spec/models/user_spec.rb') diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index fffd92e3d..04c39de3b 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -22,6 +22,12 @@ RSpec.describe User, type: :model do user.valid? expect(user).to model_have_error_on_field(:email) end + + it 'cleans out empty string from languages' do + user = Fabricate.build(:user, allowed_languages: ['']) + user.valid? + expect(user.allowed_languages).to eq [] + end end describe 'settings' do -- cgit