Adds validation of email present in valid_email? method
This commit is contained in:
@@ -28,7 +28,7 @@ class EmailDigest
|
||||
end
|
||||
|
||||
def valid_email?
|
||||
user.email.match(/\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\Z/i)
|
||||
user.email.present? && user.email.match(/\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\Z/i)
|
||||
end
|
||||
|
||||
end
|
||||
|
||||
@@ -152,6 +152,14 @@ describe EmailDigest do
|
||||
email_digest = described_class.new(user)
|
||||
expect(email_digest.valid_email?).to be(nil)
|
||||
end
|
||||
|
||||
it "returns false if email does not exist" do
|
||||
user = create(:user)
|
||||
user.update_attribute(:email, nil)
|
||||
|
||||
email_digest = described_class.new(user)
|
||||
expect(email_digest.valid_email?).to be(false)
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user