fat_free_crm 0.15.0.beta.2 → 0.15.0

data/fat_free_crm.gemspec

@@ -20,10 +20,9 @@ Gem::Specification.new do |gem|
   gem.required_ruby_version = '>= 2.0.0'
   gem.license = 'MIT'
 
-  gem.add_dependency 'rails',               '~> 4.2'
+  gem.add_dependency 'rails',               '~> 5.0.0'
   gem.add_dependency 'rails-observers'
-  # Sprockets 2.2.3 appears to have a bug resolving assets from the engine update to '>= 3.0.0 once released'
-  gem.add_dependency 'sprockets-rails',     '2.2.2'
+  gem.add_dependency 'sprockets-rails',     '>= 3.0.0'
   gem.add_dependency 'responders',          '~> 2.0'
   gem.add_dependency 'jquery-rails'
   gem.add_dependency 'jquery-migrate-rails'
@@ -35,7 +34,7 @@ Gem::Specification.new do |gem|
   # Manually added paperclip gem dependency "cocaine" in order to fix load error: "no such file to load -- cocaine"
   gem.add_dependency 'cocaine'
   gem.add_dependency 'paper_trail',         '~> 6.0.0'
-  gem.add_dependency 'authlogic',           '>= 3.4.4', '< 3.5.0' # View tests fail on authlogic 3.5.0+
+  gem.add_dependency 'authlogic',           '>= 3.4.4'
   gem.add_dependency 'acts_as_commentable'
   gem.add_dependency 'acts-as-taggable-on', '>= 3.4.3'
   gem.add_dependency 'dynamic_form'
@@ -49,7 +48,6 @@ Gem::Specification.new do |gem|
   gem.add_dependency 'nokogiri'
   gem.add_dependency 'responds_to_parent'
   gem.add_dependency 'rails3-jquery-autocomplete'
-  gem.add_dependency 'psych', '~> 1'        if RUBY_VERSION.to_f < 2.0
   gem.add_dependency 'thor'
   gem.add_dependency 'rails_autolink'
   gem.add_dependency 'coffee-script-source', '~> 1.8', '>= 1.8.0' # pegged until https://github.com/jashkenas/coffeescript/issues/3829 is resolved
@@ -58,6 +56,6 @@ Gem::Specification.new do |gem|
   # FatFreeCRM has released it's own versions of the following gems:
   #-----------------------------------------------------------------
   gem.add_dependency 'ransack_ui',          '~> 1.3', '>= 1.3.1'
-  gem.add_dependency 'ransack',             '~> 1.6', '>= 1.6.2'
+  gem.add_dependency 'ransack',             '~> 1.7', '>= 1.6.2'
   gem.add_dependency 'email_reply_parser_ffcrm'
 end

data/lib/development_tasks/license.rake

@@ -22,11 +22,13 @@ namespace :license do
   ],
             js: [
               "app/assets/javascripts/**/*.js",
+              "app/assets/javascripts/**/*.js.erb",
               "app/assets/stylesheets/**/*.sass", # Sass also uses javascript style comments
               "app/assets/stylesheets/**/*.scss"
             ],
             css: [
-              "app/assets/stylesheets/**/*.css"
+              "app/assets/stylesheets/**/*.css",
+              "app/assets/stylesheets/**/*.css.erb"
             ] }
 
   LICENSE_RB = %{# Copyright (c) 2008-2013 Michael Dvorkin and contributors.
@@ -37,12 +39,11 @@ namespace :license do
 }
   LICENSES = { ruby: LICENSE_RB,
                js: LICENSE_RB.gsub(/^#/, "//"),
-               css: LICENSE_RB.gsub(/^# Fat Free/, "/*\n * Fat Free")
-                    .gsub(/^#/, " \*").sub(/---\n/, "---\n */") }
+               css: "/*\n" + LICENSE_RB.gsub(/^#/, ' *').sub(/---\n/, "---\n */\n") }
 
-  REGEXPS  = { ruby: /^# Fat Free CRM\n# Copyright \(C\).*?\n(#.*\n)*#-{10}-*\n*/,
-               js: /^\/\/ Fat Free CRM\n\/\/ Copyright \(C\).*?\n(\/\/.*\n)*\/\/-{10}-*\n*/,
-               css: /^\/\*\n \* Fat Free CRM\n \* Copyright \(C\).*?\n( \*.*\n)* \*-{10}-*\n \*\/\n*/ }
+  REGEXPS  = { ruby: /^# Copyright \(c\).*?\n(?:#.*\n)*?#-{10}-*\n/,
+               js: /^\/\/ Copyright \(c\).*?\n(?:\/\/.*\n)*?\/\/-{10}-*\n/,
+               css: /^\/\*\n \* Copyright \(c\).*?\n(?: \*.*\n)*? \*-{10}-*\n \*\/\n/ }
 
   def expand_globs(globs)
     globs.map { |f| Dir.glob(f) }.flatten.uniq
@@ -52,11 +53,13 @@ namespace :license do
   task :add do
     FILES.each do |lang, globs|
       expand_globs(globs).each do |file|
-        puts "== Adding license to '#{file}'..."
         old_content = File.read(file)
         new_content = LICENSES[lang] + old_content.sub(REGEXPS[lang], '')
 
-        File.open(file, "wb") { |f| f.puts new_content }
+        if new_content != old_content
+          File.open(file, "wb") { |f| f.puts new_content }
+          puts "== Added license to #{file}"
+        end
       end
     end
   end
@@ -69,7 +72,7 @@ namespace :license do
         new_content = old_content.sub(REGEXPS[lang], '')
         if new_content != old_content
           File.open(file, "wb") { |f| f.puts new_content }
-          puts "Removed license from '#{file}'."
+          puts "== Removed license from #{file}"
         end
       end
     end

data/lib/fat_free_crm/callback.rb

@@ -98,7 +98,7 @@ module FatFreeCRM
           add_view_hook(hook, proc { "" }, :replace)
         end
       end
-    end # class Base
+    end
 
     # This makes it possible to call hook() without FatFreeCRM::Callback prefix.
     # Returns stringified data when called from within templates, and the actual
@@ -137,9 +137,9 @@ module FatFreeCRM
           FatFreeCRM::Callback.hook(method, caller, context)
         end
       end
-    end # module Helper
-  end # module Callback
-end # module FatFreeCRM
+    end
+  end
+end
 
 ActionView::Base.send(:include, FatFreeCRM::Callback::Helper)
 ActionController::Base.send(:include, FatFreeCRM::Callback::Helper)

data/lib/fat_free_crm/engine.rb

@@ -8,8 +8,8 @@ module FatFreeCRM
     config.autoload_paths += Dir[root.join("app/models/**")] +
                              Dir[root.join("app/controllers/entities")]
 
-    config.active_record.observers = [:lead_observer, :opportunity_observer,
-                                      :task_observer, :entity_observer]
+    config.active_record.observers = %i[lead_observer opportunity_observer
+                                        task_observer entity_observer]
 
     initializer "model_core.factories", after: "factory_girl.set_factory_paths" do
       FactoryGirl.definition_file_paths << File.expand_path('../../../spec/factories', __FILE__) if defined?(FactoryGirl)

data/lib/fat_free_crm/exportable.rb

@@ -43,7 +43,7 @@ module FatFreeCRM
 
     module SingletonMethods
     end
-  end # Exportable
-end # FatFreeCRM
+  end
+end
 
 ActiveRecord::Base.send(:include, FatFreeCRM::Exportable)

data/lib/fat_free_crm/fields.rb

@@ -22,7 +22,7 @@ module FatFreeCRM
 
     module SingletonMethods
       def field_groups
-        if ActiveRecord::Base.connection.table_exists? 'field_groups'
+        if ActiveRecord::Base.connection.data_source_exists? 'field_groups'
           FieldGroup.where(klass_name: name).order(:position)
         else
           []
@@ -35,9 +35,7 @@ module FatFreeCRM
 
       def serialize_custom_fields!
         fields.each do |field|
-          if !serialized_attributes.keys.include?(field.name) && field.as == 'check_boxes'
-            serialize(field.name.to_sym, Array)
-          end
+          serialize(field.name.to_sym, Array) if field.as == 'check_boxes'
         end
       end
 

data/lib/fat_free_crm/gem_ext.rb

@@ -3,7 +3,6 @@
 # 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
 #------------------------------------------------------------------------------
-require "fat_free_crm/gem_ext/active_record/schema_dumper"
 require "fat_free_crm/gem_ext/active_support/buffered_logger"
 require "fat_free_crm/gem_ext/action_controller/base"
 require "fat_free_crm/gem_ext/simple_form/action_view_extensions/form_helper"

data/lib/fat_free_crm/gem_ext/rake/task.rb

@@ -3,6 +3,8 @@
 # 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
 #------------------------------------------------------------------------------
+require 'rake/task'
+
 module Rake
   Task.class_eval do
     # Removes a Rake task

data/lib/fat_free_crm/mail_processor/base.rb

@@ -12,7 +12,7 @@ require 'nokogiri'
 module FatFreeCRM
   module MailProcessor
     class Base
-      KEYWORDS = %w(account campaign contact lead opportunity).freeze
+      KEYWORDS = %w[account campaign contact lead opportunity].freeze
 
       #--------------------------------------------------------------------------------------
       def initialize
@@ -94,7 +94,7 @@ module FatFreeCRM
 
       #--------------------------------------------------------------------------------------
       def with_new_emails
-        @imap.uid_search(%w(NOT SEEN)).each do |uid|
+        @imap.uid_search(%w[NOT SEEN]).each do |uid|
           begin
             email = Mail.new(@imap.uid_fetch(uid, 'RFC822').first.attr['RFC822'])
             log "fetched new message...", email
@@ -104,7 +104,7 @@ module FatFreeCRM
               discard(uid)
             end
           rescue Exception => e
-            if %w(test development).include?(Rails.env)
+            if %w[test development].include?(Rails.env)
               $stderr.puts e
               $stderr.puts e.backtrace
             end
@@ -184,7 +184,7 @@ module FatFreeCRM
       # Centralized logging.
       #--------------------------------------------------------------------------------------
       def log(message, email = nil)
-        unless %w(test cucumber).include?(Rails.env)
+        unless %w[test cucumber].include?(Rails.env)
           klass = self.class.to_s.split("::").last
           klass << " [Dry Run]" if @dry_run
           puts "[#{Time.now.rfc822}] #{klass}: #{message}"

data/lib/fat_free_crm/mail_processor/dropbox.rb

@@ -8,7 +8,7 @@ require 'fat_free_crm/mail_processor/base'
 module FatFreeCRM
   module MailProcessor
     class Dropbox < Base
-      KEYWORDS = %w(account campaign contact lead opportunity).freeze
+      KEYWORDS = %w[account campaign contact lead opportunity].freeze
 
       #--------------------------------------------------------------------------------------
       def initialize

data/lib/fat_free_crm/permissions.rb

@@ -33,7 +33,7 @@ module FatFreeCRM
     module InstanceMethods
       # Save shared permissions to the model, if any.
       #--------------------------------------------------------------------------
-      %w(group user).each do |model|
+      %w[group user].each do |model|
         class_eval %{
 
           def #{model}_ids=(value)
@@ -44,7 +44,7 @@ module FatFreeCRM
               permissions_to_remove = Permission.where(
                 #{model}_id: self.#{model}_ids - value,
                 asset_id: self.id,
-                asset_type: self.class
+                asset_type: self.class.name
               )
               permissions_to_remove.each {|p| (permissions.delete(p); p.destroy)}
               (value - self.#{model}_ids).each {|id| permissions.build(:#{model}_id => id)}
@@ -69,7 +69,7 @@ module FatFreeCRM
       def remove_permissions
         # we don't use dependent => :destroy so must manually remove
         if id && self.class
-          permissions_to_remove = Permission.where(asset_id: id, asset_type: self.class.to_s).to_a
+          permissions_to_remove = Permission.where(asset_id: id, asset_type: self.class.name).to_a
         else
           permissions_to_remove = []
         end
@@ -103,7 +103,7 @@ module FatFreeCRM
 
     module SingletonMethods
     end
-  end # Permissions
-end # FatFreeCRM
+  end
+end
 
 ActiveRecord::Base.send(:include, FatFreeCRM::Permissions)

data/lib/fat_free_crm/sortable.rb

@@ -36,7 +36,7 @@ module FatFreeCRM
           end
         ]
       end
-    end # ClassMethods
+    end
   end
 end
 

data/lib/fat_free_crm/version.rb

@@ -8,7 +8,7 @@ module FatFreeCRM
     MAJOR = 0
     MINOR = 15
     TINY  = 0
-    PRE   = 'beta.2'
+    PRE   = nil
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join('.')
   end

data/lib/gravatar_image_tag.rb

@@ -22,6 +22,7 @@ module GravatarImageTag
 
   module InstanceMethods
     def gravatar_image_tag(email, options = {})
+      raise ArgumentError, "Options must be a hash, got #{options.inspect}" unless options.is_a? Hash
       options[:alt] ||= 'Gravatar'
       image_tag(GravatarImageTag.gravatar_url(email, options.delete(:gravatar)), options)
     end

data/lib/tasks/ffcrm/demo.rake

@@ -10,7 +10,7 @@ namespace :ffcrm do
       # Load fixtures
       require 'active_record/fixtures'
       Dir.glob(FatFreeCRM.root.join('db', 'demo', '*.{yml,csv}')).each do |fixture_file|
-        ActiveRecord::Fixtures.create_fixtures(FatFreeCRM.root.join('db/demo'), File.basename(fixture_file, '.*'))
+        ActiveRecord::FixtureSet.create_fixtures(FatFreeCRM.root.join('db/demo'), File.basename(fixture_file, '.*'))
       end
 
       def create_version(options)
@@ -22,7 +22,7 @@ namespace :ffcrm do
       # Simulate random user activities.
       $stdout.sync = true
       puts "Generating user activities..."
-      %w(Account Address Campaign Comment Contact Email Lead Opportunity Task).map do |model|
+      %w[Account Address Campaign Comment Contact Email Lead Opportunity Task].map do |model|
         model.constantize.all
       end.flatten.each do |item|
         user = if item.respond_to?(:user)

data/spec/controllers/admin/users_controller_spec.rb

@@ -28,7 +28,7 @@ describe Admin::UsersController do
       @amy = FactoryGirl.create(:user, username: "amy_anderson")
       @bob = FactoryGirl.create(:user, username: "bob_builder")
 
-      get :index, query: "amy_anderson"
+      get :index, params: { query: "amy_anderson" }
       expect(assigns[:users]).to eq([@amy])
       expect(assigns[:current_query]).to eq("amy_anderson")
       expect(session[:users_current_query]).to eq("amy_anderson")
@@ -42,7 +42,7 @@ describe Admin::UsersController do
     it "assigns the requested user as @user and renders [show] template" do
       @user = FactoryGirl.create(:user)
 
-      get :show, id: @user.id
+      get :show, params: { id: @user.id }
       expect(assigns[:user]).to eq(@user)
       expect(response).to render_template("admin/users/show")
     end
@@ -53,7 +53,7 @@ describe Admin::UsersController do
   #----------------------------------------------------------------------------
   describe "GET new" do
     it "assigns a new user as @user and renders [new] template" do
-      xhr :get, :new
+      get :new, xhr: true
       expect(assigns[:user]).to be_new_record
       expect(response).to render_template("admin/users/new")
     end
@@ -65,7 +65,7 @@ describe Admin::UsersController do
     it "assigns the requested user as @user and renders [edit] template" do
       @user = FactoryGirl.create(:user)
 
-      xhr :get, :edit, id: @user.id
+      get :edit, params: { id: @user.id }, xhr: true
       expect(assigns[:user]).to eq(@user)
       expect(assigns[:previous]).to eq(nil)
       expect(response).to render_template("admin/users/edit")
@@ -75,7 +75,7 @@ describe Admin::UsersController do
       @user = FactoryGirl.create(:user)
       @previous = FactoryGirl.create(:user)
 
-      xhr :get, :edit, id: @user.id, previous: @previous.id
+      get :edit, params: { id: @user.id, previous: @previous.id }, xhr: true
       expect(assigns[:previous]).to eq(@previous)
     end
 
@@ -83,7 +83,7 @@ describe Admin::UsersController do
       @user = FactoryGirl.create(:user)
       @user.destroy
 
-      xhr :get, :edit, id: @user.id
+      get :edit, params: { id: @user.id }, xhr: true
       expect(flash[:warning]).not_to eq(nil)
       expect(response.body).to eq("window.location.reload();")
     end
@@ -93,7 +93,7 @@ describe Admin::UsersController do
       @previous = FactoryGirl.create(:user)
       @previous.destroy
 
-      xhr :get, :edit, id: @user.id, previous: @previous.id
+      get :edit, params: { id: @user.id, previous: @previous.id }, xhr: true
       expect(flash[:warning]).to eq(nil) # no warning, just silently remove the div
       expect(assigns[:previous]).to eq(@previous.id)
       expect(response).to render_template("admin/users/edit")
@@ -115,19 +115,19 @@ describe Admin::UsersController do
         @user = FactoryGirl.build(:user, username: @username, email: @email)
         allow(User).to receive(:new).and_return(@user)
 
-        xhr :post, :create, user: { username: @username, email: @email, password: @password, password_confirmation: @password }
+        post :create, params: { user: { username: @username, email: @email, password: @password, password_confirmation: @password } }, xhr: true
         expect(assigns[:user]).to eq(@user)
         expect(response).to render_template("admin/users/create")
       end
 
       it "creates admin user when requested so" do
-        xhr :post, :create, user: { username: @username, email: @email, admin: "1", password: @password, password_confirmation: @password }
+        post :create, params: { user: { username: @username, email: @email, admin: "1", password: @password, password_confirmation: @password } }, xhr: true
         expect(assigns[:user].admin).to eq(true)
         expect(response).to render_template("admin/users/create")
       end
 
       it "doesn't create admin user unless requested so" do
-        xhr :post, :create, user: { username: @username, email: @email, admin: "0", password: @password, password_confirmation: @password }
+        post :create, params: { user: { username: @username, email: @email, admin: "0", password: @password, password_confirmation: @password } }, xhr: true
         expect(assigns[:user].admin).to eq(false)
         expect(response).to render_template("admin/users/create")
       end
@@ -138,7 +138,7 @@ describe Admin::UsersController do
         @user = FactoryGirl.build(:user, username: "", email: "")
         allow(User).to receive(:new).and_return(@user)
 
-        xhr :post, :create, user: {}
+        post :create, params: { user: {} }, xhr: true
         expect(assigns[:user]).to eq(@user)
         expect(response).to render_template("admin/users/create")
       end
@@ -153,7 +153,7 @@ describe Admin::UsersController do
       it "updates the requested user, assigns it to @user, and renders [update] template" do
         @user = FactoryGirl.create(:user, username: "flip", email: "flip@example.com")
 
-        xhr :put, :update, id: @user.id, user: { username: "flop", email: "flop@example.com" }
+        put :update, params: { id: @user.id, user: { username: "flop", email: "flop@example.com" } }, xhr: true
         expect(assigns[:user]).to eq(@user.reload)
         expect(assigns[:user].username).to eq("flop")
         expect(response).to render_template("admin/users/update")
@@ -163,14 +163,14 @@ describe Admin::UsersController do
         @user = FactoryGirl.create(:user)
         @user.destroy
 
-        xhr :put, :update, id: @user.id, user: { username: "flop", email: "flop@example.com" }
+        put :update, params: { id: @user.id, user: { username: "flop", email: "flop@example.com" } }, xhr: true
         expect(flash[:warning]).not_to eq(nil)
         expect(response.body).to eq("window.location.reload();")
       end
 
       it "assigns admin rights when requested so" do
         @user = FactoryGirl.create(:user, admin: false)
-        xhr :put, :update, id: @user.id, user: { admin: "1", username: @user.username, email: @user.email }
+        put :update, params: { id: @user.id, user: { admin: "1", username: @user.username, email: @user.email } }, xhr: true
         expect(assigns[:user]).to eq(@user.reload)
         expect(assigns[:user].admin).to eq(true)
         expect(response).to render_template("admin/users/update")
@@ -178,7 +178,7 @@ describe Admin::UsersController do
 
       it "revokes admin rights when requested so" do
         @user = FactoryGirl.create(:user, admin: true)
-        xhr :put, :update, id: @user.id, user: { admin: "0", username: @user.username, email: @user.email }
+        put :update, params: { id: @user.id, user: { admin: "0", username: @user.username, email: @user.email } }, xhr: true
         expect(assigns[:user]).to eq(@user.reload)
         expect(assigns[:user].admin).to eq(false)
         expect(response).to render_template("admin/users/update")
@@ -189,7 +189,7 @@ describe Admin::UsersController do
       it "doesn't update the requested user, but assigns it to @user and renders [update] template" do
         @user = FactoryGirl.create(:user, username: "flip", email: "flip@example.com")
 
-        xhr :put, :update, id: @user.id, user: {}
+        put :update, params: { id: @user.id, user: {} }, xhr: true
         expect(assigns[:user]).to eq(@user.reload)
         expect(assigns[:user].username).to eq("flip")
         expect(response).to render_template("admin/users/update")
@@ -203,7 +203,7 @@ describe Admin::UsersController do
     it "assigns the requested user as @user and renders [confirm] template" do
       @user = FactoryGirl.create(:user)
 
-      xhr :get, :confirm, id: @user.id
+      get :confirm, params: { id: @user.id }, xhr: true
       expect(assigns[:user]).to eq(@user)
       expect(response).to render_template("admin/users/confirm")
     end
@@ -212,7 +212,7 @@ describe Admin::UsersController do
       @user = FactoryGirl.create(:user)
       @user.destroy
 
-      xhr :get, :confirm, id: @user.id
+      get :confirm, params: { id: @user.id }, xhr: true
       expect(flash[:warning]).not_to eq(nil)
       expect(response.body).to eq("window.location.reload();")
     end
@@ -225,7 +225,7 @@ describe Admin::UsersController do
     it "destroys the requested user and renders [destroy] template" do
       @user = FactoryGirl.create(:user)
 
-      xhr :delete, :destroy, id: @user.id
+      delete :destroy, params: { id: @user.id }, xhr: true
       expect { User.find(@user.id) }.to raise_error(ActiveRecord::RecordNotFound)
       expect(response).to render_template("admin/users/destroy")
     end
@@ -234,7 +234,7 @@ describe Admin::UsersController do
       @user = FactoryGirl.create(:user)
       @account = FactoryGirl.create(:account, user: @user) # Plant artifact to prevent the user from being deleted.
 
-      xhr :delete, :destroy, id: @user.id
+      delete :destroy, params: { id: @user.id }, xhr: true
       expect(flash[:warning]).not_to eq(nil)
       expect { User.find(@user.id) }.not_to raise_error
       expect(response).to render_template("admin/users/destroy")
@@ -258,7 +258,7 @@ describe Admin::UsersController do
     it "suspends the requested user" do
       @user = FactoryGirl.create(:user)
 
-      xhr :put, :suspend, id: @user.id
+      put :suspend, params: { id: @user.id }, xhr: true
       expect(assigns[:user].suspended?).to eq(true)
       expect(response).to render_template("admin/users/suspend")
     end
@@ -266,7 +266,7 @@ describe Admin::UsersController do
     it "doesn't suspend current user" do
       @user = current_user
 
-      xhr :put, :suspend, id: @user.id
+      put :suspend, params: { id: @user.id }, xhr: true
       expect(assigns[:user].suspended?).to eq(false)
       expect(response).to render_template("admin/users/suspend")
     end
@@ -275,7 +275,7 @@ describe Admin::UsersController do
       @user = FactoryGirl.create(:user)
       @user.destroy
 
-      xhr :put, :suspend, id: @user.id
+      put :suspend, params: { id: @user.id }, xhr: true
       expect(flash[:warning]).not_to eq(nil)
       expect(response.body).to eq("window.location.reload();")
     end
@@ -288,7 +288,7 @@ describe Admin::UsersController do
     it "re-activates the requested user" do
       @user = FactoryGirl.create(:user, suspended_at: Time.now.yesterday)
 
-      xhr :put, :reactivate, id: @user.id
+      put :reactivate, params: { id: @user.id }, xhr: true
       expect(assigns[:user].suspended?).to eq(false)
       expect(response).to render_template("admin/users/reactivate")
     end
@@ -297,7 +297,7 @@ describe Admin::UsersController do
       @user = FactoryGirl.create(:user)
       @user.destroy
 
-      xhr :put, :reactivate, id: @user.id
+      put :reactivate, params: { id: @user.id }, xhr: true
       expect(flash[:warning]).not_to eq(nil)
       expect(response.body).to eq("window.location.reload();")
     end