Delete user if email isn't confirmable

bikeindex · Jan 23, 2018 · 3c8a705 · 3c8a705
1 parent 0fdc38e
commit 3c8a705
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/app/workers/email_confirmation_worker.rb b/app/workers/email_confirmation_worker.rb
@@ -6,5 +6,7 @@ class EmailConfirmationWorker
   def perform(user_id)
     user = User.find(user_id)
     CustomerMailer.confirmation_email(user).deliver_now
+  rescue Net::SMTPSyntaxError
+    user.destroy
   end
 end
diff --git a/spec/workers/email_confirmation_worker_spec.rb b/spec/workers/email_confirmation_worker_spec.rb
@@ -8,4 +8,11 @@
     EmailConfirmationWorker.new.perform(user.id)
     expect(ActionMailer::Base.deliveries.empty?).to be_falsey
   end
+
+  it 'deletes user if email is invalid' do
+    user = FactoryGirl.create(:user)
+    user.update(email: 'notaemail@fakeonotreal.blorgh')
+    EmailConfirmationWorker.new.perform(user.id)
+    expect(User.count).to be_zero
+  end
 end