diff options
author | unarist <m.unarist@gmail.com> | 2018-03-27 12:22:58 +0900 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2018-03-27 05:22:58 +0200 |
commit | a1d091558555c8a58f7f63c4a9c3002421ea9041 (patch) | |
tree | 43f59f3eb6fa1a0dfefad85e21118ea885e35aa4 | |
parent | 2a90da18375a38957ae4c94fa3e86a8180237d8a (diff) |
Add a spec for UniqueUsernameValidator (#6927)
Note that this spec has a pending test about dots in the username, because allowing it has been reverted for now.
-rw-r--r-- | spec/validators/unique_username_validator_spec.rb | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/spec/validators/unique_username_validator_spec.rb b/spec/validators/unique_username_validator_spec.rb new file mode 100644 index 000000000..b9d773bed --- /dev/null +++ b/spec/validators/unique_username_validator_spec.rb @@ -0,0 +1,33 @@ +# frozen_string_literal: true + +require 'rails_helper' + +describe UniqueUsernameValidator do + describe '#validate' do + it 'does not add errors if username is nil' do + account = double(username: nil, persisted?: false, errors: double(add: nil)) + subject.validate(account) + expect(account.errors).to_not have_received(:add) + end + + it 'does not add errors when existing one is subject itself' do + account = Fabricate(:account, username: 'abcdef') + expect(account).to be_valid + end + + it 'adds an error when the username is already used with ignoring dots' do + pending 'allowing dots in username is still in development' + Fabricate(:account, username: 'abcd.ef') + account = double(username: 'ab.cdef', persisted?: false, errors: double(add: nil)) + subject.validate(account) + expect(account.errors).to have_received(:add) + end + + it 'adds an error when the username is already used with ignoring cases' do + Fabricate(:account, username: 'ABCdef') + account = double(username: 'abcDEF', persisted?: false, errors: double(add: nil)) + subject.validate(account) + expect(account.errors).to have_received(:add) + end + end +end |