Merge branch 'gliptak-omniauth_no_email_1'

This commit is contained in:
Dmitriy Zaporozhets 2013-03-05 16:50:46 +02:00
commit 3d45d300d8
2 changed files with 8 additions and 1 deletions

View file

@ -2,7 +2,8 @@ class UserObserver < ActiveRecord::Observer
def after_create(user) def after_create(user)
log_info("User \"#{user.name}\" (#{user.email}) was created") log_info("User \"#{user.name}\" (#{user.email}) was created")
Notify.delay.new_user_email(user.id, user.password) # Dont email omniauth created users
Notify.delay.new_user_email(user.id, user.password) unless user.extern_uid?
end end
def after_destroy user def after_destroy user

View file

@ -15,7 +15,13 @@ describe UserObserver do
create(:user) create(:user)
end end
it 'no email for external' do
Notify.should_not_receive(:new_user_email)
create(:user, extern_uid: '32442eEfsafada')
end
it 'trigger logger' do it 'trigger logger' do
user = double(:user, id: 42, password: 'P@ssword!', name: 'John', email: 'u@mail.local', extern_uid?: false)
Gitlab::AppLogger.should_receive(:info) Gitlab::AppLogger.should_receive(:info)
create(:user) create(:user)
end end