Merge pull request #1 from substancelab/do-not-accept-whitespace

validator does not accept spaces in an email address
substancelab · Apr 21, 2015 · 433f38d · 433f38d
2 parents 74bdcf3 + 5aa2af9
commit 433f38d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/lib/activemodel_email_address_validator/email_address.rb b/lib/activemodel_email_address_validator/email_address.rb
@@ -25,6 +25,7 @@ def valid_using_regex?(regex)
     end
 
     def valid_using_default?
+      return false if address =~ /\s+/
       email_parts = address.split('@')
 
       return false unless email_parts.size == 2

diff --git a/test/activemodel-email_address_validator/test_email_address.rb b/test/activemodel-email_address_validator/test_email_address.rb
@@ -9,6 +9,10 @@ def test_rejects_email_address_without_at_sign
     reject("bobexample.com")
   end
 
+  def test_rejects_any_space
+    reject("John. Doe. somewhere@gmail.com")
+  end
+
   def test_rejects_nil
     reject(nil)
   end