Don't send comment email to suspended or inactive users.

app/models/polymorphic/comment.rb

@@ -53,7 +53,8 @@
   # Notify subscribed users when a comment is added, unless user created this comment
   def notify_subscribers
     commentable.subscribed_users.reject { |user_id| user_id == user.id }.each do |subscriber_id|
-      if subscriber = User.find_by_id(subscriber_id)
+      subscriber = User.find_by_id(subscriber_id)
+      if subscriber && subscriber.confirmed? && !subscriber.awaits_approval? && !subscriber.suspended? && subscriber.email.present?
         SubscriptionMailer.comment_notification(subscriber, self).deliver_later
       end
     end

spec/factories/user_factories.rb

@@ -31,6 +31,7 @@
     deleted_at          { nil }
     updated_at          { FactoryBot.generate(:time) }
     created_at          { FactoryBot.generate(:time) }
+    confirmed_at        { Time.now.utc }
     suspended_at        { nil }
     password            { "password" }
     password_confirmation { "password" }

spec/models/polymorphic/comment_spec.rb

@@ -42,4 +42,36 @@
       expect(entity.subscribed_users).to include(user.id)
     end
   end
+
+  describe "notify_subscribers" do
+    let(:subscriber) { create(:user) }
+    let(:entity) { create(:lead, subscribed_users: [subscriber.id]) }
+    before(:each) do
+      allow(SubscriptionMailer).to receive_message_chain(:comment_notification, :deliver_later)
+    end
+
+    it "should notify subscribers when a comment is added" do
+      Comment.create!(comment: "Hello", user: create(:user), commentable: entity)
+      expect(SubscriptionMailer).to have_received(:comment_notification).with(subscriber, instance_of(Comment))
+    end
+
+    it "should not notify the user who created the comment" do
+      user = create(:user, confirmed_at: Time.now, email: "[email protected]")
+      Comment.create!(comment: "Hello", user: user, commentable: entity)
+      expect(SubscriptionMailer).not_to have_received(:comment_notification).with(user, instance_of(Comment))
+    end
+
+    it "should not notify suspended users" do
+      subscriber.update(suspended_at: Time.now)
+      Comment.create!(comment: "Hello", user: create(:user), commentable: entity)
+      expect(SubscriptionMailer).not_to have_received(:comment_notification).with(subscriber, instance_of(Comment))
+    end
+
+    it "should not notify users awaiting approval" do
+      subscriber.update(sign_in_count: 0, suspended_at: Time.now)
+      allow(Setting).to receive(:user_signup).and_return(:not_allowed)
+      Comment.create!(comment: "Hello", user: create(:user), commentable: entity)
+      expect(SubscriptionMailer).not_to have_received(:comment_notification).with(subscriber, instance_of(Comment))
+    end
+  end
 end