Fix Travis integration

Author: federicoaldunate

.rubocop.yml

@@ -1,6 +1,7 @@
 AllCops:
   Exclude:
     - spec/dummy/db/schema.rb
+    - vendor/bundle/**/*
 
 Documentation:
   Enabled: false
@@ -62,6 +63,7 @@ Metrics/LineLength:
   Exclude:
     - spec/dummy/**/*
     - spec/channels/chat_channel_spec.rb
+    - config/routes.rb
 
 Metrics/ParameterLists:
   Max: 5

.travis.yml

@@ -2,3 +2,10 @@ language: ruby
 cache: bundler
 rvm:
   - 2.5.1
+
+sudo: required
+
+before_install:
+  - sudo apt-get update
+  - sudo apt-get install chromium-chromedriver
+  - "export PATH=$PATH:/usr/lib/chromium-browser/"

README.md

@@ -1,4 +1,7 @@
 # ActiveAdminChat
+
+[![Build Status](https://travis-ci.org/rootstrap/active_admin_chat.svg?branch=master)](https://travis-ci.org/rootstrap/active_admin_chat)
+
 Get a chat for your ActiveAdmin app out of the box.
 
 ## Installation

config/routes.rb

@@ -3,7 +3,7 @@
     namespace ActiveAdminChat.namespace do
       get  "#{ActiveAdminChat.page_name}/:id", to: "#{ActiveAdminChat.page_name}#show"
       post "#{ActiveAdminChat.user_model_name}/:#{ActiveAdminChat.user_relation_name}_id/#{ActiveAdminChat.page_name}",
-        to: "#{ActiveAdminChat.page_name}#create"
+           to: "#{ActiveAdminChat.page_name}#create"
     end
   end
 end

lib/active_admin_chat.rb

@@ -33,4 +33,7 @@ def setup
 
 ::ActiveAdmin.send :include, ActiveAdminChat::ActiveAdmin
 ::ActiveAdmin::Application.send :include, ActiveAdminChat::ActiveAdmin::Application
-::ActiveAdmin::Views::IndexAsTable::IndexTableFor.send :prepend, ActiveAdminChat::ActiveAdmin::Views::IndexAsTable::IndexTableFor
+::ActiveAdmin::Views::IndexAsTable::IndexTableFor.send(
+  :prepend,
+  ActiveAdminChat::ActiveAdmin::Views::IndexAsTable::IndexTableFor
+)

lib/active_admin_chat/active_admin/application.rb

@@ -23,10 +23,11 @@ def show
             end
 
             def create
-              conversation = ActiveAdminChat.conversation_klass
-                                            .find_or_create_by!(
-                                              "#{user_relation_name_id}": params[:"#{user_relation_name_id}"]
-                                            )
+              conversation =
+                ActiveAdminChat.conversation_klass
+                               .find_or_create_by!(
+                                 "#{user_relation_name_id}": params[:"#{user_relation_name_id}"]
+                               )
               redirect_to action: 'show', id: conversation
             end
 
@@ -48,7 +49,8 @@ def messages
                                                  .includes(:sender).order(created_at: :desc)
 
               if params[:created_at].present?
-                page_messages = page_messages.where('created_at < ?', DateTime.parse(params[:created_at]))
+                page_messages = page_messages.where('created_at < ?',
+                                                    DateTime.parse(params[:created_at]))
               end
               page_messages.limit(ActiveAdminChat.messages_per_page).reverse
             end

lib/active_admin_chat/active_admin/index_table_for.rb

@@ -5,9 +5,15 @@ module IndexAsTable
         module IndexTableFor
           def send_message_link(resource)
             text_node link_to 'Send Message',
-                              "#{ActiveAdminChat.user_model_name}/#{resource.id}/#{ActiveAdminChat.page_name}",
+                              send_message_path(resource.id),
                               method: :post
           end
+
+          private
+
+          def send_message_path(id)
+            "#{ActiveAdminChat.user_model_name}/#{id}/#{ActiveAdminChat.page_name}"
+          end
         end
       end
     end

spec/features/chat_page_spec.rb

@@ -46,7 +46,8 @@
       find('#send-message').native.send_keys(:return)
 
       expect(page).to have_content('A new message')
-      expect(page).to have_content(Time.current.strftime(':%M - %m/%d/%Y').sub(/(0)?(\d)\/0(\d)/, '\2/\3'))
+      expect(page).to have_content(Time.current.strftime(':%M - %m/%d/%Y')
+                                       .sub(%r{(0)?(\d)\/0(\d)}, '\2/\3'))
 
       visit admin_chat_path
 
@@ -73,7 +74,8 @@
       find('#send-message').native.send_keys(:return)
 
       expect(page).to have_content('A new message')
-      expect(page).to have_content(Time.current.strftime(':%M - %m/%d/%Y').sub(/(0)?(\d)\/0(\d)/, '\2/\3'))
+      expect(page).to have_content(Time.current.strftime(':%M - %m/%d/%Y')
+                                       .sub(%r{(0)?(\d)\/0(\d)}, '\2/\3'))
 
       visit admin_chat_path
 

spec/generators/install_generator_spec.rb

@@ -43,7 +43,7 @@
             contains "config.message_model_name = 'message'"
             contains "config.admin_user_model_name = 'admin_user'"
             contains "config.user_model_name = 'api/user'"
-            contains "config.messages_per_page = 25"
+            contains 'config.messages_per_page = 25'
           end
         end
       end

spec/rails_helper.rb

@@ -7,6 +7,7 @@
 require 'rspec/rails'
 # Add additional requires below this line. Rails is not loaded until this point!
 require 'action_cable/testing/rspec'
+require 'selenium/webdriver'
 
 # Setup ActiveAdmin
 ActiveAdmin.application.load_paths = [File.expand_path('dummy/app/admin', __dir__)]
@@ -29,6 +30,22 @@
 #
 # Dir[Rails.root.join('spec', 'support', '**', '*.rb')].each { |f| require f }
 
+Capybara.register_driver :chrome do |app|
+  Capybara::Selenium::Driver.new(app, browser: :chrome)
+end
+
+Capybara.register_driver :headless_chrome do |app|
+  capabilities = Selenium::WebDriver::Remote::Capabilities.chrome(
+    chromeOptions: { args: %w[headless no-sandbox disable-gpu] }
+  )
+
+  Capybara::Selenium::Driver.new app,
+                                 browser: :chrome,
+                                 desired_capabilities: capabilities
+end
+
+Capybara.javascript_driver = :headless_chrome
+
 # Checks for pending migrations and applies them before tests are run.
 # If you are not using ActiveRecord, you can remove these lines.
 begin