fat_free_crm 0.11.3 → 0.11.4

data/spec/lib/view_factory_spec.rb

@@ -0,0 +1,68 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require "fat_free_crm/view_factory"
+
+describe FatFreeCRM::ViewFactory do
+
+  before(:each) do
+    FatFreeCRM::ViewFactory.send(:class_variable_set, '@@views', [])
+  end
+  
+  describe "initialization" do
+  
+    before(:each) do
+      @view_params = {:name => 'brief', :title => 'Brief View', :icon => 'brief.png', :controllers => ['contacts'], :actions => ['show', 'index']}
+    end
+    
+    it "should initialize with required parameters" do
+      view = FatFreeCRM::ViewFactory.new @view_params
+      view.name.should == 'brief'
+      view.title.should == 'Brief View'
+      view.controllers.should include('contacts')
+      view.actions.should include('show')
+      view.actions.should include('index')
+    end
+    
+    it "should register view with ViewFactory" do
+      FatFreeCRM::ViewFactory.send(:class_variable_get, '@@views').size.should == 0
+      FatFreeCRM::ViewFactory.new @view_params
+      FatFreeCRM::ViewFactory.send(:class_variable_get, '@@views').size.should == 1
+    end
+    
+    it "should not register the same view twice" do
+      FatFreeCRM::ViewFactory.new @view_params
+      FatFreeCRM::ViewFactory.new @view_params
+      views = FatFreeCRM::ViewFactory.send(:class_variable_get, '@@views')
+      views.size.should == 1
+    end
+    
+  end
+
+  describe "views_for" do
+    
+    before(:each) do
+      @v1 = FatFreeCRM::ViewFactory.new :name => 'brief', :title => 'Brief View', :controllers => ['contacts'], :actions => ['show', 'index']
+      @v2 = FatFreeCRM::ViewFactory.new :name => 'long', :title => 'Long View', :controllers => ['contacts'], :actions => ['show']
+      @v3 = FatFreeCRM::ViewFactory.new :name => 'full', :title => 'Full View', :controllers => ['accounts'], :actions => ['show']
+    end
+    
+    it "should return 'brief' view for ContactsController#index" do
+      FatFreeCRM::ViewFactory.views_for(:controller => 'contacts', :action => 'index').should == [@v1]
+    end
+    
+    it "should return 'brief' and 'long' view for ContactsController#show" do
+      views = FatFreeCRM::ViewFactory.views_for(:controller => 'contacts', :action => 'show')
+      views.should include(@v1)
+      views.should include(@v2)
+    end
+    
+    it "should return 'full' view for AccountsController#show" do
+      FatFreeCRM::ViewFactory.views_for(:controller => 'accounts', :action => 'show').should == [@v3]
+    end
+    
+    it "should return no views for TasksController#show" do
+      FatFreeCRM::ViewFactory.views_for(:controller => 'tasks', :action => 'show').should == []
+    end
+    
+  end
+  
+end

data/spec/mailers/subscription_mailer_spec.rb

@@ -3,15 +3,29 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 describe SubscriptionMailer do
 
   describe "comment notification" do
-    let(:user) { FactoryGirl.create(:user) }
+    let(:user) { FactoryGirl.create(:user, :email => 'notify_me@example.com') }
     let(:commentable) { FactoryGirl.create(:opportunity, :id => 47, :name => 'Opportunity name') }
     let(:comment) { FactoryGirl.create(:comment, :commentable => commentable) }
     let(:mail) { SubscriptionMailer.comment_notification(user, comment) }
 
-    it "send user password reset url" do
+    before :each do
+      Setting.email_comment_replies.stub(:[]).with(:address).and_return("email_comment_reply@example.com")
+    end
+
+    it "uses email defined in settings as the sender" do
+      mail.from.should eql(["email_comment_reply@example.com"])
+    end
+
+    it "sets user 'notify_me@example.com' as recipient" do
+      mail.to.should eq(["notify_me@example.com"])
+    end
+
+    it "sets the subject" do
       mail.subject.should eq("RE: [opportunity:47] Opportunity name")
-      mail.to.should eq([user.email])
-      mail.body.encoded.should match(polymorphic_url(commentable))
+    end
+
+    it "includes link to opportunity in body" do
+      mail.body.encoded.should match('http://www.example.com/opportunities/47')
     end
   end
 end

data/spec/mailers/user_mailer_spec.rb

@@ -0,0 +1,134 @@
+require 'spec_helper'
+
+describe UserMailer do
+  describe "password_reset_instructions" do
+    let(:user) { FactoryGirl.create(:user, :email => "forgot_my_password@example.com") }
+    let(:mail) { UserMailer.password_reset_instructions(user) }
+
+    before(:each) do
+      I18n.stub(:t).with(:password_reset_instruction).and_return("Password Reset Instructions")
+      user.stub(:perishable_token).and_return("62fe5299b45513f9d22a2e1454f35dd43d62ba50")
+    end
+
+    it "sets fatfree as the sender" do
+      mail.from.should eql(["noreply@fatfreecrm.com"])
+    end
+
+    it "sets user 'forgot_my_password@example.com' as recipient" do
+      mail.to.should eq(["forgot_my_password@example.com"])
+    end
+
+    it "sets the subject" do
+      mail.subject.should eq("Fat Free CRM: Password Reset Instructions")
+    end
+
+    it "includes password reset link in body" do
+      mail.body.encoded.should match("http://www.example.com/passwords/62fe5299b45513f9d22a2e1454f35dd43d62ba50/edit")
+    end
+  end
+
+  describe "assigned_entity_notification" do
+    let(:assigner) { FactoryGirl.create(:user, :first_name => "Bob", :last_name => "Hope") }
+    let(:assignee) { FactoryGirl.create(:user, :email => "assignee@example.com") }
+
+    context "for an account" do
+      let(:account) { FactoryGirl.create(:account, :id => 16, :name => 'Ghostbusters', :user => assigner, :assignee => assignee) }
+      let(:mail) { UserMailer.assigned_entity_notification(account, assigner) }
+
+      it "sets fatfree as the sender" do
+        mail.from.should eql(["notifications@fatfreecrm.com"])
+      end
+
+      it "sets user 'assignee@example.com' as recipient" do
+        mail.to.should eq(["assignee@example.com"])
+      end
+
+      it "sets the subject" do
+        mail.subject.should eq("Fat Free CRM: You have been assigned Ghostbusters Account")
+      end
+
+      it "includes the name of the person who re-assigned the lead in the body" do
+        mail.body.encoded.should match("Bob")
+      end
+
+      it "includes link to the lead in the body" do
+        mail.body.encoded.should match("http://www.example.com/accounts/16")
+      end
+    end
+
+    context "for a contact" do
+      let(:contact) { FactoryGirl.create(:contact, :id => 56, :first_name => 'Harold', :last_name => 'Ramis', :user => assigner, :assignee => assignee) }
+      let(:mail) { UserMailer.assigned_entity_notification(contact, assigner) }
+
+      it "sets fatfree as the sender" do
+        mail.from.should eql(["notifications@fatfreecrm.com"])
+      end
+
+      it "sets user 'assignee@example.com' as recipient" do
+        mail.to.should eq(["assignee@example.com"])
+      end
+
+      it "sets the subject" do
+        mail.subject.should eq("Fat Free CRM: You have been assigned Harold Ramis Contact")
+      end
+
+      it "includes the name of the person who re-assigned the lead in the body" do
+        mail.body.encoded.should match("Bob")
+      end
+
+      it "includes link to the lead in the body" do
+        mail.body.encoded.should match("http://www.example.com/contacts/56")
+      end
+    end
+
+    context "for a lead" do
+      let(:lead) { FactoryGirl.create(:lead, :id => 42, :first_name => 'Bill', :last_name => 'Murray', :user => assigner, :assignee => assignee) }
+      let(:mail) { UserMailer.assigned_entity_notification(lead, assigner) }
+
+      it "sets fatfree as the sender" do
+        mail.from.should eql(["notifications@fatfreecrm.com"])
+      end
+
+      it "sets user 'assignee@example.com' as recipient" do
+        mail.to.should eq(["assignee@example.com"])
+      end
+
+      it "sets the subject" do
+        mail.subject.should eq("Fat Free CRM: You have been assigned Bill Murray Lead")
+      end
+
+      it "includes the name of the person who re-assigned the lead in the body" do
+        mail.body.encoded.should match("Bob")
+      end
+
+      it "includes link to the lead in the body" do
+        mail.body.encoded.should match("http://www.example.com/leads/42")
+      end
+    end
+
+    context "for an opportunity" do
+      let(:opportunity) { FactoryGirl.create(:opportunity, :id => 24, :name => 'Big', :user => assigner, :assignee => assignee) }
+      let(:mail) { UserMailer.assigned_entity_notification(opportunity, assigner) }
+
+      it "sets fatfree as the sender" do
+        mail.from.should eql(["notifications@fatfreecrm.com"])
+      end
+
+      it "sets user 'assignee@example.com' as recipient" do
+        mail.to.should eq(["assignee@example.com"])
+      end
+
+      it "sets the subject" do
+        mail.subject.should eq("Fat Free CRM: You have been assigned Big Opportunity")
+      end
+
+      it "includes the name of the person who re-assigned the lead in the body" do
+        mail.body.encoded.should match("Bob")
+      end
+
+      it "includes link to the lead in the body" do
+        mail.body.encoded.should match("http://www.example.com/opportunities/24")
+      end
+    end
+  end
+end

data/spec/models/entities/account_spec.rb

@@ -36,7 +36,7 @@ describe Account do
     end
 
     it "should return nil when attaching existing asset" do
-      @task = FactoryGirl.create(:task, :asset => @account, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @account, :user => current_user)
       @contact = FactoryGirl.create(:contact)
       @account.contacts << @contact
       @opportunity = FactoryGirl.create(:opportunity)
@@ -48,7 +48,7 @@ describe Account do
     end
 
     it "should return non-empty list of attachments when attaching new asset" do
-      @task = FactoryGirl.create(:task, :user => @current_user)
+      @task = FactoryGirl.create(:task, :user => current_user)
       @contact = FactoryGirl.create(:contact)
       @opportunity = FactoryGirl.create(:opportunity)
 
@@ -64,7 +64,7 @@ describe Account do
     end
 
     it "should discard a task" do
-      @task = FactoryGirl.create(:task, :asset => @account, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @account, :user => current_user)
       @account.tasks.count.should == 1
 
       @account.discard!(@task)
@@ -136,5 +136,45 @@ describe Account do
   describe "permissions" do
     it_should_behave_like Ability, Account
   end
-end
 
+  describe "scopes" do
+    context "visible_on_dashboard" do
+      before :each do
+        @user = FactoryGirl.create(:user)
+        @a1 = FactoryGirl.create(:account, :user => @user)
+        @a2 = FactoryGirl.create(:account, :user => @user, :assignee => FactoryGirl.create(:user))
+        @a3 = FactoryGirl.create(:account, :user => FactoryGirl.create(:user), :assignee => @user)
+        @a4 = FactoryGirl.create(:account, :user => FactoryGirl.create(:user), :assignee => FactoryGirl.create(:user))
+        @a5 = FactoryGirl.create(:account, :user => FactoryGirl.create(:user), :assignee => @user)
+      end
+
+      it "should show accounts which have been created by the user and are unassigned" do
+        Account.visible_on_dashboard(@user).should include(@a1)
+      end
+
+      it "should show accounts which are assigned to the user" do
+        Account.visible_on_dashboard(@user).should include(@a3, @a5)
+      end
+
+      it "should not show accounts which are not assigned to the user" do
+        Account.visible_on_dashboard(@user).should_not include(@a4)
+      end
+
+      it "should not show accounts which are created by the user but assigned" do
+        Account.visible_on_dashboard(@user).should_not include(@a2)
+      end
+    end
+
+    context "by_name" do
+      it "should show accounts ordered by name" do
+        @a1 = FactoryGirl.create(:account, :name => "Account A")
+        @a2 = FactoryGirl.create(:account, :name => "Account Z")
+        @a3 = FactoryGirl.create(:account, :name => "Account J")
+        @a4 = FactoryGirl.create(:account, :name => "Account X")
+        @a5 = FactoryGirl.create(:account, :name => "Account L")
+
+        Account.by_name.should == [@a1, @a3, @a5, @a4, @a2]
+      end
+    end
+  end
+end

data/spec/models/entities/campaign_spec.rb

@@ -40,7 +40,7 @@ describe Campaign do
     end
 
     it "should return nil when attaching existing asset" do
-      @task = FactoryGirl.create(:task, :asset => @campaign, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @campaign, :user => current_user)
       @lead = FactoryGirl.create(:lead, :campaign => @campaign)
       @opportunity = FactoryGirl.create(:opportunity, :campaign => @campaign)
 
@@ -50,7 +50,7 @@ describe Campaign do
     end
 
     it "should return non-empty list of attachments when attaching new asset" do
-      @task = FactoryGirl.create(:task, :user => @current_user)
+      @task = FactoryGirl.create(:task, :user => current_user)
       @lead = FactoryGirl.create(:lead)
       @opportunity = FactoryGirl.create(:opportunity)
 
@@ -81,7 +81,7 @@ describe Campaign do
     end
 
     it "should discard a task" do
-      @task = FactoryGirl.create(:task, :asset => @campaign, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @campaign, :user => current_user)
       @campaign.tasks.count.should == 1
 
       @campaign.discard!(@task)
@@ -136,4 +136,3 @@ describe Campaign do
     it_should_behave_like Ability, Campaign
   end
 end
-

data/spec/models/entities/contact_spec.rb

@@ -91,7 +91,7 @@ describe Contact do
     end
 
     it "should return nil when attaching existing asset" do
-      @task = FactoryGirl.create(:task, :asset => @contact, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @contact, :user => current_user)
       @opportunity = FactoryGirl.create(:opportunity)
       @contact.opportunities << @opportunity
 
@@ -100,7 +100,7 @@ describe Contact do
     end
 
     it "should return non-empty list of attachments when attaching new asset" do
-      @task = FactoryGirl.create(:task, :user => @current_user)
+      @task = FactoryGirl.create(:task, :user => current_user)
       @opportunity = FactoryGirl.create(:opportunity)
 
       @contact.attach!(@task).should == [ @task ]
@@ -114,7 +114,7 @@ describe Contact do
     end
 
     it "should discard a task" do
-      @task = FactoryGirl.create(:task, :asset => @contact, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @contact, :user => current_user)
       @contact.tasks.count.should == 1
 
       @contact.discard!(@task)
@@ -156,9 +156,49 @@ describe Contact do
       end
     end
   end
-
+  
   describe "permissions" do
     it_should_behave_like Ability, Contact
   end
-end
+  
+  describe "text_search" do
+  
+    before(:each) do
+      @contact = FactoryGirl.create(:contact, :first_name => "Bob", :last_name => "Dillion", :email => 'bob_dillion@example.com', :phone => '+1 123 456 789')
+    end
+    
+    it "should search first_name" do
+      Contact.text_search('Bob').should == [@contact]
+    end
 
+    it "should search last_name" do
+      Contact.text_search('Dillion').should == [@contact]
+    end
+    
+    it "should search whole name" do
+      Contact.text_search('Bob Dillion').should == [@contact]
+    end
+    
+    it "should search whole name reversed" do
+      Contact.text_search('Dillion Bob').should == [@contact]
+    end
+
+    it "should search email" do
+      Contact.text_search('example').should == [@contact]
+    end
+    
+    it "should search phone" do
+      Contact.text_search('123').should == [@contact]
+    end
+    
+    it "should not break with a single quote" do
+      contact2 = FactoryGirl.create(:contact, :first_name => "Shamus", :last_name => "O'Connell", :email => 'bob_dillion@example.com', :phone => '+1 123 456 789')
+      Contact.text_search("O'Connell").should == [contact2]
+    end
+    
+    it "should not break on special characters" do
+      Contact.text_search('@$%#^@!').should == []
+    end
+  
+  end
+end

data/spec/models/entities/lead_spec.rb

@@ -47,13 +47,13 @@ describe Lead do
     end
 
     it "should return nil when attaching existing task" do
-      @task = FactoryGirl.create(:task, :asset => @lead, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @lead, :user => current_user)
 
       @lead.attach!(@task).should == nil
     end
 
     it "should return non-empty list of tasks when attaching new task" do
-      @task = FactoryGirl.create(:task, :user => @current_user)
+      @task = FactoryGirl.create(:task, :user => current_user)
 
       @lead.attach!(@task).should == [ @task ]
     end
@@ -65,7 +65,7 @@ describe Lead do
     end
 
     it "should discard a task" do
-      @task = FactoryGirl.create(:task, :asset => @lead, :user => @current_user)
+      @task = FactoryGirl.create(:task, :asset => @lead, :user => current_user)
       @lead.tasks.count.should == 1
 
       @lead.discard!(@task)
@@ -102,4 +102,3 @@ describe Lead do
     it_should_behave_like Ability, Lead
   end
 end
-