summaryrefslogtreecommitdiff
path: root/users/test
diff options
context:
space:
mode:
authorjessib <jessib@riseup.net>2013-09-23 12:23:08 -0700
committerjessib <jessib@riseup.net>2013-09-23 12:23:08 -0700
commita9c68ba0bbba7a95e9b4a3ff24554d1b0af6cbc5 (patch)
treec8a3bf7fe4c8001718ea31e56d036b267ee362e6 /users/test
parent906e177b35606835c4f10a9de65d9bd83cbe21ad (diff)
This ensures that email addresses contain only lowercase letters, and that an identity's destination is a valid Email.
Diffstat (limited to 'users/test')
-rw-r--r--users/test/unit/identity_test.rb17
1 files changed, 14 insertions, 3 deletions
diff --git a/users/test/unit/identity_test.rb b/users/test/unit/identity_test.rb
index b3918f1..02f14c0 100644
--- a/users/test/unit/identity_test.rb
+++ b/users/test/unit/identity_test.rb
@@ -70,13 +70,13 @@ class IdentityTest < ActiveSupport::TestCase
id.destroy
end
- test "fail to end non-local email address as identity address" do
- id = Identity.for @user, address: 'blah@sdlfksjdfljk.com'
+ test "fail to add non-local email address as identity address" do
+ id = Identity.for @user, address: forward_address
assert !id.valid?
assert_match /needs to end in/, id.errors[:address].first
end
- test "only lowercase alias" do
+ test "alias must meet some conditions as login" do
id = Identity.create_for @user, address: alias_name.capitalize
assert !id.valid?
#hacky way to do this, but okay for now:
@@ -84,6 +84,17 @@ class IdentityTest < ActiveSupport::TestCase
assert id.errors.messages.flatten(2).include? "Only lowercase letters, digits, . - and _ allowed."
end
+ test "destination must be valid email address" do
+ id = Identity.create_for @user, address: @user.email_address, destination: 'ASKJDLFJD'
+ assert !id.valid?
+ assert id.errors.messages[:destination].include? "needs to be a valid email address"
+ end
+
+ test "only lowercase destination" do
+ id = Identity.create_for @user, address: @user.email_address, destination: forward_address.capitalize
+ assert !id.valid?
+ assert id.errors.messages[:destination].include? "letters must be lowercase"
+ end
def alias_name
@alias_name ||= Faker::Internet.user_name