From 5e6781a656265cca142ae944967b78b55a868892 Mon Sep 17 00:00:00 2001 From: Thibault Jouan Date: Fri, 30 Dec 2011 13:40:03 +0000 Subject: [PATCH] Improve User specs about email uniqueness --- spec/models/user_spec.rb | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index c987b09..9d04df0 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -11,11 +11,12 @@ describe User do it { should validate_presence_of :password_hash } context 'when a user with the same email address already exists' do - it 'should not be valid' do - user = Factory.create(:user, :email => 'unique@example.net') - new_user = Factory.build(:user, :email => user.email) - new_user.should_not be_valid - end + let(:old_user) { Factory.create(:user, :email => 'unique@example.net') } + subject { Factory.build(:user, :email => old_user.email) } + + it { should_not be_valid } + + it { should have(1).error_on(:email) } end context 'when password_confirmation does not match password' do