feat(Rails) support 5.1

Author: rmosolgo

Appraisals

@@ -65,8 +65,8 @@ appraise "rails-5-no_sprockets" do
   gem 'rails', '~> 5.0.0'
 end
 
-appraise "rails-5-sprockets_4" do
-  gem "rails", "~> 5.0.0"
+appraise "rails-5.1-sprockets_4" do
+  gem "rails", "5.1.0.rc1"
   gem "sprockets", "~> 4.0.x"
   gem "turbolinks", "~> 5.0.0"
 end

gemfiles/rails_5_sprockets_4.gemfile renamed to gemfiles/rails_5.1_sprockets_4.gemfile

@@ -3,7 +3,7 @@
 source "http://rubygems.org"
 
 gem "turbolinks", "~> 5.0.0"
-gem "rails", "~> 5.0.0"
+gem "rails", "5.1.0.rc1"
 gem "sprockets", "~> 4.0.x"
 
 gemspec :path => "../"

lib/react/rails/railtie.rb

@@ -33,7 +33,7 @@ class Railtie < ::Rails::Railtie
         end
 
         # Rails checks these objects for changes:
-        app.reloaders << app.config.file_watcher.new([], reload_paths)
+        app.reloaders << app.config.file_watcher.new([], reload_paths) {}
         # Reload renderers in dev when files change
         config.to_prepare { React::ServerRendering.reset_pool }
       end

test/react/rails/pages_controller_test.rb

@@ -1,12 +1,13 @@
 require 'test_helper'
 
 class PagesControllerTest < ActionController::TestCase
+  include ParamsHelper
   setup do
     WebpackerHelpers.compile_if_missing
   end
 
   test 'renders successfully' do
-    get :show, id: 1
+    get :show, query_params(id: 1)
     assert_equal(200, response.status)
   end
 

test/test_helper.rb

@@ -101,3 +101,14 @@ def when_stateful_js_context_available
     yield
   end
 end
+
+module ParamsHelper
+  # Normalize params for Rails 5.1+
+  def query_params(params)
+    if Rails::VERSION.to_s > "5.0"
+      {params: params}
+    else
+      params
+    end
+  end
+end