about summary refs log tree commit diff
diff options
context:
space:
mode:
authorunarist <m.unarist@gmail.com>2018-03-27 12:22:58 +0900
committerEugen Rochko <eugen@zeonfederated.com>2018-03-27 05:22:58 +0200
commita1d091558555c8a58f7f63c4a9c3002421ea9041 (patch)
tree43f59f3eb6fa1a0dfefad85e21118ea885e35aa4
parent2a90da18375a38957ae4c94fa3e86a8180237d8a (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.rb33
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