RubyGems - commontator - Versions diffs - 5.1.0 → 6.0.0.pre.1 - Mend

commontator 5.1.0 → 6.0.0.pre.1

Files changed (319) hide show

data/spec/lib/commontator_spec.rb~ DELETED

@@ -1,26 +0,0 @@
-require 'rails_helper'
-module Commontator
-  RSpec.describe Commontator, type: :lib do
-    it 'must respond to all attributes' do
-      (ENGINE_ATTRIBUTES + COMMONTATOR_ATTRIBUTES + \
-        COMMONTABLE_ATTRIBUTES).each do |attribute|
-        expect(Commontator).to respond_to(attribute)
-      end
-    end
-    it 'must be configurable' do
-      l1 = lambda { |controller| controller.current_user }
-      l2 = lambda { |controller| controller.current_user }
-      Commontator.configure do |config|
-        config.current_user_proc = l1
-      end
-      expect(Commontator.current_user_proc).to eq l1
-      Commontator.configure do |config|
-        config.current_user_proc = l2
-      end
-      assert_equal(Commontator.current_user_proc, l2)
-    end
-  end
-end

data/spec/mailers/commontator/subscriptions_mailer_spec.rb DELETED

@@ -1,49 +0,0 @@
-require 'rails_helper'
-require 'mailgun_rails'
-module Commontator
-  RSpec.describe SubscriptionsMailer, type: :mailer do
-    before(:each) do
-      setup_mailer_spec
-      @user2 = DummyUser.create
-      @thread.subscribe(@user)
-      @thread.subscribe(@user2)
-      @comment = Comment.new
-      @comment.thread = @thread
-      @comment.creator = @user
-      @comment.body = 'Something'
-      @comment.save!
-      @recipients = @thread.subscribers.reject{|s| s == @user}
-    end
-    it 'must create deliverable mail' do
-      mail = described_class.comment_created(@comment, @recipients)
-      expect(mail.to).to be_nil
-      expect(mail.cc).to be_nil
-      expect(mail.bcc.size).to eq 1
-      expect(mail.bcc).to include(@user2.email)
-      expect(mail.subject).not_to be_empty
-      expect(mail.body).not_to be_empty
-      expect(mail.deliver_now).to eq mail
-    end
-    context 'uses Mailgun' do
-      let(:recipient_variables) do
-        @recipients.each_with_object({}) { |user, memo| memo[user.email] = {} }
-      end
-      before { allow(Rails.application.config.action_mailer).to receive(:delivery_method).and_return(:mailgun) }
-      it 'must create deliverable mail' do
-        mail = SubscriptionsMailer.comment_created(@comment, @recipients)
-        expect(mail.to.size).to eq 1
-        expect(mail.to).to include(@user2.email)
-        expect(mail.cc).to be_nil
-        expect(mail.bcc).to be_nil
-        expect(mail.subject).not_to be_empty
-        expect(mail.body).not_to be_empty
-        expect(mail.deliver_now).to eq mail
-        expect(mail.mailgun_recipient_variables.size).to eq 1
-      end
-    end
-  end
-end

data/spec/mailers/commontator/subscriptions_mailer_spec.rb~ DELETED

@@ -1,30 +0,0 @@
-require 'rails_helper'
-module Commontator
-  RSpec.describe SubscriptionsMailer, type: :mailer do
-    before(:each) do
-      setup_mailer_spec
-      @user2 = DummyUser.create
-      @thread.subscribe(@user)
-      @thread.subscribe(@user2)
-      @comment = Comment.new
-      @comment.thread = @thread
-      @comment.creator = @user
-      @comment.body = 'Something'
-      @comment.save!
-      @recipients = @thread.subscribers.reject{|s| s == @user}
-    end
-    it 'must create deliverable mail' do
-      mail = SubscriptionsMailer.comment_created(@comment, @recipients)
-      expect(mail.to).to eq I18n.t('commontator.email.undisclosed_recipients')
-      expect(mail.cc).to be_nil
-      expect(mail.bcc.size).to eq 1
-      expect(mail.bcc).to include(@user2.email)
-      expect(mail.subject).not_to be_empty
-      expect(mail.body).not_to be_empty
-      expect(mail.deliver).to eq mail
-    end
-  end
-end

data/spec/models/commontator/comment_spec.rb DELETED

@@ -1,82 +0,0 @@
-require 'rails_helper'
-require 'acts_as_votable'
-module Commontator
-  RSpec.describe Comment, type: :model do
-    before(:each) do
-      setup_model_spec
-      @comment = Comment.new
-      @comment.thread = @thread
-      @comment.creator = @user
-      @comment.body = 'Something'
-    end
-    it 'must be votable if acts_as_votable is installed' do
-      expect(Comment).to respond_to(:acts_as_votable)
-      expect(@comment.is_votable?).to eq true
-      expect(@comment.acts_as_votable_initialized).to eq true
-    end
-    it 'must know if it has been modified' do
-      @comment.save!
-      expect(@comment.is_modified?).to eq false
-      @comment.body = 'Something else'
-      @comment.editor = @user
-      @comment.save!
-      expect(@comment.is_modified?).to eq true
-    end
-    it 'must know if it has been deleted' do
-      user = DummyUser.new
-      expect(@comment.is_deleted?).to eq false
-      expect(@comment.editor).to be_nil
-      @comment.delete_by(user)
-      expect(@comment.is_deleted?).to eq true
-      expect(@comment.editor).to eq user
-      @comment.undelete_by(user)
-      expect(@comment.is_deleted?).to eq false
-    end
-    it 'must make proper timestamps' do
-      @comment.save!
-      expect(@comment.created_timestamp).to eq(
-        I18n.t('commontator.comment.status.created_at',
-               created_at: I18n.l(@comment.created_at,
-                                     format: :commontator))
-      )
-      expect(@comment.updated_timestamp).to eq(
-        I18n.t('commontator.comment.status.updated_at',
-               editor_name: @user.name,
-               updated_at: I18n.l(@comment.updated_at,
-                                     format: :commontator))
-      )
-      user2 = DummyUser.create
-      @comment.body = 'Something else'
-      @comment.editor = user2
-      @comment.save!
-      expect(@comment.created_timestamp).to eq(
-        I18n.t('commontator.comment.status.created_at',
-               created_at: I18n.l(@comment.created_at,
-                                     format: :commontator))
-      )
-      expect(@comment.updated_timestamp).to eq(
-        I18n.t('commontator.comment.status.updated_at',
-               editor_name: user2.name,
-               updated_at: I18n.l(@comment.updated_at,
-                                     format: :commontator))
-      )
-    end
-  end
-end

data/spec/models/commontator/comment_spec.rb~ DELETED

@@ -1,82 +0,0 @@
-require 'rails_helper'
-require 'acts_as_votable'
-module Commontator
-  RSpec.describe Comment, type: :model do
-    before(:each) do
-      setup_model_spec
-      @comment = Comment.new
-      @comment.thread = @thread
-      @comment.creator = @user
-      @comment.body = 'Something'
-    end
-    it 'must be votable if acts_as_votable is installed' do
-      Comment.must_respond_to(:acts_as_votable)
-      expect(@comment.is_votable?).to eq true
-      expect(@comment.acts_as_votable_initialized).to eq true
-    end
-    it 'must know if it has been modified' do
-      @comment.save!
-      expect(@comment.is_modified?).to eq false
-      @comment.body = 'Something else'
-      @comment.editor = @user
-      @comment.save!
-      expect(@comment.is_modified?).to eq true
-    end
-    it 'must know if it has been deleted' do
-      user = DummyUser.new
-      expect(@comment.is_deleted?).to eq false
-      expect(@comment.editor).to be_nil
-      @comment.delete_by(user)
-      expect(@comment.is_deleted?).to eq true
-      expect(@comment.editor).to eq user
-      @comment.undelete_by(user)
-      expect(@comment.is_deleted?).to eq false
-    end
-    it 'must make proper timestamps' do
-      @comment.save!
-      expect(@comment.created_timestamp).to eq(
-        I18n.t('commontator.comment.status.created_at',
-               :created_at => I18n.l(@comment.created_at,
-                                     :format => :commontator))
-      )
-      expect(@comment.updated_timestamp).to eq(
-        I18n.t('commontator.comment.status.updated_at',
-               :editor_name => @user.name,
-               :updated_at => I18n.l(@comment.updated_at,
-                                     :format => :commontator))
-      )
-      user2 = DummyUser.create
-      @comment.body = 'Something else'
-      @comment.editor = user2
-      @comment.save!
-      expect(@comment.created_timestamp).to eq(
-        I18n.t('commontator.comment.status.created_at',
-               :created_at => I18n.l(@comment.created_at,
-                                     :format => :commontator))
-      )
-      expect(@comment.updated_timestamp).to eq(
-        I18n.t('commontator.comment.status.updated_at',
-               :editor_name => user2.name,
-               :updated_at => I18n.l(@comment.updated_at,
-                                     :format => :commontator))
-      )
-    end
-  end
-end

data/spec/models/commontator/subscription_spec.rb DELETED

@@ -1,38 +0,0 @@
-require 'rails_helper'
-module Commontator
-  RSpec.describe Subscription, type: :model do
-    before(:each) do
-      setup_model_spec
-      @subscription = Subscription.new
-      @subscription.thread = @thread
-      @subscription.subscriber = @user
-      @subscription.save!
-    end
-    it 'must count unread comments' do
-      expect(@subscription.unread_comments.count).to eq 0
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Something'
-      comment.save!
-      expect(@subscription.reload.unread_comments.count).to eq 1
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Something else'
-      comment.save!
-      expect(@subscription.reload.unread_comments.count).to eq 2
-      @subscription.touch
-      expect(@subscription.reload.unread_comments.count).to eq 0
-    end
-  end
-end

data/spec/models/commontator/subscription_spec.rb~ DELETED

@@ -1,38 +0,0 @@
-require 'rails_helper'
-module Commontator
-  RSpec.describe Subscription, type: :model do
-    before(:each) do
-      setup_model_spec
-      @subscription = Subscription.new
-      @subscription.thread = @thread
-      @subscription.subscriber = @user
-      @subscription.save!
-    end
-    it 'must count unread comments' do
-      expect(@subscription.unread_comments.count).to eq 0
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Something'
-      comment.save!
-      expect(@subscription.reload.unread_comments.count).to eq 1
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Something else'
-      comment.save!
-      expect(@subscription.reload.unread_comments.count).to eq 2
-      @subscription.touch
-      expect(@subscription.reload.unread_comments.count).to eq 0
-    end
-  end
-end

data/spec/models/commontator/thread_spec.rb DELETED

@@ -1,159 +0,0 @@
-require 'rails_helper'
-module Commontator
-  RSpec.describe Thread, type: :model do
-    before(:each) do
-      setup_model_spec
-    end
-    it 'must have a config' do
-      expect(@thread.config).to be_a(CommontableConfig)
-      @thread.update_attribute(:commontable_id, nil)
-      expect(Thread.find(@thread.id).config).to eq Commontator
-    end
-    it 'must order all comments' do
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Something'
-      comment.save!
-      comment2 = Comment.new
-      comment2.thread = @thread
-      comment2.creator = @user
-      comment2.body = 'Something else'
-      comment2.save!
-      comments = @thread.comments
-      ordered_comments = @thread.ordered_comments
-      comments.each { |c| expect(ordered_comments).to include(c) }
-      ordered_comments.each { |oc| expect(comments).to include(oc) }
-    end
-    it 'must list all subscribers' do
-      @thread.subscribe(@user)
-      @thread.subscribe(DummyUser.create)
-      @thread.subscriptions.each do |sp|
-        expect(@thread.subscribers).to include(sp.subscriber)
-      end
-    end
-    it 'must find the subscription for each user' do
-      @thread.subscribe(@user)
-      user2 = DummyUser.create
-      @thread.subscribe(user2)
-      subscription = @thread.subscription_for(@user)
-      expect(subscription.thread).to eq @thread
-      expect(subscription.subscriber).to eq @user
-      subscription = @thread.subscription_for(user2)
-      expect(subscription.thread).to eq @thread
-      expect(subscription.subscriber).to eq user2
-    end
-    it 'must know if it is closed' do
-      expect(@thread.is_closed?).to eq false
-      @thread.close(@user)
-      expect(@thread.is_closed?).to eq true
-      expect(@thread.closer).to eq @user
-      @thread.reopen
-      expect(@thread.is_closed?).to eq false
-    end
-    it 'must mark comments as read' do
-      @thread.subscribe(@user)
-      subscription = @thread.subscription_for(@user)
-      expect(subscription.unread_comments.count).to eq 0
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Something'
-      comment.save!
-      expect(subscription.reload.unread_comments.count).to eq 1
-      @thread.mark_as_read_for(@user)
-      expect(subscription.reload.unread_comments.count).to eq 0
-    end
-    it 'must be able to clear comments' do
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Something'
-      comment.save!
-      @thread.close(@user)
-      expect(@thread.commontable).to eq @commontable
-      expect(@thread.comments).to include(comment)
-      expect(@thread.is_closed?).to eq true
-      expect(@thread.closer).to eq @user
-      @commontable = DummyModel.find(@commontable.id)
-      expect(@commontable.thread).to eq @thread
-      @thread.clear
-      expect(@thread.commontable).to be_nil
-      expect(@thread.comments).to include(comment)
-      @commontable = DummyModel.find(@commontable.id)
-      expect(@commontable.thread).not_to be_nil
-      expect(@commontable.thread).not_to eq @thread
-      expect(@commontable.thread.comments).not_to include(comment)
-    end
-    it 'must preserve the thread and comments by default when commontable is gone' do
-      comment = Comment.new
-      comment.thread = @thread
-      comment.creator = @user
-      comment.body = 'Undead'
-      comment.save!
-      comment.reload
-      expect(Thread.find(@thread.id)).to eq @thread
-      expect(Comment.find(comment.id)).to eq comment
-      @commontable.destroy!
-      expect(Thread.find(@thread.id)).to eq @thread
-      expect(Comment.find(comment.id)).to eq comment
-    end
-    it 'must delete the thread and comments when commontable has dependent :destroy' do
-      commontable = DummyDependentModel.create
-      thread = commontable.thread
-      comment = Comment.new
-      comment.thread = thread
-      comment.creator = @user
-      comment.body = 'Undead'
-      comment.save!
-      comment.reload
-      expect(Thread.find(thread.id)).to eq thread
-      expect(Comment.find(comment.id)).to eq comment
-      commontable.destroy!
-      expect{Thread.find(thread.id)}.to raise_exception(ActiveRecord::RecordNotFound)
-      expect{Comment.find(comment.id)}.to raise_exception(ActiveRecord::RecordNotFound)
-    end
-    it 'must return nil subscription for nil or false subscriber' do
-      expect(@thread.subscription_for(nil)).to eq nil
-      expect(@thread.subscription_for(false)).to eq nil
-    end
-  end
-end