feat(ComponentGenerator) output components to Webpacker source dir if present

Author: rmosolgo

lib/generators/react/component_generator.rb

@@ -4,7 +4,7 @@ class ComponentGenerator < ::Rails::Generators::NamedBase
       source_root File.expand_path '../../templates', __FILE__
       desc <<-DESC.strip_heredoc
       Description:
-          Scaffold a react component into app/assets/javascripts/components.
+          Scaffold a React component into `components/` of your Webpacker source or asset pipeline.
           The generated component will include a basic render function and a PropTypes
           hash to help with development.
 
@@ -90,17 +90,29 @@ class ComponentGenerator < ::Rails::Generators::NamedBase
       }
 
       def create_component_file
-        extension = case
-                      when options[:es6]
-                        'es6.jsx'
-                      when options[:coffee]
-                        'js.jsx.coffee'
-                      else
-                        'js.jsx'
-                    end
-
-        file_path = File.join('app/assets/javascripts/components', "#{file_name}.#{extension}")
-        template("component.#{extension}", file_path)
+        template_extension = case
+        when options[:es6]
+          'es6.jsx'
+        when options[:coffee]
+          'js.jsx.coffee'
+        else
+          'js.jsx'
+        end
+
+        # Prefer webpacker to sprockets:
+        if defined?(Webpacker)
+          extension = options[:coffee] ? "coffee" : "js"
+          target_dir = Webpacker::Configuration.source_path
+            .join("components")
+            .relative_path_from(::Rails.root)
+            .to_s
+        else
+          extension = template_extension
+          target_dir = 'app/assets/javascripts/components'
+        end
+
+        file_path = File.join(target_dir, "#{file_name}.#{extension}")
+        template("component.#{template_extension}", file_path)
       end
 
       private

test/generators/coffee_component_generator_test.rb

@@ -6,10 +6,15 @@ class CoffeeComponentGeneratorTest < Rails::Generators::TestCase
   setup :prepare_destination
   tests React::Generators::ComponentGenerator
 
-  def filename
-    'app/assets/javascripts/components/generated_component.js.jsx.coffee'
+  if WebpackerHelpers.available?
+    def filename
+      "app/javascript/components/generated_component.coffee"
+    end
+  else
+    def filename
+      'app/assets/javascripts/components/generated_component.js.jsx.coffee'
+    end
   end
-
   def class_name
     'GeneratedComponent'
   end

test/generators/component_generator_test.rb

@@ -6,8 +6,14 @@ class ComponentGeneratorTest < Rails::Generators::TestCase
   setup :prepare_destination
   tests React::Generators::ComponentGenerator
 
-  def filename
-    'app/assets/javascripts/components/generated_component.js.jsx'
+  if WebpackerHelpers.available?
+    def filename
+      "app/javascript/components/generated_component.js"
+    end
+  else
+    def filename
+      'app/assets/javascripts/components/generated_component.js.jsx'
+    end
   end
 
   test "creates the component file" do

test/generators/es6_component_generator_test.rb

@@ -6,8 +6,14 @@ class Es6ComponentGeneratorTest < Rails::Generators::TestCase
   setup :prepare_destination
   tests React::Generators::ComponentGenerator
 
-  def filename
-    'app/assets/javascripts/components/generated_component.es6.jsx'
+  if WebpackerHelpers.available?
+    def filename
+      "app/javascript/components/generated_component.js"
+    end
+  else
+    def filename
+      'app/assets/javascripts/components/generated_component.es6.jsx'
+    end
   end
 
   def class_name

test/support/webpacker_helpers.rb

@@ -1,8 +1,11 @@
 module WebpackerHelpers
   module_function
+  def available?
+    defined?(Webpacker)
+  end
 
   def when_webpacker_available
-    if defined?(Webpacker)
+    if available?
       clear_webpacker_packs
       Dir.chdir("./test/dummy") do
         Rake::Task['webpacker:compile'].invoke