per self review

Author: Judahmeek

.github/workflows/ruby.yml

@@ -21,8 +21,8 @@ jobs:
     - uses: actions/setup-node@v3
     - run: npm -g install yalc
     - run: yalc publish
-    - run: cd test/dummy_shakapacker && yalc add react_ujs
-    - run: cd test/dummy_shakapacker && yarn
+    - run: cd test/app && yalc add react_ujs
+    - run: cd test/app && yarn
     - uses: ruby/setup-ruby@v1
       with:
         ruby-version: 2.7.3

.gitignore

@@ -8,4 +8,4 @@ test/*/public/packs
 react-builds/build
 coverage/
 **/.yalc
-**/yalc.*
+**/yalc.*

Rakefile

@@ -84,7 +84,7 @@ task default: :test
 
 task :test_setup do
   load 'webdrivers/Rakefile'
-  Dir.chdir('./test/dummy_shakapacker') do
+  Dir.chdir('./test/dummy') do
     `yarn install`
   end
 end

lib/react/server_rendering/bundle_renderer.rb

@@ -49,7 +49,7 @@ def after_render(component_name, props, prerender_options)
       end
 
       class << self
-        attr_accessor :asset_container_class, :dont_use_webpacker_manifest
+        attr_accessor :asset_container_class
       end
 
       # Get an object which exposes assets by their logical path.

test/dummy_shakapacker/.gitignore renamed to test/dummy/.gitignore

@@ -7,8 +7,6 @@ module.exports = createReactClass({
     if (typeof global !== "undefined" && global.ctx && global.ctx.greeting) {
       initialGreeting = global.ctx.greeting
     }
-    console.log(`global: ${global}`)
-    console.log(`global.ctx: ${JSON.stringify(global.ctx)}`)
 
     return {
       greeting: initialGreeting

test/dummy_shakapacker/.postcssrc.yml renamed to test/dummy/.postcssrc.yml

@@ -6,7 +6,7 @@ def available?
     # We also can't scan for defined?(Sprockets) because this is used to
     #   require Sprockets in the config/application.rb
     # !!Bundler.locked_gems.specs.find {|gem_spec| gem_spec.name == 'sprockets'}
-    false
+    true
   end
 
   # The block depends on sprockets, don't run it if sprockets is missing
@@ -42,16 +42,8 @@ def precompile_assets
       # - assets:precompile runs webpacker:compile when availabled
       # - webpacker:compile depends on `./bin/webpack`, so `.` must be the app root
       Dir.chdir("./test/#{DUMMY_LOCATION}") do
-
-        ENV['RAILS_GROUPS'] = 'assets' # required for Rails 3.2
         Rake::Task['assets:precompile'].reenable
 
-        if Rails::VERSION::MAJOR == 3
-          Rake::Task['assets:precompile:all'].reenable
-          Rake::Task['assets:precompile:primary'].reenable
-          Rake::Task['assets:precompile:nondigest'].reenable
-        end
-
         Rake::Task['assets:precompile'].invoke
       end
     end

test/dummy_shakapacker/README.rdoc renamed to test/dummy/README.rdoc

@@ -3,7 +3,7 @@
   SimpleCov.start
 end
 
-DUMMY_LOCATION = 'dummy_shakapacker'
+DUMMY_LOCATION = 'dummy'
 
 support_path = File.expand_path('../support/*.rb', __FILE__)
 Dir.glob(support_path).each do |f|