fat_free_crm 0.11.3 → 0.11.4

data/spec/acceptance/opportunities_overview_spec.rb

@@ -0,0 +1,82 @@
+require File.expand_path("../acceptance_helper.rb", __FILE__)
+
+feature 'Opportunities Overview', %q{
+  In order to keep track of my team's responsibilities
+  As a user
+  I want to see an overview of opportunities broken down by user
+} do
+
+  background do
+    @me = FactoryGirl.create(:user)
+
+    login_as_user(@me)
+  end
+
+  scenario "Accessing Opportunity Overview via the nav" do
+    visit homepage
+    within ".tabs" do
+      click_link "Team"
+    end
+
+    current_path.should == opportunity_overview_page
+  end
+
+  scenario "Viewing Opportunity Overview when all opportunities have been assigned" do
+    user1 = FactoryGirl.create(:user, :first_name => "Brian", :last_name => 'Doyle-Murray')
+    FactoryGirl.create(:opportunity, :name => "Acting", :stage => 'prospecting', :assignee => user1)
+    FactoryGirl.create(:opportunity, :name => "Directing", :stage => 'won', :assignee => user1)
+
+    user2 = FactoryGirl.create(:user, :first_name => "Dean", :last_name => 'Stockwell')
+    account1 = FactoryGirl.create(:account, :name => 'Quantum Leap')
+    FactoryGirl.create(:opportunity, :name => "Leaping", :stage => 'prospecting', :account => account1, :assignee => user2)
+    FactoryGirl.create(:opportunity, :name => "Return Home", :stage => 'prospecting', :account => account1, :assignee => user2)
+
+    user3 = FactoryGirl.create(:user, :first_name => "Chris", :last_name => 'Jarvis')
+    FactoryGirl.create(:opportunity, :stage => 'won', :assignee => user3)
+    FactoryGirl.create(:opportunity, :stage => 'lost', :assignee => user3)
+
+    visit opportunity_overview_page
+
+    within "#user_#{user1.id}" do
+      page.should have_selector('.title', :text => 'Brian Doyle-Murray')
+      page.should have_content('Acting')
+      page.should_not have_content('Directing')
+    end
+
+    within "#user_#{user2.id}" do
+      page.should have_selector('.title', :text => 'Dean Stockwell')
+      page.should have_content('Leaping from Quantum Leap')
+      page.should have_content('Return Home from Quantum Leap')
+    end
+
+    page.should_not have_selector("#user_#{user3.id}")
+
+    page.should_not have_selector('#unassigned')
+  end
+
+  scenario "Viewing Opportunity Overview when all opportunities are unassigned" do
+    FactoryGirl.create(:opportunity, :name => "Acting", :stage => 'prospecting', :assignee => nil)
+    FactoryGirl.create(:opportunity, :name => "Presenting", :stage => 'won', :assignee => nil)
+
+    visit opportunity_overview_page
+
+    within "#unassigned" do
+      page.should have_selector('.title', :text => 'Unassigned Opportunities')
+      page.should have_content('Acting')
+      page.should_not have_content('Presenting')
+    end
+  end
+
+  scenario "Viewing Opportunity Overview when there are no opportunities in the pipeline" do
+    FactoryGirl.create(:opportunity, :name => "Presenting", :stage => 'lost', :assignee => FactoryGirl.create(:user))
+    FactoryGirl.create(:opportunity, :name => "Eating", :stage => 'won', :assignee => nil)
+
+    visit opportunity_overview_page
+
+    page.should have_content('There are currently no outstanding opportunities.')
+    within "#main" do
+      page.should_not have_content("Presenting")
+      page.should_not have_content("Eating")
+    end
+  end
+end

data/spec/acceptance/opportunities_spec.rb

@@ -0,0 +1,103 @@
+require File.expand_path("../acceptance_helper.rb", __FILE__)
+
+feature 'Opportunities', %q{
+  In order to increase sales
+  As a user
+  I want to manage opportunities
+} do
+
+  before :each do
+    do_login_if_not_already(:first_name => 'Bill', :last_name => 'Murray')
+  end
+
+  scenario 'should view a list of opportunities' do
+    3.times { |i| FactoryGirl.create(:opportunity, :name => "Opportunity #{i}") }
+    visit opportunities_page
+    page.should have_content('Opportunity 0')
+    page.should have_content('Opportunity 1')
+    page.should have_content('Opportunity 2')
+    page.should have_content('Create Opportunity')
+  end
+
+  scenario 'should create a new opportunity', :js => true do
+    FactoryGirl.create(:account, :name => 'Example Account')
+    visit opportunities_page
+    click_link 'Create Opportunity'
+    page.should have_selector('#opportunity_name', :visible => true)
+    fill_in 'opportunity_name', :with => 'My Awesome Opportunity'
+    chosen_select('Example Account', :from => 'account_id')
+    select 'prospecting', :from => 'opportunity_stage'
+    click_link 'Comment'
+    fill_in 'comment_body', :with => 'This is a very important opportunity.'
+    click_button 'Create Opportunity'
+    page.should have_content('My Awesome Opportunity')
+
+    click_link 'My Awesome Opportunity'
+    page.should have_content('This is a very important opportunity.')
+
+    click_link "Dashboard"
+    page.should have_content("Bill Murray created opportunity My Awesome Opportunity")
+    page.should have_content("Bill Murray created comment on My Awesome Opportunity")
+  end
+
+  scenario "remembers the comment field when the creation was unsuccessful", :js => true do
+    visit opportunities_page
+    click_link 'Create Opportunity'
+    select 'prospecting', :from => 'opportunity_stage'
+
+    click_link 'Comment'
+    fill_in 'comment_body', :with => 'This is a very important opportunity.'
+    click_button 'Create Opportunity'
+
+    page.should have_field('comment_body', :with => 'This is a very important opportunity.')
+  end
+
+  scenario 'should view and edit an opportunity', :js => true do
+    FactoryGirl.create(:account, :name => 'Example Account')
+    FactoryGirl.create(:account, :name => 'Other Example Account')
+    FactoryGirl.create(:opportunity, :name => 'A Cool Opportunity')
+    visit opportunities_page
+    click_link 'A Cool Opportunity'
+    click_link 'Edit'
+    fill_in 'opportunity_name', :with => 'An Even Cooler Opportunity'
+    chosen_select('Other Example Account', :from => 'account_id')
+    select 'analysis', :from => 'opportunity_stage'
+    click_button 'Save Opportunity'
+    page.should have_content('An Even Cooler Opportunity')
+    click_link 'Opportunities'
+    page.should have_content('An Even Cooler Opportunity')
+
+    click_link "Dashboard"
+    page.should have_content("Bill Murray updated opportunity An Even Cooler Opportunity")
+  end
+
+  scenario 'should delete an opportunity', :js => true do
+    FactoryGirl.create(:opportunity, :name => 'Outdated Opportunity')
+    visit opportunities_page
+    click_link 'Outdated Opportunity'
+    click_link 'Delete?'
+    page.should have_content('Are you sure you want to delete this opportunity?')
+    click_link 'Yes'
+    page.should have_content('Outdated Opportunity has been deleted.')
+    click_link('Opportunities')
+    page.should_not have_content('Outdated Opportunity')
+  end
+
+  scenario 'should search for an opportunity', :js => true do
+    3.times { |i| FactoryGirl.create(:opportunity, :name => "Opportunity #{i}") }
+    visit opportunities_page
+    find('#opportunities').should have_content("Opportunity 0")
+    find('#opportunities').should have_content("Opportunity 1")
+    find('#opportunities').should have_content("Opportunity 2")
+    fill_in 'query', :with => "Opportunity 0"
+    find('#opportunities').should have_content("Opportunity 0")
+    find('#opportunities').has_selector?('li', :count => 1)
+    fill_in 'query', :with => "Opportunity"
+    find('#opportunities').should have_content("Opportunity 0")
+    find('#opportunities').should have_content("Opportunity 1")
+    find('#opportunities').should have_content("Opportunity 2")
+    find('#opportunities').has_selector?('li', :count => 3)
+    fill_in 'query', :with => "Non-existant opportunity"
+    find('#opportunities').has_selector?('li', :count => 0)
+  end
+end

data/{acceptance → spec/acceptance}/support/helpers.rb

@@ -9,6 +9,13 @@ module HelperMethods
     click_button "Login"
   end
 
+  def login_as_user(user)
+    visit '/login'
+    fill_in "authentication_username", :with => user.username
+    fill_in "authentication_password", :with => user.password
+    click_button "Login"
+  end
+
   # if we're already logged in, don't bother doing it again
   def do_login_if_not_already(options = {})
     do_login(options) unless @user.present?
@@ -16,4 +23,4 @@ module HelperMethods
 
 end
 
-RSpec.configuration.include HelperMethods, :type => :acceptance
+RSpec.configuration.include HelperMethods, :type => :request

data/spec/acceptance/support/paths.rb

@@ -0,0 +1,42 @@
+module NavigationHelpers
+  # Put helper methods related to the paths in your application here.
+
+  def homepage
+    "/"
+  end
+
+  def accounts_page
+    accounts_path
+  end
+
+  def leads_page
+    leads_path
+  end
+
+  def opportunities_page
+    opportunities_path
+  end
+
+  def contacts_page
+    contacts_path
+  end
+
+  def campaigns_page
+    campaigns_path
+  end
+
+  def tasks_page
+    tasks_path
+  end
+  
+  def groups_page
+    admin_groups_path
+  end
+  
+  def opportunity_overview_page
+    opportunities_overview_users_path
+  end
+  
+end
+
+RSpec.configuration.include NavigationHelpers, :type => :request

data/spec/acceptance/support/selector_helpers.rb

@@ -0,0 +1,31 @@
+module SelectorHelpers
+  def chosen_select(item_text, options)
+    field_id = find_field(options[:from])[:id]
+    option_value = page.evaluate_script("jQuery(\"##{field_id} option:contains('#{item_text}')\").val()")#page.evaluate_script("$(\"##{field_id} option:contains('#{item_text}')\").val()")
+    page.execute_script("jQuery('##{field_id}').val('#{option_value}')")
+  end
+
+  def check_filter(filter_name)
+    filter_checkbox = find(:xpath, "//input[@type='checkbox'][@value='due_#{filter_name}']")
+    filter_checkbox.click
+  end
+
+  def click_filter_tab(filter_name)
+    tab = find(:xpath, "//div[@class='filters']//td[contains(text(), '#{filter_name}')]")
+    tab.click
+  end
+
+  def click_edit_for_task_id(task_id)
+    within("#task_#{task_id}") do
+      page.execute_script "jQuery('#task_#{task_id} a')[0].click()"
+    end
+  end
+
+  def click_delete_for_task_id(task_id)
+    within("#task_#{task_id}") do
+      page.execute_script "jQuery('#task_#{task_id} a')[1].click()"
+    end
+  end
+end
+
+RSpec.configuration.include SelectorHelpers, :type => :request

data/spec/acceptance/tasks_spec.rb

@@ -0,0 +1,85 @@
+require File.expand_path("../acceptance_helper.rb", __FILE__)
+
+feature 'Tasks', %q{
+  In order to increase keep track of things
+  As a user
+  I want to manage tasks
+} do
+
+  before :each do
+    do_login_if_not_already(:first_name => 'Bill', :last_name => 'Murray')
+  end
+
+  scenario 'should view a list of tasks which are assigned to the logged in user' do
+    4.times { |i| FactoryGirl.create(:task, :name => "Task #{i}", :user => @user) }
+    visit tasks_page
+    page.should have_content('Task 0')
+    page.should have_content('Task 1')
+    page.should have_content('Task 2')
+    page.should have_content('Task 3')
+    page.should have_content('Create Task')
+  end
+
+  scenario 'should create a new task', :js => true do
+    visit tasks_page
+    page.should have_content('Create Task')
+    click_link 'Create Task'
+    page.should have_selector('#task_name', :visible => true)
+    fill_in 'task_name', :with => 'Task I Need To Do'
+    chosen_select('Tomorrow', :from => 'task_bucket')
+    chosen_select('Myself', :from => 'task_assigned_to')
+    chosen_select('Call', :from => 'task_category')
+    click_button 'Create Task'
+    page.should have_content('Task I Need To Do')
+
+    click_link 'Dashboard'
+    page.should have_content('Bill Murray created task Task I Need To Do')
+  end
+
+  scenario 'creating a task for another user', :js => true do
+    FactoryGirl.create(:user, :first_name => 'Another', :last_name => 'User')
+    visit tasks_page
+    click_link 'Create Task'
+    page.should have_selector('#task_name', :visible => true)
+    fill_in 'task_name', :with => 'Task For Someone Else'
+    chosen_select('Tomorrow', :from => 'task_bucket')
+    chosen_select('Another User', :from => 'task_assigned_to')
+    chosen_select('Call', :from => 'task_category')
+    click_button 'Create Task'
+    page.should have_content('The task has been created and assigned to Another User')
+
+    click_link 'Tasks'
+    check_filter 'tomorrow'
+    page.should_not have_content('Task For Someone Else')
+
+    click_filter_tab('Assigned')
+    check_filter 'tomorrow'
+    page.should have_content('Task For Someone Else')
+    page.should have_content('Another User')
+
+    click_link 'Dashboard'
+    page.should have_content('Bill Murray created task Task For Someone Else')
+  end
+
+  scenario 'should view and edit a task', :js => true do
+    FactoryGirl.create(:task, :id => 42, :name => 'Example Task', :user => @user)
+    visit tasks_page
+    click_edit_for_task_id(42)
+    fill_in 'task_name', :with => 'Updated Task'
+    click_button 'Save Task'
+    page.should have_content('Updated Task')
+    click_link 'Task'
+    page.should have_content('Updated Task')
+
+    click_link 'Dashboard'
+    page.should have_content('Bill Murray updated task Updated Task')
+  end
+
+  scenario 'should delete a task', :js => true do
+    FactoryGirl.create(:task, :id => 42, :name => 'Outdated Task', :user => @user)
+    visit tasks_page
+    click_delete_for_task_id(42)
+    click_link 'Tasks'
+    page.should_not have_content('Outdated Task')
+  end
+end

data/spec/controllers/admin/groups_controller_spec.rb

@@ -0,0 +1,5 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe Admin::GroupsController do
+
+end

data/spec/controllers/admin/users_controller_spec.rb

@@ -12,7 +12,7 @@ describe Admin::UsersController do
   #----------------------------------------------------------------------------
   describe "GET index" do
     it "assigns all users as @users and renders [index] template" do
-      @users = [ @current_user, FactoryGirl.create(:user) ]
+      @users = [ current_user, FactoryGirl.create(:user) ]
 
       get :index
       assigns[:users].first.should == @users.last # get_users() sorts by id DESC
@@ -264,7 +264,7 @@ describe Admin::UsersController do
     end
 
     it "doesn't suspend current user" do
-      @user = @current_user
+      @user = current_user
 
       xhr :put, :suspend, :id => @user.id
       assigns[:user].suspended?.should == false
@@ -304,4 +304,3 @@ describe Admin::UsersController do
   end
 
 end
-

data/spec/controllers/applications_controller_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe ApplicationController do
+
+  describe "auto_complete_ids_to_exclude" do
+  
+    it "should return [] when related is nil" do
+      controller.send(:auto_complete_ids_to_exclude, nil).should == []
+    end
+    
+    it "should return [] when related is ''" do
+      controller.send(:auto_complete_ids_to_exclude, '').should == []
+    end
+    
+    it "should return campaign id 5 when related is '5' and controller is campaigns" do
+      controller.send(:auto_complete_ids_to_exclude, '5').sort.should == [5]
+    end
+    
+    it "should return [6, 9] when related is 'campaigns/7'" do
+      controller.stub!(:controller_name).and_return('opportunities')
+      campaign = mock(Campaign, :opportunities => [mock(:id => 6), mock(:id => 9)])
+      Campaign.should_receive(:find_by_id).with('7').and_return(campaign)
+      controller.send(:auto_complete_ids_to_exclude, 'campaigns/7').sort.should == [6, 9]
+    end
+    
+    it "should return [] when related object is not found" do
+      Campaign.should_receive(:find_by_id).with('7').and_return(nil)
+      controller.send(:auto_complete_ids_to_exclude, 'campaigns/7').should == []
+    end
+    
+    it "should return [] when related object association is not found" do
+      controller.stub!(:controller_name).and_return('not_a_method_that_exists')
+      campaign = mock(Campaign)
+      Campaign.should_receive(:find_by_id).with('7').and_return(campaign)
+      controller.send(:auto_complete_ids_to_exclude, 'campaigns/7').should == []
+    end
+    
+  end
+
+end