diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2014-01-19 22:29:06 +0400 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2014-01-19 22:29:06 +0400 |
commit | 3628eb84c4e50ffb73776683aabd19b37c4cd299 (patch) | |
tree | b8136613c6d991432ff9560105d3b117831f6f10 /spec | |
parent | 25da1415d385bc5d662998688c0b166b39392f29 (diff) | |
parent | 05e4af5b4c7709ab08194d109ddec8e19f44758d (diff) |
Merge pull request #6072 from jvanbaarsen/fix-3516
Better check on the validity of emails
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/user_spec.rb | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index c33b6879cec..5e53ed09b58 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -76,6 +76,27 @@ describe User do it { should_not allow_value(-1).for(:projects_limit) } it { should ensure_length_of(:bio).is_within(0..255) } + + describe 'email' do + it 'accepts info@example.com' do + user = build(:user, email: 'info@example.com') + expect(user).to be_valid + end + it 'accepts info+test@example.com' do + user = build(:user, email: 'info+test@example.com') + expect(user).to be_valid + end + + it 'rejects test@test@example.com' do + user = build(:user, email: 'test@test@example.com') + expect(user).to be_invalid + end + + it 'rejects mailto:test@example.com' do + user = build(:user, email: 'mailto:test@example.com') + expect(user).to be_invalid + end + end end describe "Respond to" do |