activity_notification 1.3.0 → 1.4.0

data/spec/concerns/models/notifier_spec.rb

@@ -5,7 +5,7 @@ shared_examples_for :notifier do
   describe "with association" do
     it "has many sent_notifications" do
       notification_1 = create(:notification, notifier: test_instance)
-      notification_2 = create(:notification, notifier: test_instance)
+      notification_2 = create(:notification, notifier: test_instance, created_at: notification_1.created_at + 10.second)
       expect(test_instance.sent_notifications.count).to    eq(2)
       expect(test_instance.sent_notifications.earliest).to eq(notification_1)
       expect(test_instance.sent_notifications.latest).to   eq(notification_2)

data/spec/concerns/models/subscriber_spec.rb

@@ -11,11 +11,11 @@ shared_examples_for :subscriber do
   describe "with association" do
     it "has many subscriptions" do
       subscription_1 = create(:subscription, target: test_instance, key: 'subscription_key_1')
-      subscription_2 = create(:subscription, target: test_instance, key: 'subscription_key_2')
+      subscription_2 = create(:subscription, target: test_instance, key: 'subscription_key_2', created_at: subscription_1.created_at + 10.second)
       expect(test_instance.subscriptions.count).to                eq(2)
       expect(test_instance.subscriptions.earliest_order.first).to eq(subscription_1)
       expect(test_instance.subscriptions.latest_order.first).to   eq(subscription_2)
-      expect(test_instance.subscriptions).to                      eq(ActivityNotification::Subscription.filtered_by_target(test_instance))
+      expect(test_instance.subscriptions.latest_order).to         eq(ActivityNotification::Subscription.filtered_by_target(test_instance).latest_order)
     end
   end    
 
@@ -182,7 +182,7 @@ shared_examples_for :subscriber do
       context "when the target has subscriptions" do
         before do
           @subscription2 = create(:subscription, target: test_instance, key: 'subscription_key_2')
-          @subscription1 = create(:subscription, target: test_instance, key: 'subscription_key_1')
+          @subscription1 = create(:subscription, target: test_instance, key: 'subscription_key_1', created_at: @subscription2.created_at + 10.second)
         end
 
         context "without any options" do
@@ -221,7 +221,7 @@ shared_examples_for :subscriber do
         context 'with custom_filter options' do
           it "returns filtered notifications only" do
             if ActivityNotification.config.orm == :active_record
-              options = { custom_filter: ["key = ?", 'subscription_key_2'] }
+              options = { custom_filter: ["subscriptions.key = ?", 'subscription_key_2'] }
               expect(test_instance.subscription_index(options)[0]).to eq(@subscription2)
               expect(test_instance.subscription_index(options).size).to eq(1)
             end
@@ -258,8 +258,8 @@ shared_examples_for :subscriber do
 
       context "when the target has notifications" do
         before do
-          create(:notification, target: test_instance, key: 'notification_key_2')
-          create(:notification, target: test_instance, key: 'notification_key_1')
+          notification = create(:notification, target: test_instance, key: 'notification_key_2')
+          create(:notification, target: test_instance, key: 'notification_key_1', created_at: notification.created_at + 10.second)
           create(:subscription, target: test_instance, key: 'notification_key_1')
         end
 
@@ -323,7 +323,7 @@ shared_examples_for :subscriber do
         context 'with custom_filter options' do
           it "returns filtered notifications only" do
             if ActivityNotification.config.orm == :active_record
-              options = { custom_filter: ["key = ?", 'notification_key_2'] }
+              options = { custom_filter: ["notifications.key = ?", 'notification_key_2'] }
               expect(test_instance.notification_keys(options)[0]).to eq('notification_key_2')
               expect(test_instance.notification_keys(options).size).to eq(1)
             end

data/spec/concerns/models/target_spec.rb

@@ -6,7 +6,7 @@ shared_examples_for :target do
   describe "with association" do
     it "has many notifications" do
       notification_1 = create(:notification, target: test_instance)
-      notification_2 = create(:notification, target: test_instance)
+      notification_2 = create(:notification, target: test_instance, created_at: notification_1.created_at + 10.second)
       expect(test_instance.notifications.count).to    eq(2)
       expect(test_instance.notifications.earliest).to eq(notification_1)
       expect(test_instance.notifications.latest).to   eq(notification_2)
@@ -108,15 +108,15 @@ shared_examples_for :target do
           @target_2 = create(test_class_name)
           @target_3 = create(test_class_name)
           notification_1  = create(:notification, target: @target_1)
-          @notification_2 = create(:notification, target: @target_1)
-          notification_3  = create(:notification, target: @target_1, group_owner: @notification_2)
-          @notification_4 = create(:notification, target: @target_1, group_owner: @notification_2)
-          notification_5  = create(:notification, target: @target_2)
-          notification_6  = create(:notification, target: @target_2)
+          @notification_2 = create(:notification, target: @target_1, created_at: notification_1.created_at + 10.second)
+          notification_3  = create(:notification, target: @target_1, group_owner: @notification_2, created_at: notification_1.created_at + 20.second)
+          @notification_4 = create(:notification, target: @target_1, group_owner: @notification_2, created_at: notification_1.created_at + 30.second)
+          notification_5  = create(:notification, target: @target_2, created_at: notification_1.created_at + 40.second)
+          notification_6  = create(:notification, target: @target_2, created_at: notification_1.created_at + 50.second)
           notification_6.open!
-          notification_7  = create(:notification, target: @target_3)
+          notification_7  = create(:notification, target: @target_3, created_at: notification_1.created_at + 60.second)
           notification_7.open!
-          notification_8  = create(:notification, target: test_notifiable)
+          notification_8  = create(:notification, target: test_notifiable, created_at: notification_1.created_at + 70.second)
         end
 
         context "as default" do
@@ -517,9 +517,9 @@ shared_examples_for :target do
           @group         = create(:article)
           @key           = 'test.key.1'
           @notification2 = create(:notification, target: test_instance, notifiable: @notifiable)
-          @notification1 = create(:notification, target: test_instance, notifiable: create(:comment), group: @group)
-          @member1       = create(:notification, target: test_instance, notifiable: create(:comment), group_owner: @notification1)
-          @notification3 = create(:notification, target: test_instance, notifiable: create(:article), key: @key)
+          @notification1 = create(:notification, target: test_instance, notifiable: create(:comment), group: @group, created_at: @notification2.created_at + 10.second)
+          @member1       = create(:notification, target: test_instance, notifiable: create(:comment), group_owner: @notification1, created_at: @notification2.created_at + 20.second)
+          @notification3 = create(:notification, target: test_instance, notifiable: create(:article), key: @key, created_at: @notification2.created_at + 30.second)
           @notification3.open!
         end
 
@@ -615,7 +615,7 @@ shared_examples_for :target do
         context 'with custom_filter options' do
           it "returns filtered notifications only" do
             if ActivityNotification.config.orm == :active_record
-              options = { custom_filter: ["key = ?", @key] }
+              options = { custom_filter: ["notifications.key = ?", @key] }
               expect(test_instance.notification_index(options)[0]).to eq(@notification3)
               expect(test_instance.notification_index(options).size).to eq(1)
             end
@@ -629,8 +629,8 @@ shared_examples_for :target do
 
       context "when the target has no unopened notifications" do
         before do
-          create(:notification, target: test_instance, opened_at: Time.current)
-          create(:notification, target: test_instance, opened_at: Time.current)
+          notification = create(:notification, target: test_instance, opened_at: Time.current)
+          create(:notification, target: test_instance, opened_at: Time.current, created_at: notification.created_at + 10.second)
         end
 
         it "calls unopened_notification_index" do
@@ -665,7 +665,7 @@ shared_examples_for :target do
       context "when the target has unopened notifications" do
         before do
           @notification_1 = create(:notification, target: test_instance)
-          @notification_2 = create(:notification, target: test_instance)
+          @notification_2 = create(:notification, target: test_instance, created_at: @notification_1.created_at + 10.second)
         end
 
         context "without limit" do
@@ -721,7 +721,7 @@ shared_examples_for :target do
       context "when the target has opened notifications" do
         before do
           @notification_1 = create(:notification, target: test_instance, opened_at: Time.current)
-          @notification_2 = create(:notification, target: test_instance, opened_at: Time.current)
+          @notification_2 = create(:notification, target: test_instance, opened_at: Time.current, created_at: @notification_1.created_at + 10.second)
         end
 
         context "without limit" do
@@ -808,8 +808,8 @@ shared_examples_for :target do
           @group         = create(:article)
           @key           = 'test.key.1'
           @notification2 = create(:notification, target: test_instance, notifiable: @notifiable)
-          @notification1 = create(:notification, target: test_instance, notifiable: create(:comment), group: @group)
-          @notification3 = create(:notification, target: test_instance, notifiable: create(:article), key: @key)
+          @notification1 = create(:notification, target: test_instance, notifiable: create(:comment), group: @group, created_at: @notification2.created_at + 10.second)
+          @notification3 = create(:notification, target: test_instance, notifiable: create(:article), key: @key, created_at: @notification2.created_at + 20.second)
           @notification3.open!
         end
 
@@ -884,8 +884,8 @@ shared_examples_for :target do
 
       context "when the target has no unopened notifications" do
         before do
-          create(:notification, target: test_instance, opened_at: Time.current)
-          create(:notification, target: test_instance, opened_at: Time.current)
+          notification = create(:notification, target: test_instance, opened_at: Time.current)
+          create(:notification, target: test_instance, opened_at: Time.current, created_at: notification.created_at + 10.second)
         end
 
         it "calls unopened_notification_index_with_attributes" do

data/spec/controllers/notifications_controller_shared_examples.rb

@@ -57,9 +57,15 @@ shared_examples_for :notification_controller do
       end
 
       it "raises ActiveRecord::RecordNotFound" do
-        expect {
-          get_with_compatibility :index, target_params.merge({ typed_target_param => 0 }), valid_session
-        }.to raise_error(ActiveRecord::RecordNotFound)
+        if ENV['AN_TEST_DB'] == 'mongodb'
+          expect {
+            get_with_compatibility :index, target_params.merge({ typed_target_param => 0 }), valid_session
+          }.to raise_error(Mongoid::Errors::DocumentNotFound)
+        else
+          expect {
+            get_with_compatibility :index, target_params.merge({ typed_target_param => 0 }), valid_session
+          }.to raise_error(ActiveRecord::RecordNotFound)
+        end
       end
     end
 
@@ -74,8 +80,13 @@ shared_examples_for :notification_controller do
       end
 
       it "returns json format" do
-        expect(JSON.parse(response.body).first)
-        .to include("target_id" => test_target.id, "target_type" => test_target.to_class_name)
+        if ActivityNotification.config.orm == :active_record
+          expect(JSON.parse(response.body).first)
+          .to include("target_id" => test_target.id, "target_type" => test_target.to_class_name)
+        else
+          expect(JSON.parse(response.body).first)
+          .to include("target_id" => test_target.id.to_s, "target_type" => test_target.to_class_name)
+        end
       end
     end
 
@@ -155,9 +166,9 @@ shared_examples_for :notification_controller do
         @notifiable    = create(:article)
         @group         = create(:article)
         @key           = 'test.key.1'
-        create(:notification, target: test_target, notifiable: @notifiable)
-        create(:notification, target: test_target, notifiable: create(:comment), group: @group)
-        create(:notification, target: test_target, notifiable: create(:article), key: @key).open!
+        notification   = create(:notification, target: test_target, notifiable: @notifiable)
+        create(:notification, target: test_target, notifiable: create(:comment), group: @group, created_at: notification.created_at + 10.second)
+        create(:notification, target: test_target, notifiable: create(:article), key: @key, created_at: notification.created_at + 20.second).open!
         @notification1 = test_target.notification_index[0]
         @notification2 = test_target.notification_index[1]
         @notification3 = test_target.notification_index[2]

data/spec/controllers/subscriptions_controller_shared_examples.rb

@@ -69,9 +69,15 @@ shared_examples_for :subscription_controller do
       end
 
       it "raises ActiveRecord::RecordNotFound" do
-        expect {
-          get_with_compatibility :index, target_params.merge({ typed_target_param => 0 }), valid_session
-        }.to raise_error(ActiveRecord::RecordNotFound)
+        if ENV['AN_TEST_DB'] == 'mongodb'
+          expect {
+            get_with_compatibility :index, target_params.merge({ typed_target_param => 0 }), valid_session
+          }.to raise_error(Mongoid::Errors::DocumentNotFound)
+        else
+          expect {
+            get_with_compatibility :index, target_params.merge({ typed_target_param => 0 }), valid_session
+          }.to raise_error(ActiveRecord::RecordNotFound)
+        end
       end
     end
 
@@ -87,8 +93,13 @@ shared_examples_for :subscription_controller do
       end
 
       it "returns json format" do
-        expect(JSON.parse(response.body)["subscriptions"].first)
-        .to include("target_id" => test_target.id, "target_type" => test_target.to_class_name)
+        if ActivityNotification.config.orm == :active_record
+          expect(JSON.parse(response.body)["subscriptions"].first)
+          .to include("target_id" => test_target.id, "target_type" => test_target.to_class_name)
+        else
+          expect(JSON.parse(response.body)["subscriptions"].first)
+          .to include("target_id" => test_target.id.to_s, "target_type" => test_target.to_class_name)
+        end
         expect(JSON.parse(response.body)["unconfigured_notification_keys"].first)
         .to eq('test_notification_key')
       end

data/spec/models/notification_spec.rb

@@ -277,7 +277,7 @@ describe ActivityNotification::Notification, type: :model do
         context 'with custom_filter options' do
           it "works with filtered_by_options scope" do
             if ActivityNotification.config.orm == :active_record
-              notifications = ActivityNotification::Notification.filtered_by_options({ custom_filter: ["key = ?", @key_1] })
+              notifications = ActivityNotification::Notification.filtered_by_options({ custom_filter: ["notifications.key = ?", @key_1] })
               expect(notifications.to_a.size).to eq(1)
               expect(notifications.first).to eq(@notification_1)
             end
@@ -301,9 +301,9 @@ describe ActivityNotification::Notification, type: :model do
       before do
         ActivityNotification::Notification.delete_all
         unopened_group_owner   = create(:notification, group_owner: nil)
-        unopened_group_member  = create(:notification, group_owner: unopened_group_owner)
-        opened_group_owner     = create(:notification, group_owner: nil, opened_at: Time.current)
-        opened_group_member    = create(:notification, group_owner: opened_group_owner, opened_at: Time.current)
+        unopened_group_member  = create(:notification, group_owner: unopened_group_owner, created_at: unopened_group_owner.created_at + 10.second)
+        opened_group_owner     = create(:notification, group_owner: nil, opened_at: Time.current, created_at: unopened_group_owner.created_at + 20.second)
+        opened_group_member    = create(:notification, group_owner: opened_group_owner, opened_at: Time.current, created_at: unopened_group_owner.created_at + 30.second)
         @earliest_notification = unopened_group_owner
         @latest_notification   = opened_group_member
       end

data/spec/models/subscription_spec.rb

@@ -93,7 +93,7 @@ describe ActivityNotification::Subscription, type: :model do
         context 'with custom_filter options' do
           it "works with filtered_by_options scope" do
             if ActivityNotification.config.orm == :active_record
-              subscriptions = ActivityNotification::Subscription.filtered_by_options({ custom_filter: ["key = ?", @key_1] })
+              subscriptions = ActivityNotification::Subscription.filtered_by_options({ custom_filter: ["subscriptions.key = ?", @key_1] })
               expect(subscriptions.size).to eq(1)
               expect(subscriptions.first).to eq(@subscription_1)
             end
@@ -117,9 +117,9 @@ describe ActivityNotification::Subscription, type: :model do
       before do
         ActivityNotification::Subscription.delete_all
         @subscription_1 = create(:subscription, key: 'key.1')
-        @subscription_2 = create(:subscription, key: 'key.2')
-        @subscription_3 = create(:subscription, key: 'key.3')
-        @subscription_4 = create(:subscription, key: 'key.4')
+        @subscription_2 = create(:subscription, key: 'key.2', created_at: @subscription_1.created_at + 10.second)
+        @subscription_3 = create(:subscription, key: 'key.3', created_at: @subscription_1.created_at + 20.second)
+        @subscription_4 = create(:subscription, key: 'key.4', created_at: @subscription_1.created_at + 30.second)
       end
 
       it "works with latest_order scope" do
@@ -137,17 +137,21 @@ describe ActivityNotification::Subscription, type: :model do
       end
 
       it "works with latest_subscribed_order scope" do
-        @subscription_2.subscribe
-        subscriptions = ActivityNotification::Subscription.latest_subscribed_order
-        expect(subscriptions.size).to eq(4)
-        expect(subscriptions.first).to eq(@subscription_2)
+        Timecop.travel(1.minute.from_now) do
+          @subscription_2.subscribe
+          subscriptions = ActivityNotification::Subscription.latest_subscribed_order
+          expect(subscriptions.size).to eq(4)
+          expect(subscriptions.first).to eq(@subscription_2)
+        end
       end
 
       it "works with earliest_subscribed_order scope" do
-        @subscription_3.subscribe
-        subscriptions = ActivityNotification::Subscription.earliest_subscribed_order
-        expect(subscriptions.size).to eq(4)
-        expect(subscriptions.last).to eq(@subscription_3)
+        Timecop.travel(1.minute.from_now) do
+          @subscription_3.subscribe
+          subscriptions = ActivityNotification::Subscription.earliest_subscribed_order
+          expect(subscriptions.size).to eq(4)
+          expect(subscriptions.last).to eq(@subscription_3)
+        end
       end
 
       it "works with key_order scope" do

data/spec/rails_app/app/controllers/articles_controller.rb

@@ -52,7 +52,7 @@ class ArticlesController < ApplicationController
   private
     # Use callbacks to share common setup or constraints between actions.
     def set_article
-      @article = Article.includes(:user).find_by_id(params[:id])
+      @article = Article.includes(:user).find(params[:id])
     end
 
     # Only allow a trusted parameter "white list" through.

data/spec/rails_app/app/controllers/comments_controller.rb

@@ -8,7 +8,6 @@ class CommentsController < ApplicationController
 
     if @comment.save
       @comment.notify :users
-      @comment.notify :admins
       redirect_to @comment.article, notice: 'Comment was successfully created.'
     else
       redirect_to @comment.article

data/spec/rails_app/app/models/admin.rb

@@ -1,9 +1,30 @@
-class Admin < ActiveRecord::Base
-  belongs_to :user
-  validates :user, presence: true
-
-  acts_as_notification_target email_allowed: false,
-    subscription_allowed: true,
-    devise_resource: :user,
-    printable_name: ->(admin) { "admin (#{admin.user.name})" }
+if ENV['AN_TEST_DB'] != 'mongodb'
+  class Admin < ActiveRecord::Base
+    belongs_to :user
+    validates :user, presence: true
+
+    acts_as_notification_target email_allowed: false,
+      subscription_allowed: true,
+      devise_resource: :user,
+      printable_name: ->(admin) { "admin (#{admin.user.name})" }
+  end
+else
+  require 'mongoid'
+  class Admin
+    include Mongoid::Document
+    include Mongoid::Timestamps
+    include GlobalID::Identification
+
+    belongs_to :user
+    validates :user, presence: true
+
+    field :phone_number,   type: String
+    field :slack_username, type: String
+
+    include ActivityNotification::Models
+    acts_as_notification_target email_allowed: false,
+      subscription_allowed: true,
+      devise_resource: :user,
+      printable_name: ->(admin) { "admin (#{admin.user.name})" }
+  end
 end

data/spec/rails_app/app/models/article.rb

@@ -1,17 +1,49 @@
-class Article < ActiveRecord::Base
-  belongs_to :user
-  has_many :comments, dependent: :destroy
-  has_many :commented_users, through: :comments, source: :user
-  validates :user, presence: true
+if ENV['AN_TEST_DB'] != 'mongodb'
+  class Article < ActiveRecord::Base
+    belongs_to :user
+    has_many :comments, dependent: :destroy
+    has_many :commented_users, through: :comments, source: :user
+    validates :user, presence: true
 
-  acts_as_notifiable :users,
-    targets: ->(article) { User.all.to_a - [article.user] },
-    notifier: :user, email_allowed: true,
-    printable_name: ->(article) { "new article \"#{article.title}\"" },
-    dependent_notifications: :delete_all
-  acts_as_notification_group printable_name: ->(article) { "article \"#{article.title}\"" }
+    acts_as_notifiable :users,
+      targets: ->(article) { User.all.to_a - [article.user] },
+      notifier: :user, email_allowed: true,
+      printable_name: ->(article) { "new article \"#{article.title}\"" },
+      dependent_notifications: :delete_all
+    acts_as_notification_group printable_name: ->(article) { "article \"#{article.title}\"" }
 
-  def author?(user)
-    self.user == user
+    def author?(user)
+      self.user == user
+    end
+  end
+else
+  require 'mongoid'
+  class Article
+    include Mongoid::Document
+    include Mongoid::Timestamps
+    include GlobalID::Identification
+
+    belongs_to :user
+    has_many :comments, dependent: :destroy
+    validates :user, presence: true
+
+    field :title, type: String
+    field :body,  type: String
+
+    include ActivityNotification::Models
+    acts_as_notifiable :users,
+      targets: ->(article) { User.all.to_a - [article.user] },
+      notifier: :user, email_allowed: true,
+      printable_name: ->(article) { "new article \"#{article.title}\"" },
+      dependent_notifications: :delete_all
+    acts_as_notification_group printable_name: ->(article) { "article \"#{article.title}\"" }
+
+    def commented_users
+      User.where(:id.in => comments.pluck(:user_id))
+    end
+
+    def author?(user)
+      self.user == user
+    end
   end
 end

data/spec/rails_app/app/models/comment.rb

@@ -1,51 +1,116 @@
-class Comment < ActiveRecord::Base
-  belongs_to :article
-  belongs_to :user
-  validates :article, presence: true
-  validates :user, presence: true
-
-  acts_as_notifiable :users,
-    targets: ->(comment, key) { ([comment.article.user] + comment.article.commented_users.to_a - [comment.user]).uniq },
-    group: :article, notifier: :user, email_allowed: true,
-    parameters: { 'test_default_param' => '1' },
-    notifiable_path: :article_notifiable_path,
-    printable_name: ->(comment) { "comment \"#{comment.body}\"" },
-    dependent_notifications: :update_group_and_delete_all
-
-  require 'custom_optional_targets/console_output'
-  optional_targets = { CustomOptionalTarget::ConsoleOutput => {} }
-  if ENV['OPTIONAL_TARGET_AMAZON_SNS']
-    require 'activity_notification/optional_targets/amazon_sns'
-    if ENV['OPTIONAL_TARGET_AMAZON_SNS_TOPIC_ARN']
+if ENV['AN_TEST_DB'] != 'mongodb'
+  class Comment < ActiveRecord::Base
+    belongs_to :article
+    belongs_to :user
+    validates :article, presence: true
+    validates :user, presence: true
+
+    acts_as_notifiable :users,
+      targets: ->(comment, key) { ([comment.article.user] + comment.article.commented_users.to_a - [comment.user]).uniq },
+      group: :article, notifier: :user, email_allowed: true,
+      parameters: { 'test_default_param' => '1' },
+      notifiable_path: :article_notifiable_path,
+      printable_name: ->(comment) { "comment \"#{comment.body}\"" },
+      dependent_notifications: :update_group_and_delete_all
+
+    require 'custom_optional_targets/console_output'
+    optional_targets = { CustomOptionalTarget::ConsoleOutput => {} }
+    if ENV['OPTIONAL_TARGET_AMAZON_SNS']
+      require 'activity_notification/optional_targets/amazon_sns'
+      if ENV['OPTIONAL_TARGET_AMAZON_SNS_TOPIC_ARN']
+        optional_targets = optional_targets.merge(
+          ActivityNotification::OptionalTarget::AmazonSNS => { topic_arn: ENV['OPTIONAL_TARGET_AMAZON_SNS_TOPIC_ARN'] }
+        )
+      elsif ENV['OPTIONAL_TARGET_AMAZON_SNS_PHONE_NUMBER']
+        optional_targets = optional_targets.merge(
+          ActivityNotification::OptionalTarget::AmazonSNS => { phone_number: :phone_number }
+        )
+      end
+    end
+    if ENV['OPTIONAL_TARGET_SLACK']
+      require 'activity_notification/optional_targets/slack'
       optional_targets = optional_targets.merge(
-        ActivityNotification::OptionalTarget::AmazonSNS => { topic_arn: ENV['OPTIONAL_TARGET_AMAZON_SNS_TOPIC_ARN'] }
+        ActivityNotification::OptionalTarget::Slack  => {
+          webhook_url: ENV['OPTIONAL_TARGET_SLACK_WEBHOOK_URL'], target_username: :slack_username,
+          channel:  ENV['OPTIONAL_TARGET_SLACK_CHANNEL']  || 'activity_notification',
+          username: 'ActivityNotification', icon_emoji: ":ghost:"
+        }
       )
-    elsif ENV['OPTIONAL_TARGET_AMAZON_SNS_PHONE_NUMBER']
+    end
+    acts_as_notifiable :admins, targets: [Admin.first].compact,
+      group: :article, notifier: :user, notifiable_path: :article_notifiable_path,
+      tracked: { only: [:create] },
+      printable_name: ->(comment) { "comment \"#{comment.body}\"" },
+      dependent_notifications: :delete_all,
+      optional_targets: optional_targets
+
+    def article_notifiable_path
+      article_path(article)
+    end
+
+    def author?(user)
+      self.user == user
+    end
+  end
+else
+  require 'mongoid'
+  class Comment
+    include Mongoid::Document
+    include Mongoid::Timestamps
+    include GlobalID::Identification
+
+    belongs_to :article
+    belongs_to :user
+    validates :article, presence: true
+    validates :user, presence: true
+    field :body,  type: String
+
+    include ActivityNotification::Models
+    acts_as_notifiable :users,
+      targets: ->(comment, key) { ([comment.article.user] + comment.article.commented_users.to_a - [comment.user]).uniq },
+      group: :article, notifier: :user, email_allowed: true,
+      parameters: { 'test_default_param' => '1' },
+      notifiable_path: :article_notifiable_path,
+      printable_name: ->(comment) { "comment \"#{comment.body}\"" },
+      dependent_notifications: :update_group_and_delete_all
+
+    require 'custom_optional_targets/console_output'
+    optional_targets = { CustomOptionalTarget::ConsoleOutput => {} }
+    if ENV['OPTIONAL_TARGET_AMAZON_SNS']
+      require 'activity_notification/optional_targets/amazon_sns'
+      if ENV['OPTIONAL_TARGET_AMAZON_SNS_TOPIC_ARN']
+        optional_targets = optional_targets.merge(
+          ActivityNotification::OptionalTarget::AmazonSNS => { topic_arn: ENV['OPTIONAL_TARGET_AMAZON_SNS_TOPIC_ARN'] }
+        )
+      elsif ENV['OPTIONAL_TARGET_AMAZON_SNS_PHONE_NUMBER']
+        optional_targets = optional_targets.merge(
+          ActivityNotification::OptionalTarget::AmazonSNS => { phone_number: :phone_number }
+        )
+      end
+    end
+    if ENV['OPTIONAL_TARGET_SLACK']
+      require 'activity_notification/optional_targets/slack'
       optional_targets = optional_targets.merge(
-        ActivityNotification::OptionalTarget::AmazonSNS => { phone_number: :phone_number }
+        ActivityNotification::OptionalTarget::Slack  => {
+          webhook_url: ENV['OPTIONAL_TARGET_SLACK_WEBHOOK_URL'], target_username: :slack_username,
+          channel:  ENV['OPTIONAL_TARGET_SLACK_CHANNEL']  || 'activity_notification',
+          username: 'ActivityNotification', icon_emoji: ":ghost:"
+        }
       )
     end
-  end
-  if ENV['OPTIONAL_TARGET_SLACK']
-    require 'activity_notification/optional_targets/slack'
-    optional_targets = optional_targets.merge(
-      ActivityNotification::OptionalTarget::Slack  => {
-        webhook_url: ENV['OPTIONAL_TARGET_SLACK_WEBHOOK_URL'], target_username: :slack_username,
-        channel:  ENV['OPTIONAL_TARGET_SLACK_CHANNEL']  || 'activity_notification',
-        username: 'ActivityNotification', icon_emoji: ":ghost:"
-      }
-    )
-  end
-  acts_as_notifiable :admins, targets: Admin.all,
-    group: :article, notifier: :user, notifiable_path: :article_notifiable_path,
-    printable_name: ->(comment) { "comment \"#{comment.body}\"" }, dependent_notifications: :delete_all,
-    optional_targets: optional_targets
+    acts_as_notifiable :admins, targets: [Admin.first].compact,
+      group: :article, notifier: :user, notifiable_path: :article_notifiable_path,
+      tracked: { only: [:create] },
+      printable_name: ->(comment) { "comment \"#{comment.body}\"" },
+      dependent_notifications: :delete_all,
+      optional_targets: optional_targets
 
-  def article_notifiable_path
-    article_path(article)
-  end
+    def article_notifiable_path
+      article_path(article)
+    end
 
-  def author?(user)
-    self.user == user
+    def author?(user)
+      self.user == user
+    end
   end
 end