inboxes 0.2.0 → 0.2.1

data/spec/fake_app.rb

@@ -0,0 +1,97 @@
+require 'active_record'
+require 'action_controller/railtie'
+require 'action_view/railtie'
+
+require "cancan"
+require "cancan/ability"
+require "cancan/controller_resource"
+require "cancan/controller_additions"
+
+require 'devise'
+require 'devise/orm/active_record'
+
+# database
+ActiveRecord::Base.configurations = {'test' => {:adapter => 'sqlite3', :database => ':memory:'}}
+ActiveRecord::Base.establish_connection('test')
+
+# config
+app = Class.new(Rails::Application)
+app.config.secret_token = "3b7cd727ee24e8444053437c36cc66c4"
+app.config.session_store :cookie_store, :key => "_myapp_session"
+app.config.active_support.deprecation = :log
+app.initialize!
+
+require 'devise_config'
+
+# models
+class User < ActiveRecord::Base
+  devise :database_authenticatable, :registerable,
+         :recoverable, :rememberable, :trackable, :validatable
+  attr_accessible :email, :password, :password_confirmation, :remember_me, :name
+  validates :name, :presence => true, :uniqueness => true
+  has_inboxes
+end
+
+# routes
+app.routes.draw do
+  devise_for :users
+end
+
+#migrations
+ActiveRecord::Base.silence do
+  ActiveRecord::Migration.verbose = false
+  ActiveRecord::Schema.define :version => 0 do
+    create_table "users", :force => true do |t|
+      t.string   "email",                                 :default => "", :null => false
+      t.string   "encrypted_password",     :limit => 128, :default => "", :null => false
+      t.string   "reset_password_token"
+      t.datetime "reset_password_sent_at"
+      t.datetime "remember_created_at"
+      t.integer  "sign_in_count",                         :default => 0
+      t.datetime "current_sign_in_at"
+      t.datetime "last_sign_in_at"
+      t.string   "current_sign_in_ip"
+      t.string   "last_sign_in_ip"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.string   "name"
+    end
+    
+    create_table "discussions", :force => true do |t|
+      t.integer  "user_id"
+      t.integer  "messages_count", :default => 0
+      t.boolean  "is_private",     :default => true
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    create_table "messages", :force => true do |t|
+      t.integer  "user_id"
+      t.integer  "discussion_id"
+      t.text     "body"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    create_table "speakers", :force => true do |t|
+      t.integer  "user_id"
+      t.integer  "discussion_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+  end
+end
+
+# controllers
+class ApplicationController < ActionController::Base
+  before_filter :assign_unread_discussions
+  
+  private
+  
+  def assign_unread_discussions
+    @unread_discussions_count = Discussion.unread_for(current_user).count if user_signed_in?
+  end
+end
+
+# helpers
+Object.const_set(:ApplicationHelper, Module.new)

data/spec/fake_gem.rb

@@ -0,0 +1,6 @@
+# Simulate a gem providing a subclass of ActiveRecord::Base before the Railtie is loaded.
+
+require 'active_record'
+
+class GemDefinedModel < ActiveRecord::Base
+end

data/spec/inboxes/discussion_spec.rb

@@ -0,0 +1,104 @@
+require 'spec_helper'
+
+describe Discussion do
+
+  it "should create valid group discussion" do
+    discussion = Discussion.new
+    discussion.recipient_ids = [Factory(:user), Factory(:user), Factory(:user)].map { |u| u.id }
+    discussion.save.should be true
+    
+    discussion.users.count.should be == 3
+    # discussion.private?.should be false
+  end
+  
+  it "should create valid private discussion" do
+    discussion = Discussion.new
+    discussion.recipient_ids = [Factory(:user).id, Factory(:user).id]
+    discussion.save.should be true
+    discussion.private?.should be true
+  end
+  
+  it "should add messages to valid discussion" do
+    discussion = Discussion.new
+    discussion.recipient_ids = [Factory(:user), Factory(:user)].map { |u| u.id }
+    discussion.save.should be true
+    
+    message = Message.new(:user => Factory(:user), :body => Factory.next(:string), :discussion => discussion)
+    message.save.should be true
+  end
+  
+  it "should not create discussion without repicients" do
+    discussion = Discussion.new
+    discussion.save.should be false
+  end
+  
+  it "should not add speakers if discussion is not saved" do
+    discussion = Discussion.new
+    lambda { discussion.add_speaker(Factory(:user)) }.should raise_error(ArgumentError)
+  end
+  
+  it "should add speakers to discussion" do
+    discussion = Factory(:discussion)
+    user = Factory(:user)
+    discussion.add_speaker(user)
+    discussion.users.should include user
+  end
+  
+  it "should remove speaker from discussion" do
+    discussion = Factory(:discussion)
+    user = Factory(:user)
+    discussion.add_speaker(user)
+    discussion.remove_speaker(user)
+    
+    discussion.users.should_not include user
+  end
+  
+  it "should assign discussion as viewed for user" do
+    user = Factory(:user)
+    discussion = Factory(:discussion)
+    discussion.mark_as_read_for(user)
+    discussion.unread_for?(user).should be false
+  end
+  
+  it "model with new message should be unread for user" do
+    discussion = Factory(:discussion)
+    message = Message.create!(:user => Factory(:user), :body => Factory.next(:string), :discussion => discussion)
+    discussion.unread_for?(Factory(:user)).should be true
+  end
+  
+  it "attribute user_invited_at should be right" do
+    discussion = Factory(:discussion)
+    user = Factory(:user)
+    speaker = discussion.add_speaker(user)
+    speaker.created_at.to_i.should be == discussion.user_invited_at(user).to_i
+  end
+  
+  it "method exists_between_users? should be right" do
+    user = Factory(:user)
+    user2 = Factory(:user)
+    discussion = Discussion.create!(:recipient_ids => [user.id, user2.id])
+    
+    Discussion.find_between_users(user, user2).should be == discussion
+    Discussion.find_between_users(user2, user).should be == discussion
+  end
+  
+  it "method exists_between_users? should be right if there is no discussion" do
+    user = Factory(:user)
+    user2 = Factory(:user)
+    
+    Discussion.find_between_users(user, user2).should be_nil
+    Discussion.find_between_users(user2, user).should be_nil
+  end
+  
+  it "method exists_between_users? should be right if 2 users are in group discussion" do
+    user = Factory(:user)
+    user2 = Factory(:user)
+    user3 = Factory(:user)
+    discussion = Discussion.create!(:recipient_ids => [user.id, user2.id, user3.id])
+    
+    Discussion.find_between_users(user, user2).should be_nil
+    Discussion.find_between_users(user2, user).should be_nil
+    Discussion.find_between_users(user2, user3).should be_nil
+  end
+  
+end

data/spec/inboxes/discussions_controller_spec.rb

@@ -0,0 +1,96 @@
+require 'spec_helper'
+
+describe Inboxes::DiscussionsController do
+  context "Guest" do
+    it "should not see discussions list" do
+      get :index
+      response.should redirect_to(sign_in_url)
+    end
+    
+    it "should not see new action" do
+      get :new
+      response.should redirect_to(sign_in_url)
+    end
+    
+    it "should not create discussion if model is valid" do
+      recipient_ids = [Factory(:user).id, Factory(:user).id]
+      post(:create,
+        :discussion => {
+          :recipient_ids => recipient_ids,
+          :messages_attributes => {
+            0 => {:body => Factory.next(:string)}
+          }
+        } 
+      )
+      
+      response.should redirect_to(sign_in_url)
+    end
+
+  end
+  
+  context("Authenticated admin") do
+    before(:each) do
+      @request.env["devise.mapping"] = Devise.mappings[:user]
+      @user = Factory(:user)
+      @user.set_role(:admin)
+      sign_in @user
+    end
+    
+    it "should see discussions list" do
+      get :index
+      response.should render_template(:index)
+    end
+    
+    it "should see new action" do
+      get :new
+      response.should render_template(:new)
+    end
+    
+    it "should open discussion" do
+      discussion = Factory.build(:discussion)
+      discussion.recipient_ids = [@user, Factory(:user)].map { |u| u.id }
+      discussion.save.should be true
+      
+      get(:show, :id => discussion)
+      response.status.should be 200
+    end
+    
+    it "should create private discussion if model is valid" do
+      recipient_ids = [Factory(:user).id, Factory(:user).id]
+      post(:create,
+        :discussion => {
+          :recipient_ids => recipient_ids,
+          :messages_attributes => {
+            0 => {:body => Factory.next(:string)}
+          }
+        } 
+      )
+      
+      response.should redirect_to(discussion_url(assigns[:discussion]))
+    end
+    
+    it "should create group discussion if model is valid" do
+      recipient_ids = [Factory(:user).id, Factory(:user).id, Factory(:user).id]
+      post(:create,
+        :discussion => {
+          :recipient_ids => recipient_ids,
+          :messages_attributes => {
+            0 => {:body => Factory.next(:string)}
+          }
+        } 
+      )
+      
+      response.should redirect_to(discussion_url(assigns[:discussion]))
+    end
+    
+    it "should not create discussion with empty message" do
+      discussion = Discussion.new
+      discussion.recipient_ids = [Factory(:user).id, Factory(:user).id, Factory(:user).id]
+      post(:create, :discussion => discussion)
+      
+      response.should render_template(:new)
+    end
+    
+  end
+
+end

data/spec/inboxes/discussions_routing_spec.rb

@@ -0,0 +1,23 @@
+require "spec_helper"
+
+describe Inboxes::DiscussionsController do
+  describe "routing" do
+
+    it "routes to #index" do
+      get("/discussions").should route_to("inboxes/discussions#index")
+    end
+
+    it "routes to #new" do
+      get("/discussions/new").should route_to("inboxes/discussions#new")
+    end
+
+    it "routes to #show" do
+      get("/discussions/1").should route_to("inboxes/discussions#show", :id => "1")
+    end
+
+    it "routes to #create" do
+      post("/discussions").should route_to("inboxes/discussions#create")
+    end
+
+  end
+end

data/spec/inboxes/message_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe Message do
+  it "should not be visible to new speaker" do
+    discussion = Factory(:discussion)
+    old_user = Factory(:user)
+    discussion.add_speaker(old_user)
+    message = Message.create!(:discussion => discussion, :user => old_user, :body => Factory.next(:string))
+    
+    sleep 2
+    
+    new_user = Factory(:user)
+    discussion.add_speaker(new_user)
+    message.visible_for?(new_user).should be_false
+  end
+  
+  it "should be visible to old speaker" do
+    discussion = Factory(:discussion)
+    user = Factory(:user)
+    discussion.add_speaker(user)
+    message = Message.create!(:discussion => discussion, :user => user, :body => Factory.next(:string))
+    
+    sleep 5
+    
+    message.visible_for?(user).should be_true
+  end
+end

data/spec/inboxes/messages_controller_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+
+describe Inboxes::MessagesController do
+
+  render_views
+  
+  context "Guest" do
+    it "should redirect guest if he wants to create message" do
+      discussion = Factory(:discussion)
+      # puts discussion.id
+      post :create, :discussion_id => discussion.id
+      response.should redirect_to(sign_in_url)
+    end
+  end
+  
+  context "Authenticated admin" do
+    before(:each) do
+      @request.env["devise.mapping"] = Devise.mappings[:user]
+      @admin = Factory(:admin)
+      @admin.set_role(:admin)
+      sign_in @admin
+    end
+
+    it "create action should redirect to discussion when model is valid" do
+      Message.any_instance.stubs(:valid?).returns(true)
+      message = Factory(:message)
+      user = Factory(:user)
+      discussion = Factory(:discussion, :recipient_ids => [@admin.id, user.id])
+      post(:create, :discussion_id => discussion.id)
+      response.should redirect_to(discussion_url(discussion))
+    end
+
+    # it "create action should assign flash with error message" do
+    #   Comment.any_instance.stubs(:valid?).returns(false)
+    #   first_post = Factory(:post)
+    #   post :create, :post_id => first_post
+    # 
+    #   flash[:notice].should =~ /Введите текст комментария!/i
+    # end
+  end
+  
+end

data/spec/inboxes/speaker_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe Speaker do
+  # it "should create valid model" do
+  #   speaker = Speaker.new
+  #   speaker.user = Factory(:user)
+  #   speaker.discussion = Factory(:discussion)
+  #   speaker.save.should be true
+  # end
+  # 
+  # it "should bot create model without discussion" do
+  #   speaker = Speaker.new
+  #   speaker.user = Factory(:user)
+  #   speaker.save.should be false
+  # end
+end

data/spec/inboxes/speakers_controller_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+describe Inboxes::SpeakersController do
+
+  context("Authenticated admin") do
+    before(:each) do
+      @request.env["devise.mapping"] = Devise.mappings[:user]
+      @user = Factory(:user)
+      @user.set_role(:admin)
+      sign_in @user
+    end
+
+    it "should add speaker to discussion" do
+      discussion = Factory(:discussion, :recipient_ids => [@user.id, Factory(:user).id])
+      post(:create,
+        :discussion_id => discussion,
+        :speaker => {
+          :user_id => Factory(:user).id
+        }
+      )
+      response.should redirect_to(discussion_url(discussion))
+      flash[:notice].should =~ /Собеседник успешно добавлен/i
+    end
+    
+    # it "should not add bad speaker to discussion" do
+    #   discussion = Factory(:discussion)
+    #   
+    #   lambda {
+    #     post(:create,
+    #       :discussion_id => discussion
+    #     )
+    #   }.should raise_error(ActiveRecord::RecordNotFound)
+    # end
+  end
+  
+  context("User") do
+    before(:each) do
+      @request.env["devise.mapping"] = Devise.mappings[:user]
+      @user = Factory(:user)
+      @user.set_role(:user)
+      sign_in @user
+    end
+    
+    it "should add speaker to discussion if he is participant if this discussion" do
+      Speaker.any_instance.stubs(:valid?).returns(true)
+      discussion = Factory(:discussion, :recipient_ids => [@user.id, Factory(:user).id])
+      # puts discussion.can_participate?(@user)
+      # new_user = Factory(:user)
+      post(:create,
+        :discussion_id => discussion,
+        :speaker => {
+          :user_id => Factory(:user).id
+        }
+      )
+      response.should redirect_to(discussion_url(discussion))
+      # response.should redirect_to(root_url)
+      flash[:notice].should =~ /Собеседник успешно добавлен/i
+    end
+    
+    # дописать спек
+    it "should not add speaker to discussion if he is not participant if this discussion" do
+      discussion = Factory(:discussion)
+      post(:create,
+        :discussion_id => discussion,
+        :speaker => {
+          :user_id => Factory(:user).id
+        }
+      )
+      response.should redirect_to(root_url)
+    end
+  end
+end