muck-activities 0.1.27 → 3.0.0

data/test/rails_test/spec/mailers/comment_mailer_spec.rb

@@ -0,0 +1,24 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe CommentMailer do
+
+  describe "deliver emails" do
+
+    def setup
+      ActionMailer::Base.delivery_method = :test
+      ActionMailer::Base.perform_deliveries = true
+      ActionMailer::Base.deliveries = []
+      @expected = TMail::Mail.new
+      @expected.set_content_type "text", "plain", { "charset" => 'utf-8' }
+    end
+    
+    it "should send new comment email" do
+      comment = Factory(:comment)
+      email = CommentMailer.new_comment(comment).deliver
+      ActionMailer::Base.deliveries.should_not be_empty
+      email.to.should == [user.email]
+      email.from.should == [MuckEngine.configuration.from_email]
+    end
+    
+  end  
+end

data/test/rails_test/spec/models/comment_spec.rb

@@ -0,0 +1,60 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Comment do
+  
+  before do
+    @user = Factory(:user)
+    @comment = @user.comments.build(:body => 'a test comment')
+    @comment.user = @user
+    @comment.save!
+  end
+  
+  it { should validate_presence_of :body }
+  it { should belong_to :user }
+  it { should belong_to :commentable }
+  
+  it { should scope_by_newest }
+  it { should scope_by_oldest }
+  it { should scope_newer_than }
+  
+  it "should require body" do
+    assert_no_difference 'Comment.count' do
+      u = Factory.build(:comment, :body => nil)
+      assert !u.valid?
+      assert u.errors.on(:body)
+    end
+  end
+      
+  it "should not have a parent if it is a root Comment" do
+    assert_nil @comment.parent
+  end
+
+  it "should be able to see how many child Comments it has" do
+    0.should == @comment.children.size
+  end
+
+  it "should be able to add child Comments" do
+    # Set commentable to be the same for each comment or else you will get 'Impossible move, target node cannot be inside moved tree.' since the comments would be in a different scope.  (Comments are scoped to commentable)
+    parent = Factory(:comment, :commentable => @user)
+    child = Factory(:comment, :commentable => @user)
+    child.move_to_child_of(parent)
+    1.should == parent.children.size
+  end    
+
+  describe "after having a child added" do
+    before do
+      # Set commentable to be the same for each comment or else you will get 'Impossible move, target node cannot be inside moved tree.' since the comments would be in a different scope.  (Comments are scoped to commentable)
+      @child = Factory(:comment, :commentable => @user)
+      @child.move_to_child_of(@comment)
+    end
+  
+    it "should be able to be referenced by its child" do
+      @comment.should == @child.parent
+    end
+  
+    it "should be able to see its child" do
+      @child.should == @comment.children.first
+    end
+  end
+  
+end

data/test/rails_test/spec/models/user_spec.rb

@@ -0,0 +1,22 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe User do
+
+  describe "A class that is commentable" do
+    before do
+      @user = Factory(:user)
+      @comment = @user.comments.build(:body => 'a test comment')
+      @comment.user = @user
+      @comment.save!
+    end
+    
+    it "should have comments" do
+      @user.comments.length.should == 1
+    end
+    it "should have comment cache" do
+      @user.reload
+      @user.comment_count.should == 1
+    end
+  end
+
+end

data/test/rails_test/spec/spec_helper.rb

@@ -0,0 +1,29 @@
+$:.reject! { |e| e.include? 'TextMate' }
+ENV["RAILS_ENV"] ||= 'test'
+require File.expand_path("../../config/environment", __FILE__)
+require 'rspec/rails'
+require 'muck_test_helper'
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[Rails.root.join("spec/support/**/*.rb")].each {|f| require f}
+
+RSpec.configure do |config|
+  # == Mock Framework
+  #
+  # If you prefer to use mocha, flexmock or RR, uncomment the appropriate line:
+  #
+  # config.mock_with :mocha
+  # config.mock_with :flexmock
+  # config.mock_with :rr
+  config.mock_with :rspec
+
+  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, remove the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = true
+  
+end

data/test/{rails_root → rails_test}/test/functional/activities_controller_test.rb

@@ -1,99 +1,99 @@
-require File.dirname(__FILE__) + '/../test_helper'
+require File.dirname(__FILE__) + '/../spec_helper'
 
 class Muck::ActivitiesControllerTest < ActionController::TestCase
 
   tests Muck::ActivitiesController
 
-  context "activities controller - anonymous" do
-    context "on GET to index" do
-      setup do
+  describe "activities controller - anonymous" do
+    describe "on GET to index" do
+      before do
         @user = Factory(:user)
         @activity = Factory(:activity, :template => 'status_update')
         @user.activities << @activity
       end
-      context 'on GET to index (js)' do
-        setup do
+      describe "on GET to index (js)" do
+        before do
           get :index, :parent_type => @user.class, :parent_id => @user, :format => 'js', :latest_activity_id => @activity.to_param
         end
-        should_respond_with :success
+        it { should respond_with :success }
       end
     end
   end
   
-  context "activities controller - logged in" do
-    setup do
+  describe "activities controller - logged in" do
+    before do
       activate_authlogic
       @user = Factory(:user)
       login_as @user
     end
 
-    context "on GET to index" do
-      setup do
+    describe "on GET to index" do
+      before do
         @activity = Factory(:activity, :template => 'status_update')
         @user.activities << @activity
       end
     
-      context 'on GET to index (js)' do
-        setup do
+      describe "on GET to index (js)" do
+        before do
           get :index, :parent_type => @user.class, :parent_id => @user, :format => 'js', :latest_activity_id => @activity.to_param
         end
-        should_respond_with :success
+        it { should respond_with :success }
       end
 
-      context 'on GET to index (js) no latest activity id' do
-        setup do
+      describe "on GET to index (js) no latest activity id" do
+        before do
           get :index, :parent_type => @user.class, :parent_id => @user, :format => 'js', :latest_activity_id => nil
         end
-        should_respond_with :success
+        it { should respond_with :success }
       end
     end
 
-    context 'fail on POST to create (json)' do
-      setup do
+    describe "fail on POST to create (json)" do
+      before do
         post :create, :activity => { :content => 'test activity' }, :parent_type => @user.class.to_s, :parent_id => @user, :format => 'json'
       end      
-      should_respond_with :success
-      should_not_set_the_flash
-      should "return create json errors" do
+      it { should respond_with :success }
+      it { should_not set_the_flash }
+      it "should return create json errors" do
         json = ActiveSupport::JSON.decode(@response.body).symbolize_keys!
-        assert !json[:success]
-        assert_equal json[:message], I18n.t("muck.activities.create_error", :error => assigns(:activity).errors.full_messages.to_sentence)
+        json[:success].should be_false
+        json[:message].should == I18n.t("muck.activities.create_error", :error => assigns(:activity).errors.full_messages.to_sentence)
       end
     end
         
-    context 'on POST to create (json)' do
-      setup do
+    describe "on POST to create (json)" do
+      before do
         @activity_content = 'test content for my new activity'
         post :create, :activity => { :content => @activity_content, :template => 'status_update' }, :parent_type => @user.class, :parent_id => @user, :format => 'json'
       end      
-      should_respond_with :success
-      should_not_set_the_flash
-      should "return valid create json" do
+      it { should respond_with :success }
+      it { should_not set_the_flash }
+      it "should return valid create json" do
         json = ActiveSupport::JSON.decode(@response.body).symbolize_keys!
-        assert json[:success]
-        assert !json[:is_status_update]
-        assert json[:html].include?(@activity_content)
-        assert_equal json[:message], I18n.t("muck.activities.created")
+        json[:success].should be_true
+        json[:is_status_update].should be_false
+        json[:html].should include(@activity_content)
+        I18n.t("muck.activities.created").should == json[:message]
       end
     end
        
-    context 'on POST to create (js)' do
-      setup do
+    describe "on POST to create (js)" do
+      before do
         post :create, :activity => { :content => 'test activity' }, :parent_type => @user.class, :parent_id => @user, :format => 'js'
       end      
-      should_respond_with :success
-      should_not_set_the_flash
+      it { should respond_with :success }
+      it { should_not set_the_flash }
     end
      
-    context 'on POST to create' do
-      setup do
+    describe "on POST to create" do
+      before do
         post :create, :activity => { :content => 'test activity' }, :parent_type => @user.class, :parent_id => @user
       end      
-      should_redirect_to('show user page (user dashboard)') { @user }
+      it { should redirect_to( @user ) }
     end
 
-    context 'on DELETE to destroy' do
-      setup do
+    describe "on DELETE to destroy" do
+      before do
         @activity = Factory(:activity, :source => @user)
         delete :destroy, :id => @activity.id
       end
@@ -101,53 +101,53 @@ class Muck::ActivitiesControllerTest < ActionController::TestCase
       should_set_the_flash_to(I18n.t("muck.activities.item_removed"))
     end
   
-    context 'on DELETE to destroy (js)' do
-      setup do
+    describe "on DELETE to destroy (js)" do
+      before do
         @activity = Factory(:activity, :source => @user, :is_status_update => false)
         delete :destroy, :id => @activity.id, :format => 'js'
       end
-      should_respond_with :success
-      should_not_set_the_flash
+      it { should respond_with :success }
+      it { should_not set_the_flash }
     end
     
-    context 'on DELETE to destroy, status update (js)' do
-      setup do
+    describe "on DELETE to destroy, status update (js)" do
+      before do
         @activity = Factory(:activity, :source => @user, :is_status_update => true)
         delete :destroy, :id => @activity.id, :format => 'js'
       end
-      should_respond_with :success
-      should_not_set_the_flash
+      it { should respond_with :success }
+      it { should_not set_the_flash }
     end
     
-    context 'on DELETE to destroy (json)' do
-      setup do
+    describe "on DELETE to destroy (json)" do
+      before do
         @activity = Factory(:activity, :source => @user, :is_status_update => false)
         delete :destroy, :id => @activity.id, :format => 'json'
       end
-      should_respond_with :success
-      should_not_set_the_flash
-      should "return valid delete json" do
+      it { should respond_with :success }
+      it { should_not set_the_flash }
+      it "should return valid delete json" do
         json = ActiveSupport::JSON.decode(@response.body).symbolize_keys!
-        assert json[:success]
-        assert !json[:is_status_update]
-        assert json[:html].blank?
-        assert_equal json[:message], I18n.t("muck.activities.item_removed")
+        json[:success].should be_true
+        json[:is_status_update].should be_false
+        json[:html].should be_blank
+        I18n.t("muck.activities.item_removed").should == json[:message]
       end
     end
     
-    context 'on DELETE to destroy, status update (json)' do
-      setup do
+    describe "on DELETE to destroy, status update (json)" do
+      before do
         @activity = Factory(:activity, :source => @user, :is_status_update => true)
         delete :destroy, :id => @activity.id, :format => 'json'
       end
-      should_respond_with :success
-      should_not_set_the_flash
-      should "return valid delete json" do
+      it { should respond_with :success }
+      it { should_not set_the_flash }
+      it "should return valid delete json" do
         json = ActiveSupport::JSON.decode(@response.body).symbolize_keys!
-        assert json[:success]
-        assert json[:is_status_update]
-        assert json[:html].length > 0
-        assert_equal json[:message], I18n.t("muck.activities.item_removed")
+        json[:success].should be_true
+        json[:is_status_update].should be_true
+        json[:html].length.should > 0
+        I18n.t("muck.activities.item_removed").should == json[:message]
       end
     end
   end

data/test/{rails_root → rails_test}/test/functional/default_controller_test.rb

@@ -1,25 +1,25 @@
-require File.dirname(__FILE__) + '/../test_helper'
+require File.dirname(__FILE__) + '/../spec_helper'
 
 class DefaultControllerTest < ActionController::TestCase
 
   tests DefaultController
 
-  context "default controller" do
-    setup do
+  describe "default controller" do
+    before do
       activate_authlogic
       @user = Factory(:user)      
       login_as @user
     end
 
-    context 'on GET to index' do
+    describe "on GET to index" do
       # The default view calls the helpers.  This isn't a great test but
       # it will make sure they don't blow up
-      setup do
+      before do
         @user.activities << Factory(:activity,  :template => 'status_update')
         @user.activities << Factory(:activity,  :template => 'status_update')
         get :index
       end
-      should_respond_with :success
+      it { should respond_with :success }
     end
 
   end

data/test/rails_test/test/unit/activity_feed_test.rb

@@ -0,0 +1,12 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+class ActivityFeedTest < ActiveSupport::TestCase
+  describe "ActivityFeed" do
+    before do
+      @activity_feed = Factory(:activity_feed)
+    end
+    subject { @activity_feed }
+    it { should belong_to :activity }
+    it { should belong_to :ownable }
+  end
+end

data/test/rails_test/test/unit/activity_test.rb

@@ -0,0 +1,139 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Activity do
+
+  before do
+    @activity = Factory(:activity)
+  end
+  
+  it { should validate_presence_of :source }
+  
+  it { should belong_to :item }
+  it { should belong_to :source }
+  it { should have_many :activity_feeds }
+  it { should have_many :comments }
+  
+  it { should scope_newer_than }
+  it { should scope_older_than }
+  it { should scope_by_newest }
+  it { should scope_by_oldest }
+  it { should scope_by_latest }
+  it { should scope_is_public }
+
+  describe "named scopes" do
+    before do
+      @user = Factory(:user)
+      @status_activity = Factory(:activity, :template => 'status')
+      @other_activity = Factory(:activity, :template => 'other')
+      @private_activity = Factory(:activity, :is_public => false)
+      @public_activity = Factory(:activity, :is_public => true)
+      @item_activity = Factory(:activity, :item => @user)
+      @source_activity = Factory(:activity, :source => @user)
+    end
+    describe "after" do
+      #scope :after, lambda { |id| {:conditions => ["activities.id > ?", id] } }
+      it "should only find activites after the given id" do
+        
+      end
+    end
+    describe "status_updates" do
+      before do
+        @status_update = Factory(:activity, :item => @user, :is_status_update => true)
+        @non_status_update = Factory(:activity, :item => @user, :is_status_update => false)
+      end
+      it "should find only status updates" do
+        Activity.status_updates.should include(@status_update)
+        Activity.status_updates.should_not include(@non_status_update)
+      end
+    end
+    describe "filter_by_template" do
+      it "should only find status template" do
+        activities = Activity.filter_by_template('status')
+        activities.should include(@status_activity)
+        activities.should_not include(@other_activity)
+      end
+    end
+    describe "by_item" do
+      it "should find activities by the item they are associated with" do
+        activities = Activity.by_item(@user)
+        activities.should include(@item_activity)
+        activities.should_not include(@public_activity)
+      end
+    end
+    describe "created_by" do
+      it "should find activities by the source they are associated with" do
+        activities = Activity.created_by(@user)
+        activities.should include(@source_activity)
+        activities.should_not include(@public_activity)
+      end
+    end
+  end
+
+  it "should require template or item" do
+    activity = Factory.build(:activity, :template => nil, :item => nil)
+    activity.should_not be_valid
+  end
+
+  it "should get the partial from the template" do
+    template = 'status_update'
+    activity = Factory(:activity, :template => template, :item => nil)
+    activity.partial.should == template, "The activity partial was not set to the specified template"
+  end
+
+  it "should get the partial from the item" do
+    user = Factory(:user)
+    activity = Factory(:activity, :item => user, :template => nil)
+    activity.partial.should == user.class.name.underscore
+  end
+
+  it "should be able to edit the activity" do
+    user = Factory(:user)
+    activity = Factory(:activity, :source => user)
+    activity.can_edit?(user).should be_true
+  end
+  
+  it "should not be able to edit the activity" do
+    user = Factory(:user)
+    activity = Factory(:activity)
+    activity.can_edit?(user).should be_false
+  end
+  
+  it "should filter the activities by template" do
+    @template_name = 'test_template'
+    user = Factory(:user)
+    activity = Factory(:activity, :source => user, :template => @template_name)
+    user.activities << activity
+    user.activities.filter_by_template(@template_name).should include(activity)
+  end
+  
+  it "should only find activities created by the source" do
+    user = Factory(:user)
+    activity = Factory(:activity, :source => user, :template => @template_name)
+    user.activities << activity
+    
+    user2 = Factory(:user)
+    activity2 = Factory(:activity, :source => user2, :template => @template_name)
+    user2.activities << activity2
+    
+    user.activities.created_by(user).should include(activity)
+    user.activities.created_by(user).should_not include(activity2)
+  end
+  
+  describe "comments" do
+    before do
+      @user = Factory(:user)
+      @activity = Factory(:activity)
+      @comment = @activity.comments.build(:body => 'a test comment')
+      @comment.user = @user
+      @comment.save!
+    end
+    it "should have comments" do
+      @activity.comments.length.should == 1
+    end
+    it "should have comment cache" do
+      @activity.reload
+      @activity.comment_count.should == 1
+    end
+  end
+  
+end