reduced_fat_crm 0.14.0 → 0.15.0.beta

data/spec/support/auth_macros.rb

@@ -25,7 +25,14 @@ alias :require_user :login
 
 #----------------------------------------------------------------------------
 def login_and_assign(user_stubs = {}, session_stubs = {})
-  login(user_stubs, session_stubs)
+  User.current_user = @current_user = FactoryGirl.build_stubbed(:user, user_stubs)
+  @current_user_session = double(Authentication, {record: current_user}.merge(session_stubs))
+  allow(Authentication).to receive(:find).and_return(@current_user_session)
+  #set_timezone
+  assigns[:current_user] = current_user
+end
+def login_and_assign!(user_stubs = {}, session_stubs = {})
+  login
   assigns[:current_user] = current_user
 end
 

data/spec/views/accounts/_edit.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/accounts/_edit" do
 
   before do
     login_and_assign
-    assign(:account, @account = FactoryGirl.create(:account))
+    assign(:account, @account = FactoryGirl.build_stubbed(:account))
     assign(:users, [current_user])
   end
 

data/spec/views/accounts/create.js.haml_spec.rb

@@ -16,7 +16,7 @@ describe "/accounts/create" do
   # core object Account partial is not embedded.
   describe "create success" do
     before do
-      assign(:account, @account = FactoryGirl.create(:account))
+      assign(:account, @account = FactoryGirl.build_stubbed(:account))
       assign(:accounts, [@account].paginate)
       assign(:account_category_total, Hash.new(1))
       render

data/spec/views/accounts/destroy.js.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/accounts/destroy" do
 
   before do
     login_and_assign
-    assign(:account, @account = FactoryGirl.create(:account))
+    assign(:account, @account = FactoryGirl.build_stubbed(:account))
     assign(:accounts, [@account].paginate)
     assign(:account_category_total, Hash.new(1))
     render

data/spec/views/accounts/edit.js.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/accounts/edit" do
 
   before do
     login_and_assign
-    assign(:account, @account = FactoryGirl.create(:account, user: current_user))
+    assign(:account, @account = FactoryGirl.build_stubbed(:account, user: current_user))
     assign(:users, [current_user])
   end
 
@@ -31,7 +31,7 @@ describe "/accounts/edit" do
 
   it "edit: should hide previously open [Edit Account] for and replace it with account partial" do
     params[:cancel] = nil
-    assign(:previous, previous = FactoryGirl.create(:account, user: current_user))
+    assign(:previous, previous = FactoryGirl.build_stubbed(:account, user: current_user))
 
     render
     expect(rendered).to include("account_#{previous.id}")

data/spec/views/accounts/index.haml_spec.rb

@@ -17,13 +17,13 @@ describe "/accounts/index" do
   end
 
   it "should render account name" do
-    assign(:accounts, [FactoryGirl.create(:account, name: 'New Media Inc'), FactoryGirl.create(:account)].paginate)
+    assign(:accounts, [FactoryGirl.build_stubbed(:account, name: 'New Media Inc'), FactoryGirl.build_stubbed(:account)].paginate)
     render
     expect(rendered).to have_tag('a', text: "New Media Inc")
   end
 
   it "should render list of accounts if list of accounts is not empty" do
-    assign(:accounts, [FactoryGirl.create(:account), FactoryGirl.create(:account)].paginate)
+    assign(:accounts, [FactoryGirl.build_stubbed(:account), FactoryGirl.build_stubbed(:account)].paginate)
 
     render
     expect(view).to render_template(partial: "_account")

data/spec/views/accounts/index.js.haml_spec.rb

@@ -13,7 +13,7 @@ describe "/accounts/index" do
   end
 
   it "should render [account] template with @accounts collection if there are accounts" do
-    assign(:accounts, [FactoryGirl.create(:account, id: 42)].paginate)
+    assign(:accounts, [FactoryGirl.build_stubbed(:account, id: 42)].paginate)
 
     render template: 'accounts/index', formats: [:js]
 

data/spec/views/accounts/update.js.haml_spec.rb

@@ -11,7 +11,7 @@ describe "/accounts/update" do
   before do
     login_and_assign
 
-    assign(:account, @account = FactoryGirl.create(:account, user: current_user))
+    assign(:account, @account = FactoryGirl.build_stubbed(:account, user: current_user))
     assign(:users, [current_user])
     assign(:account_category_total, Hash.new(1))
   end

data/spec/views/admin/field_groups/create.js.haml_spec.rb

@@ -11,7 +11,7 @@ describe "admin/field_groups/create" do
     assign(:field_group, field_group)
   end
 
-  let(:field_group) { FactoryGirl.create(:field_group, label: 'test') }
+  let(:field_group) { FactoryGirl.build_stubbed(:field_group, label: 'test') }
 
   it "renders javascript" do
     render

data/spec/views/admin/field_groups/edit.js.haml_spec.rb

@@ -11,7 +11,7 @@ describe "admin/field_groups/edit" do
     assign(:field_group, field_group)
   end
 
-  let(:field_group) { FactoryGirl.create(:field_group, label: 'test') }
+  let(:field_group) { FactoryGirl.build_stubbed(:field_group, label: 'test') }
 
   it "renders javascript" do
     render

data/spec/views/admin/field_groups/new.js.haml_spec.rb

@@ -11,7 +11,7 @@ describe "admin/field_groups/new" do
     assign(:field_group, field_group)
   end
 
-  let(:field_group) { FactoryGirl.create(:field_group, label: 'test') }
+  let(:field_group) { FactoryGirl.build_stubbed(:field_group, label: 'test') }
 
   it "renders javascript" do
     render

data/spec/views/admin/field_groups/update.js.haml_spec.rb

@@ -11,7 +11,7 @@ describe "admin/field_groups/update" do
     assign(:field_group, field_group)
   end
 
-  let(:field_group) { FactoryGirl.create(:field_group, label: 'test') }
+  let(:field_group) { FactoryGirl.build_stubbed(:field_group, label: 'test') }
 
   it "renders javascript" do
     render

data/spec/views/admin/users/create.js.haml_spec.rb

@@ -12,7 +12,7 @@ describe "admin/users/create" do
 
   describe "create success" do
     before do
-      assign(:user, @user = FactoryGirl.create(:user))
+      assign(:user, @user = FactoryGirl.build_stubbed(:user))
       assign(:users, [@user]) # .paginate
     end
 

data/spec/views/admin/users/destroy.js.haml_spec.rb

@@ -26,7 +26,7 @@ describe "admin/users/destroy" do
 
   describe "user was not deleted" do
     before do
-      assign(:user, @user = FactoryGirl.create(:user))
+      assign(:user, @user = FactoryGirl.build_stubbed(:user))
     end
 
     it "should remove confirmation panel" do

data/spec/views/admin/users/edit.js.haml_spec.rb

@@ -8,7 +8,7 @@ require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
 describe "admin/users/edit" do
   before do
     login_and_assign(admin: true)
-    assign(:user, @user = FactoryGirl.create(:user))
+    assign(:user, @user = FactoryGirl.build_stubbed(:user))
   end
 
   it "cancel replaces [Edit User] form with user partial" do
@@ -19,7 +19,7 @@ describe "admin/users/edit" do
   end
 
   it "edit hides previously open [Edit User] and replaces it with user partial" do
-    assign(:previous, previous = FactoryGirl.create(:user))
+    assign(:previous, previous = FactoryGirl.build_stubbed(:user))
     render
 
     expect(rendered).to include("user_#{previous.id}")

data/spec/views/admin/users/index.haml_spec.rb

@@ -11,7 +11,7 @@ describe "/admin/users/index" do
   end
 
   it "renders a list of users" do
-    assign(:users, [FactoryGirl.create(:user)].paginate)
+    assign(:users, [FactoryGirl.build_stubbed(:user)].paginate)
 
     render
     expect(view).to render_template(partial: "_user")

data/spec/views/admin/users/index.js.haml_spec.rb

@@ -11,8 +11,8 @@ describe "admin/users/index" do
   end
 
   it "renders [admin/user] template with @users collection" do
-    amy = FactoryGirl.create(:user)
-    bob = FactoryGirl.create(:user)
+    amy = FactoryGirl.build_stubbed(:user)
+    bob = FactoryGirl.build_stubbed(:user)
     assign(:users, [amy, bob].paginate)
 
     render template: 'admin/users/index', formats: [:js]

data/spec/views/admin/users/reactivate.js.haml_spec.rb

@@ -8,7 +8,7 @@ require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
 describe "admin/users/reactivate" do
   before do
     login_and_assign(admin: true)
-    assign(:user, @user = FactoryGirl.create(:user, suspended_at: Time.now.yesterday))
+    assign(:user, @user = FactoryGirl.build_stubbed(:user, suspended_at: Time.now.yesterday))
   end
 
   it "reloads the requested user partial" do

data/spec/views/admin/users/suspend.js.haml_spec.rb

@@ -8,7 +8,7 @@ require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
 describe "admin/users/suspend" do
   before do
     login_and_assign(admin: true)
-    assign(:user, @user = FactoryGirl.create(:user, suspended_at: nil))
+    assign(:user, @user = FactoryGirl.build_stubbed(:user, suspended_at: nil))
   end
 
   it "reloads the requested user partial" do

data/spec/views/admin/users/update.js.haml_spec.rb

@@ -8,7 +8,7 @@ require 'spec_helper'
 describe "admin/users/update" do
   before do
     login_and_assign(admin: true)
-    assign(:user, @user = FactoryGirl.create(:user))
+    assign(:user, @user = FactoryGirl.build_stubbed(:user))
   end
 
   describe "no errors:" do

data/spec/views/application/auto_complete.haml_spec.rb

@@ -15,11 +15,11 @@ describe "/application/_auto_complete" do
   [:account, :campaign, :contact, :lead, :opportunity].each do |model|
     it "should render autocomplete list if #{model} matches found" do
       @auto_complete = if model == :lead
-                         FactoryGirl.create(:lead, first_name: "Billy", last_name: "Bones", company: "Hello, World!")
+                         FactoryGirl.build_stubbed(:lead, first_name: "Billy", last_name: "Bones", company: "Hello, World!")
                        elsif model == :contact
-                         FactoryGirl.create(:contact, first_name: "Billy", last_name: "Bones")
+                         FactoryGirl.build_stubbed(:contact, first_name: "Billy", last_name: "Bones")
                        else
-                         FactoryGirl.create(model, name: "Hello, World!")
+                         FactoryGirl.build_stubbed(model, name: "Hello, World!")
       end
       assign(:auto_complete, [@auto_complete])
 

data/spec/views/campaigns/_edit.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/campaigns/_edit" do
 
   before do
     login_and_assign
-    assign(:campaign, @campaign = FactoryGirl.create(:campaign))
+    assign(:campaign, @campaign = FactoryGirl.build_stubbed(:campaign))
     assign(:users, [current_user])
   end
 

data/spec/views/campaigns/create.js.haml_spec.rb

@@ -12,7 +12,7 @@ describe "/campaigns/create" do
 
   describe "create success" do
     before do
-      assign(:campaign, @campaign = FactoryGirl.create(:campaign))
+      assign(:campaign, @campaign = FactoryGirl.build_stubbed(:campaign))
       assign(:campaigns, [@campaign].paginate)
       assign(:campaign_status_total, Hash.new(1))
       render
@@ -37,7 +37,7 @@ describe "/campaigns/create" do
   describe "create failure" do
     it "should re-render [create] template in :create_campaign div" do
       assign(:campaign, FactoryGirl.build(:campaign, name: nil)) # make it invalid
-      assign(:users, [FactoryGirl.create(:user)])
+      assign(:users, [FactoryGirl.build_stubbed(:user)])
 
       render
 

data/spec/views/campaigns/destroy.js.haml_spec.rb

@@ -8,7 +8,7 @@ require 'spec_helper'
 describe "/campaigns/destroy" do
   before do
     login_and_assign
-    assign(:campaign, @campaign = FactoryGirl.create(:campaign, user: current_user))
+    assign(:campaign, @campaign = FactoryGirl.build_stubbed(:campaign, user: current_user))
     assign(:campaigns, [@campaign].paginate)
     assign(:campaign_status_total, Hash.new(1))
     render

data/spec/views/campaigns/edit.js.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/campaigns/edit" do
 
   before do
     login_and_assign
-    assign(:campaign, @campaign = FactoryGirl.create(:campaign, user: current_user))
+    assign(:campaign, @campaign = FactoryGirl.build_stubbed(:campaign, user: current_user))
     assign(:users, [current_user])
   end
 
@@ -31,7 +31,7 @@ describe "/campaigns/edit" do
 
   it "edit: should hide previously open [Edit Campaign] for and replace it with campaign partial" do
     params[:cancel] = nil
-    assign(:previous, previous = FactoryGirl.create(:campaign, user: current_user))
+    assign(:previous, previous = FactoryGirl.build_stubbed(:campaign, user: current_user))
 
     render
     expect(rendered).to include("$('#campaign_#{previous.id}').replaceWith('<li class=\\'campaign highlight\\' id=\\'campaign_#{previous.id}\\'")

data/spec/views/campaigns/index.haml_spec.rb

@@ -17,7 +17,7 @@ describe "/campaigns/index" do
   end
 
   it "should render list of accounts if list of campaigns is not empty" do
-    assign(:campaigns, [FactoryGirl.create(:campaign)].paginate)
+    assign(:campaigns, [FactoryGirl.build_stubbed(:campaign)].paginate)
 
     render
     expect(view).to render_template(partial: "_campaign")

data/spec/views/campaigns/index.js.haml_spec.rb

@@ -13,7 +13,7 @@ describe "/campaigns/index" do
   end
 
   it "should render [campaign] template with @campaigns collection if there are campaigns" do
-    assign(:campaigns, [FactoryGirl.create(:campaign, id: 42)].paginate)
+    assign(:campaigns, [FactoryGirl.build_stubbed(:campaign, id: 42)].paginate)
 
     render template: 'campaigns/index', formats: [:js]
 

data/spec/views/campaigns/show.haml_spec.rb

@@ -10,13 +10,13 @@ describe "/campaigns/show" do
 
   before do
     login_and_assign
-    @campaign = FactoryGirl.create(:campaign, id: 42,
-                                              leads: [FactoryGirl.create(:lead)],
-                                              opportunities: [FactoryGirl.create(:opportunity)])
+    @campaign = FactoryGirl.build_stubbed(:campaign, id: 42,
+                                              leads: [FactoryGirl.build_stubbed(:lead)],
+                                              opportunities: [FactoryGirl.build_stubbed(:opportunity)])
     assign(:campaign, @campaign)
     assign(:users, [current_user])
     assign(:comment, Comment.new)
-    assign(:timeline, [FactoryGirl.create(:comment, commentable: @campaign)])
+    assign(:timeline, [FactoryGirl.build_stubbed(:comment, commentable: @campaign)])
     allow(view).to receive(:params) { { id: 123 } }
   end
 

data/spec/views/campaigns/update.js.haml_spec.rb

@@ -8,7 +8,7 @@ require 'spec_helper'
 describe "/campaigns/update" do
   before do
     login_and_assign
-    assign(:campaign, @campaign = FactoryGirl.create(:campaign, user: current_user))
+    assign(:campaign, @campaign = FactoryGirl.build_stubbed(:campaign, user: current_user))
     assign(:users, [current_user])
     assign(:status, Setting.campaign_status)
     assign(:campaign_status_total, Hash.new(1))

data/spec/views/contacts/_new.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/contacts/_new" do
 
   before do
     login_and_assign
-    @account = FactoryGirl.create(:account)
+    @account = FactoryGirl.build_stubbed(:account)
     assign(:contact, Contact.new)
     assign(:users, [current_user])
     assign(:account, @account)

data/spec/views/contacts/create.js.haml_spec.rb

@@ -14,7 +14,7 @@ describe "/contacts/create" do
 
   describe "create success" do
     before do
-      assign(:contact, @contact = FactoryGirl.create(:contact))
+      assign(:contact, @contact = FactoryGirl.build_stubbed(:contact))
       assign(:contacts, [@contact].paginate)
     end
 
@@ -50,8 +50,8 @@ describe "/contacts/create" do
   describe "create failure" do
     it "create (failure): should re-render [create] template in :create_contact div" do
       assign(:contact, FactoryGirl.build(:contact, first_name: nil)) # make it invalid
-      @account = FactoryGirl.create(:account)
-      assign(:users, [FactoryGirl.create(:user)])
+      @account = FactoryGirl.build_stubbed(:account)
+      assign(:users, [FactoryGirl.build_stubbed(:user)])
       assign(:account, @account)
       assign(:accounts, [@account])
 

data/spec/views/contacts/destroy.js.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/contacts/destroy" do
 
   before do
     login_and_assign
-    assign(:contact, @contact = FactoryGirl.create(:contact))
+    assign(:contact, @contact = FactoryGirl.build_stubbed(:contact))
     assign(:contacts, [@contact].paginate)
   end
 

data/spec/views/contacts/edit.js.haml_spec.rb

@@ -10,9 +10,9 @@ describe "/contacts/edit" do
 
   before do
     login_and_assign
-    assign(:contact, @contact = FactoryGirl.create(:contact, user: current_user))
+    assign(:contact, @contact = FactoryGirl.build_stubbed(:contact, user: current_user))
     assign(:users, [current_user])
-    assign(:account, @account = FactoryGirl.create(:account))
+    assign(:account, @account = FactoryGirl.build_stubbed(:account))
     assign(:accounts, [@account])
   end
 
@@ -33,7 +33,7 @@ describe "/contacts/edit" do
 
   it "edit: should hide previously open [Edit Contact] for and replace it with contact partial" do
     params[:cancel] = nil
-    assign(:previous, previous = FactoryGirl.create(:contact, user: current_user))
+    assign(:previous, previous = FactoryGirl.build_stubbed(:contact, user: current_user))
 
     render
     expect(rendered).to include("$('#contact_#{previous.id}').replaceWith")

data/spec/views/contacts/index.haml_spec.rb

@@ -17,7 +17,7 @@ describe "/contacts/index" do
   end
 
   it "should render a list of contacts if it's not empty" do
-    assign(:contacts, [FactoryGirl.create(:contact)].paginate)
+    assign(:contacts, [FactoryGirl.build_stubbed(:contact)].paginate)
 
     render
     expect(view).to render_template(partial: "_contact")

data/spec/views/contacts/index.js.html_spec.rb

@@ -13,7 +13,7 @@ describe "/contacts/index" do
   end
 
   it "should render [contact] template with @contacts collection if there are contacts" do
-    assign(:contacts, [FactoryGirl.create(:contact, id: 42)].paginate)
+    assign(:contacts, [FactoryGirl.build_stubbed(:contact, id: 42)].paginate)
 
     render template: 'contacts/index', formats: [:js]
 

data/spec/views/contacts/new.js.haml_spec.rb

@@ -10,7 +10,7 @@ describe "/contacts/new" do
 
   before do
     login_and_assign
-    @account = FactoryGirl.create(:account)
+    @account = FactoryGirl.build_stubbed(:account)
     assign(:contact, Contact.new(user: current_user))
     assign(:users, [current_user])
     assign(:account, @account)

data/spec/views/contacts/update.js.haml_spec.rb

@@ -11,9 +11,9 @@ describe "/contacts/update" do
   before do
     login_and_assign
 
-    assign(:contact, @contact = FactoryGirl.create(:contact, user: current_user))
+    assign(:contact, @contact = FactoryGirl.build_stubbed(:contact, user: current_user))
     assign(:users, [current_user])
-    assign(:account, @account = FactoryGirl.create(:account))
+    assign(:account, @account = FactoryGirl.build_stubbed(:account))
     assign(:accounts, [@account])
   end