fat_free_crm 0.18.1 → 0.20.0

data/spec/models/users/user_spec.rb

@@ -11,7 +11,7 @@
 #
 #  id                  :integer         not null, primary key
 #  username            :string(32)      default(""), not null
-#  email               :string(64)      default(""), not null
+#  email               :string(254)     default(""), not null
 #  first_name          :string(32)
 #  last_name           :string(32)
 #  title               :string(64)
@@ -23,21 +23,27 @@
 #  yahoo               :string(32)
 #  google              :string(32)
 #  skype               :string(32)
-#  password_hash       :string(255)     default(""), not null
+#  encrypted_password  :string(255)     default(""), not null
 #  password_salt       :string(255)     default(""), not null
-#  persistence_token   :string(255)     default(""), not null
-#  perishable_token    :string(255)     default(""), not null
-#  last_login_at       :datetime
-#  current_login_at    :datetime
-#  last_login_ip       :string(255)
-#  current_login_ip    :string(255)
-#  login_count         :integer         default(0), not null
+#  last_sign_in_at     :datetime
+#  current_sign_in_at  :datetime
+#  last_sign_in_ip     :string(255)
+#  current_sign_in_ip  :string(255)
+#  sign_in_count       :integer         default(0), not null
 #  deleted_at          :datetime
 #  created_at          :datetime
 #  updated_at          :datetime
 #  admin               :boolean         default(FALSE), not null
 #  suspended_at        :datetime
-#  single_access_token :string(255)
+#  unconfirmed_email   :string(254)     default(""), not null
+#  reset_password_token    :string(255)
+#  reset_password_sent_at  :datetime
+#  remember_token          :string(255)
+#  remember_created_at     :datetime
+#  authentication_token    :string(255)
+#  confirmation_token      :string(255)
+#  confirmed_at            :datetime
+#  confirmation_sent_at    :datetime
 #
 
 require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
@@ -75,6 +81,12 @@ describe User do
         end
       end
 
+      it "should not destroy the user if she owns a comment" do
+        account = build(:account, user: @current_user)
+        FactoryBot.create(:comment, user: @user, commentable: account)
+        expect(@user.destroyable?(@current_user)).to eq(false)
+      end
+
       it "should not destroy the current user" do
         expect(@user.destroyable?(@user)).to eq(false)
       end
@@ -202,22 +214,6 @@ describe User do
     end
   end
 
-  describe "Setting single access token" do
-    it "should update single_access_token attribute if it is not set already" do
-      @user = build(:user, single_access_token: nil)
-
-      @user.set_single_access_token
-      expect(@user.single_access_token).not_to eq(nil)
-    end
-
-    it "should not update single_access_token attribute if it is set already" do
-      @user = build(:user, single_access_token: "token")
-
-      @user.set_single_access_token
-      expect(@user.single_access_token).to eq("token")
-    end
-  end
-
   describe "serialization" do
     let(:user) { build(:user) }
 

data/spec/routing/users_routing_spec.rb

@@ -9,12 +9,38 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 describe UsersController do
   describe "routing" do
-    it "doesn't recognize #index" do
-      expect(get: "/users").not_to be_routable
+    it "recognizes #index" do
+      expect(get: "/users").to route_to(controller: "users", action: "index")
     end
 
-    it "recognizes and generates #new as /signup" do
-      expect(get: "/signup").to route_to(controller: "users", action: "new")
+    it "recognizes and generates Devise registrations routes" do
+      expect(get: "/users/sign_up").to route_to(controller: "registrations", action: "new")
+      expect(get: "/users/edit").to route_to(controller: "registrations", action: "edit")
+      expect(get: "/users/cancel").to route_to(controller: "registrations", action: "cancel")
+      expect(post: "/users").to route_to(controller: "registrations", action: "create")
+      expect(put: "/users").to route_to(controller: "registrations", action: "update")
+      expect(patch: "/users").to route_to(controller: "registrations", action: "update")
+      expect(delete: "/users").to route_to(controller: "registrations", action: "destroy")
+    end
+
+    it "recognizes and generates Devise sessions routes" do
+      expect(get: "/users/sign_in").to route_to(controller: "sessions", action: "new")
+      expect(post: "/users/sign_in").to route_to(controller: "sessions", action: "create")
+      expect(delete: "/users/sign_out").to route_to(controller: "sessions", action: "destroy")
+    end
+
+    it "recognizes and generates Devise passwords routes" do
+      expect(get: "/users/password/new").to route_to(controller: "passwords", action: "new")
+      expect(get: "/users/password/edit").to route_to(controller: "passwords", action: "edit")
+      expect(post: "/users/password").to route_to(controller: "passwords", action: "create")
+      expect(put: "/users/password").to route_to(controller: "passwords", action: "update")
+      expect(patch: "/users/password").to route_to(controller: "passwords", action: "update")
+    end
+
+    it "recognizes and generates Devise confirmations routes" do
+      expect(get: "/users/confirmation/new").to route_to(controller: "confirmations", action: "new")
+      expect(get: "/users/confirmation").to route_to(controller: "confirmations", action: "show")
+      expect(post: "/users/confirmation").to route_to(controller: "confirmations", action: "create")
     end
 
     it "recognizes and generates #show as /profile" do
@@ -29,10 +55,6 @@ describe UsersController do
       expect(get: "/users/aaron/edit").not_to be_routable
     end
 
-    it "recognizes and generates #create" do
-      expect(post: "/users").to route_to(controller: "users", action: "create")
-    end
-
     it "recognizes and generates #update" do
       expect(put: "/users/1").to route_to(controller: "users", action: "update", id: "1")
     end

data/spec/shared/controllers.rb

@@ -22,9 +22,7 @@ shared_examples "auto complete" do
 
     # We don't save Admin/Users autocomplete controller in a session since Users are not
     # exposed through the Jumpbox.
-    unless controller.class.to_s.starts_with?("Admin::")
-      expect(session[:auto_complete]).to eq(@controller.controller_name.to_sym)
-    end
+    expect(session[:auto_complete]).to eq(@controller.controller_name.to_sym) unless controller.class.to_s.starts_with?("Admin::")
   end
 
   it "should render application/_auto_complete template" do
@@ -39,12 +37,8 @@ shared_examples "attach" do
     expect(@model.send(@attachment.class.name.tableize)).to include(@attachment)
     expect(assigns[:attachment]).to eq(@attachment)
     expect(assigns[:attached]).to eq([@attachment])
-    if @model.is_a?(Campaign) && (@attachment.is_a?(Lead) || @attachment.is_a?(Opportunity))
-      expect(assigns[:campaign]).to eq(@attachment.reload.campaign)
-    end
-    if @model.is_a?(Account) && @attachment.respond_to?(:account) # Skip Tasks...
-      expect(assigns[:account]).to eq(@attachment.reload.account)
-    end
+    expect(assigns[:campaign]).to eq(@attachment.reload.campaign) if @model.is_a?(Campaign) && (@attachment.is_a?(Lead) || @attachment.is_a?(Opportunity))
+    expect(assigns[:account]).to eq(@attachment.reload.account) if @model.is_a?(Account) && @attachment.respond_to?(:account) # Skip Tasks...
     expect(response).to render_template("entities/attach")
   end
 

data/spec/spec_helper.rb

@@ -8,15 +8,15 @@
 require 'rubygems'
 
 ENV["RAILS_ENV"] = 'test'
-require File.expand_path("../../config/environment", __FILE__)
+require File.expand_path('../config/environment', __dir__)
 require 'rspec/rails'
 require 'capybara/rails'
 require 'paper_trail/frameworks/rspec'
 
-require 'acts_as_fu'
 require 'factory_bot_rails'
 require 'ffaker'
 require 'timecop'
+require 'webdrivers'
 
 # Requires supporting ruby files with custom matchers and macros, etc,
 # in spec/support/ and its subdirectories.
@@ -41,6 +41,14 @@ RSpec.configure do |config|
   # RSpec configuration options for Fat Free CRM.
   config.include RSpec::Rails::Matchers
   config.include FactoryBot::Syntax::Methods
+  config.include Devise::Test::ControllerHelpers, type: :controller
+  config.include Devise::Test::ControllerHelpers, type: :view
+  config.include Devise::Test::IntegrationHelpers, type: :features
+  config.include Warden::Test::Helpers
+  config.include DeviseHelpers
+  config.include FeatureHelpers
+
+  Warden.test_mode!
 
   config.before(:each) do
     # Overwrite locale settings within "config/settings.yml" if necessary.

data/spec/support/assert_select.rb

@@ -155,6 +155,7 @@ module RSpec # :nodoc:
 
       def prepare_args(args, current_scope = nil)
         return args if current_scope.nil?
+
         defaults = current_scope.options || { strict: false, xml: false }
         args << {} unless args.last.is_a?(::Hash)
         args.last[:strict] = defaults[:strict] if args.last[:strict].nil?

data/spec/support/devise_helpers.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+# Copyright (c) 2008-2013 Michael Dvorkin and contributors.
+#
+# Fat Free CRM is freely distributable under the terms of MIT license.
+# See MIT-LICENSE file or http://www.opensource.org/licenses/mit-license.php
+
+module DeviseHelpers
+  def login
+    user = create :user
+    perform_login(user)
+  end
+
+  def login_admin
+    admin = FactoryBot.create(:user, admin: true)
+    perform_login(admin)
+  end
+
+  def perform_login(user)
+    user.confirm
+    user.update_attribute(:suspended_at, nil)
+    sign_in user
+  end
+
+  def current_user
+    User.find_by_id(session['warden.user.user.key'][0][0])
+  end
+end

data/spec/{features/support/helpers.rb → support/feature_helpers.rb}

@@ -5,21 +5,23 @@
 # Fat Free CRM is freely distributable under the terms of MIT license.
 # See MIT-LICENSE file or http://www.opensource.org/licenses/mit-license.php
 #------------------------------------------------------------------------------
-module HelperMethods
+
+module FeatureHelpers
   # Put helper methods you need to be available in all acceptance specs here.
 
   def do_login(options = {})
     @user = create(:user, options)
-    visit '/login'
-    fill_in "authentication_username", with: @user.username
-    fill_in "authentication_password", with: @user.password
-    click_button "Login"
+    @user.confirm
+    @user.update_attribute(:suspended_at, nil)
+    login_as(@user)
   end
 
   def login_as_user(user)
-    visit '/login'
-    fill_in "authentication_username", with: user.username
-    fill_in "authentication_password", with: user.password
+    user.confirm
+    user.update_attribute(:suspended_at, nil)
+    visit '/users/sign_in'
+    fill_in "user[email]", with: user.username
+    fill_in "user[password]", with: user.password
     click_button "Login"
   end
 
@@ -28,5 +30,3 @@ module HelperMethods
     do_login(options) unless @user.present?
   end
 end
-
-RSpec.configuration.include HelperMethods, type: :feature

data/spec/support/macros.rb

@@ -24,7 +24,10 @@ end
 #----------------------------------------------------------------------------
 def stub_task_total(view = "pending")
   settings = (view == "completed" ? Setting.task_completed : Setting.task_bucket)
-  settings.each_with_object(all: 0) { |key, hash| hash[key] = 1; hash }
+  settings.each_with_object(all: 0) do |key, hash|
+    hash[key] = 1
+    hash
+  end
 end
 
 # Get current server timezone and set it (see rake time:zones:local for details).

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

@@ -22,7 +22,7 @@ describe "/accounts/_edit" do
     expect(view).to render_template(partial: "_contact_info")
     expect(view).to render_template(partial: "_permissions")
 
-    expect(rendered).to have_tag("form[class=edit_account]") do |form|
+    expect(rendered).to have_tag('form[class="simple_form edit_account"]') do |form|
       expect(form).to have_tag "input[type=hidden][id=account_user_id][value='#{@account.user_id}']"
     end
   end

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

@@ -23,7 +23,7 @@ describe "/accounts/_new" do
     expect(view).to render_template(partial: "_contact_info")
     expect(view).to render_template(partial: "_permissions")
 
-    expect(rendered).to have_tag("form[class=new_account]")
+    expect(rendered).to have_tag('form[class="simple_form new_account"]')
   end
 
   it "should render background info field if settings require so" do

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

@@ -25,7 +25,7 @@ describe "/accounts/create" do
     end
 
     it "should hide [Create Account] form and insert account partial" do
-      expect(rendered).to include("$('#accounts').prepend('<li class=\\'account highlight\\' id=\\'account_#{@account.id}\\'")
+      expect(rendered).to include("$('#accounts').prepend('<li class=\\'highlight account\\' id=\\'account_#{@account.id}\\'")
       expect(rendered).to include(%/$('#account_#{@account.id}').effect("highlight"/)
     end
 
@@ -47,7 +47,6 @@ describe "/accounts/create" do
       render
 
       expect(rendered).to include("#create_account")
-      expect(rendered).to include(%/$('#create_account').effect("shake"/)
     end
   end
 end

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

@@ -34,7 +34,6 @@ describe "/accounts/update" do
         render
         expect(rendered).to include("$('#sidebar').html")
         expect(rendered).to have_text("Recent Items")
-        expect(rendered).to include("$('#summary').effect('shake'")
       end
     end
 
@@ -70,11 +69,10 @@ describe "/accounts/update" do
         controller.request.env["HTTP_REFERER"] = "http://localhost/accounts/123"
       end
 
-      it "should redraw the [edit_account] form and shake it" do
+      it "should redraw the [edit_account] form" do
         render
 
         expect(rendered).to include("#edit_account")
-        expect(rendered).to include(%/$('#edit_account').effect("shake"/)
         expect(rendered).to include('focus()')
       end
     end
@@ -84,11 +82,10 @@ describe "/accounts/update" do
         controller.request.env["HTTP_REFERER"] = "http://localhost/accounts"
       end
 
-      it "should redraw the [edit_account] form and shake it" do
+      it "should redraw the [edit_account] form" do
         render
 
         expect(rendered).to include("account_#{@account.id}")
-        expect(rendered).to include(%/$('#account_#{@account.id}').effect("shake"/)
         expect(rendered).to include('focus()')
       end
     end

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

@@ -26,6 +26,5 @@ describe "admin/field_groups/create" do
     allow(field_group).to receive(:valid?).and_return(false)
     render
     expect(view).to render_template("admin/field_groups/create")
-    expect(rendered).to have_text("effect(\"shake\", { duration:250, distance: 6 });")
   end
 end

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

@@ -25,6 +25,5 @@ describe "admin/field_groups/update" do
   it "renders javascript for invalid field group" do
     allow(field_group.errors).to receive(:empty?).and_return(false)
     render
-    expect(rendered).to have_text("$('##{dom_id(field_group, :edit)}').effect('shake', { distance:5 }, 250);")
   end
 end

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

@@ -36,7 +36,6 @@ describe "admin/users/create" do
       render
 
       expect(rendered).to include('Please specify username')
-      expect(rendered).to include(%/$('#create_user').effect("shake"/)
     end
   end
 end

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

@@ -37,12 +37,6 @@ describe "admin/users/destroy" do
       expect(rendered).to include(%/crm.flick('#{dom_id(@user, :confirm)}', 'remove');/)
     end
 
-    it "should shake user partial" do
-      render
-
-      expect(rendered).to include(%/$('#user_#{@user.id}').effect('shake'/)
-    end
-
     it "should show flash message" do
       render
 

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

@@ -27,11 +27,10 @@ describe "admin/users/update" do
       @user.errors.add(:name)
     end
 
-    it "redraws [Edit User] form and shakes it" do
+    it "redraws [Edit User] form" do
       render
 
       expect(rendered).to include("user_#{@user.id}")
-      expect(rendered).to include(%/$('#user_#{@user.id}').effect("shake"/)
       expect(rendered).to include(%/$('#user_username').focus()/)
     end
   end

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

@@ -23,7 +23,7 @@ describe "/campaigns/_edit" do
     expect(view).to render_template(partial: "campaigns/_objectives")
     expect(view).to render_template(partial: "_permissions")
 
-    expect(view).to have_tag("form[class=edit_campaign]") do
+    expect(view).to have_tag('form[class="simple_form edit_campaign"]') do
       with_tag "input[type=hidden][id=campaign_user_id][value='#{@campaign.user_id}']"
     end
   end

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

@@ -22,7 +22,7 @@ describe "/campaigns/_new" do
     expect(view).to render_template(partial: "campaigns/_objectives")
     expect(view).to render_template(partial: "_permissions")
 
-    expect(rendered).to have_tag("form[class=new_campaign]")
+    expect(rendered).to have_tag('form[class="simple_form new_campaign"]')
   end
 
   it "should render background info field if settings require so" do

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

@@ -21,7 +21,7 @@ describe "/campaigns/create" do
     end
 
     it "should hide [Create Campaign] form and insert campaign partial" do
-      expect(rendered).to include("$('#campaigns').prepend('<li class=\\'campaign highlight\\' id=\\'campaign_#{@campaign.id}\\'")
+      expect(rendered).to include("$('#campaigns').prepend('<li class=\\'highlight campaign\\' id=\\'campaign_#{@campaign.id}\\'")
       expect(rendered).to include(%/$('#campaign_#{@campaign.id}').effect("highlight"/)
     end
 
@@ -44,7 +44,6 @@ describe "/campaigns/create" do
       render
 
       expect(rendered).to include("$('#create_campaign').html")
-      expect(rendered).to include(%/$('#create_campaign').effect("shake"/)
     end
   end
 end

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

@@ -23,7 +23,6 @@ describe "/campaigns/destroy" do
   it "should update Campaigns sidebar" do
     expect(rendered).to include("#sidebar")
     expect(rendered).to have_text("Recent Items")
-    expect(rendered).to include(%/$('#filters').effect('shake'/)
   end
 
   it "should update pagination" do

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

@@ -20,7 +20,7 @@ describe "/campaigns/edit" do
     params[:cancel] = "true"
 
     render
-    expect(rendered).to include("$('#campaign_#{@campaign.id}').replaceWith('<li class=\\'campaign highlight\\' id=\\'campaign_#{@campaign.id}\\'")
+    expect(rendered).to include("$('#campaign_#{@campaign.id}').replaceWith('<li class=\\'highlight campaign\\' id=\\'campaign_#{@campaign.id}\\'")
   end
 
   it "cancel from campaign landing page: should hide [Edit Campaign] form" do
@@ -36,7 +36,7 @@ describe "/campaigns/edit" do
     assign(:previous, previous = build_stubbed(:campaign, user: current_user))
 
     render
-    expect(rendered).to include("$('#campaign_#{previous.id}').replaceWith('<li class=\\'campaign highlight\\' id=\\'campaign_#{previous.id}\\'")
+    expect(rendered).to include("$('#campaign_#{previous.id}').replaceWith('<li class=\\'highlight campaign\\' id=\\'campaign_#{previous.id}\\'")
   end
 
   it "edit: should remove previously open [Edit Campaign] if it's no longer available" do

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

@@ -19,7 +19,7 @@ describe "/campaigns/index" do
 
     render template: 'campaigns/index', formats: [:js]
 
-    expect(rendered).to include("$('#campaigns').html('<li class=\\'campaign highlight\\' id=\\'campaign_42\\'")
+    expect(rendered).to include("$('#campaigns').html('<li class=\\'highlight campaign\\' id=\\'campaign_42\\'")
     expect(rendered).to include("#paginate")
   end
 

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

@@ -31,9 +31,7 @@ describe "/campaigns/update" do
       it "should update sidebar" do
         render
         expect(rendered).to include("#sidebar")
-        expect(rendered).to have_text("Campaign Summary")
         expect(rendered).to have_text("Recent Items")
-        expect(rendered).to include("$('#summary').effect('shake'")
       end
     end
 
@@ -44,7 +42,7 @@ describe "/campaigns/update" do
 
       it "should replace [Edit Campaign] with campaign partial and highlight it" do
         render
-        expect(rendered).to include("$('#campaign_#{@campaign.id}').replaceWith('<li class=\\'campaign highlight\\' id=\\'campaign_#{@campaign.id}\\'")
+        expect(rendered).to include("$('#campaign_#{@campaign.id}').replaceWith('<li class=\\'highlight campaign\\' id=\\'campaign_#{@campaign.id}\\'")
         expect(rendered).to include(%/$('#campaign_#{@campaign.id}').effect("highlight"/)
       end
     end
@@ -57,10 +55,9 @@ describe "/campaigns/update" do
         controller.request.env["HTTP_REFERER"] = "http://localhost/campaigns/123"
       end
 
-      it "should redraw the [edit_campaign] form and shake it" do
+      it "should redraw the [edit_campaign] form" do
         render
         expect(rendered).to include("$('#edit_campaign').html")
-        expect(rendered).to include(%/$('#edit_campaign').effect("shake"/)
         expect(rendered).to include('focus()')
       end
     end
@@ -71,10 +68,9 @@ describe "/campaigns/update" do
         controller.request.env["HTTP_REFERER"] = "http://localhost/campaigns"
       end
 
-      it "should redraw the [edit_campaign] form and shake it" do
+      it "should redraw the [edit_campaign] form" do
         render
         expect(rendered).to include("$('#campaign_#{@campaign.id}').html")
-        expect(rendered).to include(%/$('#campaign_#{@campaign.id}').effect("shake"/)
         expect(rendered).to include('focus()')
       end
     end

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

@@ -26,7 +26,7 @@ describe "/contacts/_edit" do
     expect(view).to render_template(partial: "contacts/_web")
     expect(view).to render_template(partial: "_permissions")
 
-    expect(rendered).to have_tag("form[class=edit_contact]") do
+    expect(rendered).to have_tag('form[class="simple_form edit_contact"]') do
       with_tag "input[type=hidden][id=contact_user_id][value='#{@contact.user_id}']"
     end
   end

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

@@ -26,7 +26,7 @@ describe "/contacts/_new" do
     expect(view).to render_template(partial: "contacts/_web")
     expect(view).to render_template(partial: "entities/_permissions")
 
-    expect(rendered).to have_tag("form[class=new_contact]")
+    expect(rendered).to have_tag('form[class="simple_form new_contact"]')
   end
 
   it "should pick default assignee (Myself)" do

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

@@ -23,7 +23,7 @@ describe "/contacts/create" do
     it "should hide [Create Contact] form and insert contact partial" do
       render
 
-      expect(rendered).to include("$('#contacts').prepend('<li class=\\'contact highlight\\' id=\\'contact_#{@contact.id}\\'")
+      expect(rendered).to include("$('#contacts').prepend('<li class=\\'highlight contact\\' id=\\'contact_#{@contact.id}\\'")
       expect(rendered).to include(%/$('#contact_#{@contact.id}').effect("highlight"/)
     end
 
@@ -60,7 +60,6 @@ describe "/contacts/create" do
       render
 
       expect(rendered).to include("$('#create_contact').html")
-      expect(rendered).to include(%/$('#create_contact').effect("shake"/)
     end
   end
 end

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

@@ -22,7 +22,7 @@ describe "/contacts/edit" do
     params[:cancel] = "true"
 
     render
-    expect(rendered).to include("$('#contact_#{@contact.id}').replaceWith('<li class=\\'contact highlight\\' id=\\'contact_#{@contact.id}\\'")
+    expect(rendered).to include("$('#contact_#{@contact.id}').replaceWith('<li class=\\'highlight contact\\' id=\\'contact_#{@contact.id}\\'")
   end
 
   it "cancel from contact landing page: should hide [Edit Contact] form" do

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

@@ -19,7 +19,7 @@ describe "/contacts/index" do
 
     render template: 'contacts/index', formats: [:js]
 
-    expect(rendered).to include("$('#contacts').html('<li class=\\'contact highlight\\' id=\\'contact_42\\'")
+    expect(rendered).to include("$('#contacts').html('<li class=\\'highlight contact\\' id=\\'contact_42\\'")
     expect(rendered).to include("#paginate")
   end