fix(tests) test on Rails 3.2, fix no-sprockets test

Author: rmosolgo

.travis.yml

@@ -26,8 +26,6 @@ matrix:
   exclude:
     - rvm: 2.1
       gemfile: gemfiles/rails_3.2.gemfile
-    - rvm: 2.3.0
-      gemfile: gemfiles/rails_3.2.gemfile
     - rvm: 2.1
       gemfile: gemfiles/rails_4.0.gemfile
     - rvm: 2.1

Appraisals

@@ -1,10 +1,8 @@
 appraise "rails-3.2" do
   gem 'rails', '~> 3.2.21'
   gem 'rack-cache', '~> 1.6.1'
-end
-
-appraise "rails-4.0" do
-  gem 'rails', '~> 4.0.13'
+  # Modern turbolinks depends on `Rails.application.assets` which no-worky
+  gem 'turbolinks', '~> 2.0'
 end
 
 appraise "rails-4.0.5" do
@@ -51,7 +49,6 @@ end
 
 appraise "rails-5-no_sprockets" do
   gem 'rails', '~> 5.0.0'
-  gem "turbolinks", "~> 5.0.0"
 end
 
 appraise "rails-5-sprockets_4" do

gemfiles/rails_3.2.gemfile

@@ -4,5 +4,6 @@ source "http://rubygems.org"
 
 gem "rails", "~> 3.2.21"
 gem "rack-cache", "~> 1.6.1"
+gem "turbolinks", "~> 2.0"
 
 gemspec :path => "../"

gemfiles/rails_5_no_sprockets.gemfile

@@ -2,7 +2,6 @@
 
 source "http://rubygems.org"
 
-gem "rails", "~> 5.0.0"
-gem "turbolinks", "~> 5.0.0", require: false
+gem "rails", "5.0.0"
 
 gemspec :path => "../"

lib/react/server_rendering/yaml_manifest_container.rb

@@ -6,18 +6,24 @@ module ServerRendering
     # but sometimes, they're compiled to other directories (or other servers)
     class YamlManifestContainer
       def initialize
-        @assets = YAML.load_file(::Rails.root.join("public", ::Rails.application.config.assets.prefix, "manifest.yml"))
+        @assets = YAML.load_file(public_asset_path("manifest.yml"))
       end
 
       def find_asset(logical_path)
         asset_path = @assets[logical_path] || raise("No compiled asset for #{logical_path}, was it precompiled?")
-        asset_full_path = ::Rails.root.join("public", ::Rails.application.config.assets.prefix, asset_path)
-        File.read(asset_full_path)
+        File.read(public_asset_path(asset_path))
       end
 
       def self.compatible?
         ::Rails::VERSION::MAJOR == 3
       end
+
+      private
+
+      def public_asset_path(asset_name)
+        asset_path = File.join("public", ::Rails.application.config.assets.prefix, asset_name)
+        ::Rails.root.join(asset_path)
+      end
     end
   end
 end

react-rails.gemspec

@@ -25,7 +25,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'listen', '~> 3.0.0' # support Ruby 2.1
   s.add_development_dependency 'poltergeist', '>= 0.3.3'
   s.add_development_dependency 'test-unit', '~> 2.5'
-  s.add_development_dependency 'turbolinks', '>= 2.0.0'
   s.add_development_dependency 'rails', '>= 3.2'
 
   s.add_dependency 'coffee-script-source', '~>1.8'

test/react/server_rendering/sprockets_renderer_test.rb

@@ -10,7 +10,7 @@ class SprocketsRendererTest < ActiveSupport::TestCase
 
     CALLBACKS.each do |callback_name|
       test "#render should pass prerender options to ##{callback_name}" do
-        mocked_method = MiniTest::Mock.new
+        mocked_method = Minitest::Mock.new
         mocked_method.expect :call, '', [
           "Todo",
           "{\"todo\":\"write tests\"}",

test/react/server_rendering/yaml_manifest_container_test.rb

@@ -18,4 +18,3 @@ def test_find_asset_gets_asset_contents
     end
   end
 end
-