Merge pull request #17 from rootstrap/task/travis-different-versions

santiagovidal · web-flow · commit a62725415c21 · 2018-12-12T17:34:33.000-03:00
Add testing in travis with different versions
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -2,6 +2,7 @@ AllCops:
   Exclude:
     - spec/dummy/db/schema.rb
     - vendor/bundle/**/*
+    - gemfiles/vendor/bundle/**/*
 
 Documentation:
   Enabled: false
diff --git a/.travis.yml b/.travis.yml
@@ -5,11 +5,16 @@ env:
 language: ruby
 cache: bundler
 rvm:
-  - 2.5.1
+  - 2.2.10
+  - 2.3.8
+  - 2.4.5
+  - 2.5.3
+  - ruby-head
 
 sudo: required
 
 before_install:
+  - bundle install
   - sudo apt-get update
   - sudo apt-get install chromium-chromedriver
   - "export PATH=$PATH:/usr/lib/chromium-browser/"
@@ -19,3 +24,16 @@ before_install:
 
 after_script:
   - ./cc-test-reporter after-build --exit-code $TRAVIS_TEST_RESULT
+
+gemfile:
+  - gemfiles/5.0.gemfile
+  - gemfiles/5.1.gemfile
+  - gemfiles/5.2.gemfile
+
+matrix:
+  fast_finish: true
+  exclude:
+    - gemfile: gemfiles/5.2.gemfile
+      rvm: 2.2.10
+  allow_failures:
+    - rvm: ruby-head
diff --git a/active_admin_chat.gemspec b/active_admin_chat.gemspec
@@ -22,6 +22,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'action-cable-testing'
   s.add_development_dependency 'byebug'
   s.add_development_dependency 'capybara'
+  s.add_development_dependency 'database_cleaner'
   s.add_development_dependency 'factory_bot_rails'
   s.add_development_dependency 'generator_spec'
   s.add_development_dependency 'puma'
diff --git a/gemfiles/5.0.gemfile b/gemfiles/5.0.gemfile
@@ -0,0 +1,20 @@
+source 'https://rubygems.org'
+
+gem 'activeadmin', '~> 1.0.0'
+gem 'rails', '~> 5.0.0'
+
+group :development, :test do
+  gem 'action-cable-testing'
+  gem 'byebug'
+  gem 'capybara'
+  gem 'database_cleaner'
+  gem 'factory_bot_rails'
+  gem 'generator_spec'
+  gem 'puma'
+  gem 'rspec-rails', '~> 3.8'
+  gem 'rubocop', '~> 0.59.2'
+  gem 'selenium-webdriver'
+  gem 'simplecov'
+  gem 'sqlite3'
+  gem 'timecop'
+end
diff --git a/gemfiles/5.1.gemfile b/gemfiles/5.1.gemfile
@@ -0,0 +1,20 @@
+source 'https://rubygems.org'
+
+gem 'activeadmin', '~> 1.2.1'
+gem 'rails', '~> 5.1.0'
+
+group :development, :test do
+  gem 'action-cable-testing'
+  gem 'byebug'
+  gem 'capybara'
+  gem 'database_cleaner'
+  gem 'factory_bot_rails'
+  gem 'generator_spec'
+  gem 'puma'
+  gem 'rspec-rails', '~> 3.8'
+  gem 'rubocop', '~> 0.59.2'
+  gem 'selenium-webdriver'
+  gem 'simplecov'
+  gem 'sqlite3'
+  gem 'timecop'
+end
diff --git a/gemfiles/5.2.gemfile b/gemfiles/5.2.gemfile
@@ -0,0 +1,20 @@
+source 'https://rubygems.org'
+
+gem 'activeadmin', '~> 1.4.3'
+gem 'rails', '~> 5.2.0'
+
+group :development, :test do
+  gem 'action-cable-testing'
+  gem 'byebug'
+  gem 'capybara'
+  gem 'database_cleaner'
+  gem 'factory_bot_rails'
+  gem 'generator_spec'
+  gem 'puma'
+  gem 'rspec-rails', '~> 3.8'
+  gem 'rubocop', '~> 0.59.2'
+  gem 'selenium-webdriver'
+  gem 'simplecov'
+  gem 'sqlite3'
+  gem 'timecop'
+end
diff --git a/spec/dummy/app/assets/javascripts/application.js b/spec/dummy/app/assets/javascripts/application.js
@@ -10,6 +10,4 @@
 // Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
 // about supported directives.
 //
-//= require rails-ujs
-//= require activestorage
 //= require_tree .
diff --git a/spec/dummy/app/views/layouts/application.html.erb b/spec/dummy/app/views/layouts/application.html.erb
@@ -3,7 +3,6 @@
   <head>
     <title>Dummy</title>
     <%= csrf_meta_tags %>
-    <%= csp_meta_tag %>
 
     <%= stylesheet_link_tag    'application', media: 'all' %>
     <%= javascript_include_tag 'application' %>
diff --git a/spec/dummy/config/application.rb b/spec/dummy/config/application.rb
@@ -5,12 +5,12 @@
 require 'active_model/railtie'
 require 'active_job/railtie'
 require 'active_record/railtie'
-require 'active_storage/engine'
 require 'action_controller/railtie'
 require 'action_mailer/railtie'
 require 'action_view/railtie'
 require 'action_cable/engine'
 require 'sprockets/railtie'
+
 # require "rails/test_unit/railtie"
 
 Bundler.require(*Rails.groups)
@@ -19,7 +19,6 @@
 module Dummy
   class Application < Rails::Application
     # Initialize configuration defaults for originally generated Rails version.
-    config.load_defaults 5.2
 
     # Settings in config/environments/* take precedence over those specified here.
     # Application configuration can go into files in config/initializers
diff --git a/spec/dummy/config/environments/development.rb b/spec/dummy/config/environments/development.rb
@@ -41,9 +41,6 @@
   # Raise an error on page load if there are pending migrations.
   config.active_record.migration_error = :page_load
 
-  # Highlight code that triggered database queries in logs.
-  config.active_record.verbose_query_logs = true
-
   # Debug mode disables concatenation and preprocessing of assets.
   # This option may cause significant delays in view rendering with a large
   # number of complex assets.
diff --git a/spec/dummy/config/environments/test.rb b/spec/dummy/config/environments/test.rb
@@ -28,9 +28,6 @@
   # Disable request forgery protection in test environment.
   config.action_controller.allow_forgery_protection = false
 
-  # Store uploaded files on the local file system in a temporary directory
-  config.active_storage.service = :test
-
   config.action_mailer.perform_caching = false
 
   # Tell Action Mailer not to deliver emails to the real world.
diff --git a/spec/dummy/config/secrets.yml b/spec/dummy/config/secrets.yml
@@ -0,0 +1,6 @@
+
+development:
+  secret_key_base: b9ed8530bded47950895f26188cc21a984e271e262d8e8b8c78c8fc57c65a65fd5818db617906414b9f8b5f608bdc115af3aaa1714431e66aa538c2b8c749304
+
+test:
+  secret_key_base: b9ed8530bded47950895f26188cc21a984e271e262d8e8b8c78c8fc57c65a65fd5818db617906414b9f8b5f608bdc115af3aaa1714431e66aa538c2b8c749304
diff --git a/spec/dummy/db/schema.rb b/spec/dummy/db/schema.rb
@@ -10,34 +10,34 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2018_10_29_153133) do
+ActiveRecord::Schema.define(version: 20181029153133) do
 
   create_table "admin_users", force: :cascade do |t|
-    t.string "email"
+    t.string   "email"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
   end
 
   create_table "api_people", force: :cascade do |t|
-    t.string "email"
+    t.string   "email"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
   end
 
   create_table "conversations", force: :cascade do |t|
-    t.integer "person_id"
+    t.integer  "person_id"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.index ["person_id"], name: "index_conversations_on_person_id"
   end
 
   create_table "texts", force: :cascade do |t|
-    t.string "content"
-    t.string "sender_type"
-    t.integer "sender_id"
-    t.integer "conversation_id"
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
+    t.string   "content"
+    t.string   "sender_type"
+    t.integer  "sender_id"
+    t.integer  "conversation_id"
+    t.datetime "created_at",      null: false
+    t.datetime "updated_at",      null: false
     t.index ["conversation_id"], name: "index_texts_on_conversation_id"
     t.index ["sender_type", "sender_id"], name: "index_texts_on_sender_type_and_sender_id"
   end
diff --git a/spec/features/user_page_spec.rb b/spec/features/user_page_spec.rb
@@ -12,7 +12,6 @@
 
   scenario 'see the conversation already created and send a message' do
     visit admin_api_people_path
-
     expect(page).to have_content('Send Message')
 
     within("#api_person_#{person1.id}") do
diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
@@ -7,8 +7,10 @@
 require 'rspec/rails'
 # Add additional requires below this line. Rails is not loaded until this point!
 require 'action_cable/testing/rspec'
+require 'database_cleaner'
 require 'selenium/webdriver'
 require 'simplecov'
+
 SimpleCov.start
 
 # Setup ActiveAdmin
@@ -63,7 +65,7 @@
   # If you're not using ActiveRecord, or you'd prefer not to run each of your
   # examples within a transaction, remove the following line or assign false
   # instead of true.
-  config.use_transactional_fixtures = true
+  config.use_transactional_fixtures = false
 
   # RSpec Rails can automatically mix in different behaviours to your tests
   # based on their file location, for example enabling you to call `get` and
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -48,6 +48,25 @@
   # triggering implicit auto-inclusion in groups with matching metadata.
   config.shared_context_metadata_behavior = :apply_to_host_groups
 
+  config.before :suite do
+    DatabaseCleaner.clean_with(
+      :truncation,
+      except: %w[ar_internal_metadata]
+    )
+  end
+
+  config.before :each do
+    DatabaseCleaner.strategy = :truncation
+    DatabaseCleaner.start
+  end
+
+  config.after do
+    DatabaseCleaner.clean_with(
+      :truncation,
+      except: %w[ar_internal_metadata]
+    )
+  end
+
   # The settings below are suggested to provide a good initial experience
   # with RSpec, but feel free to customize to your heart's content.
   #   # This allows you to limit a spec run to individual examples or groups
