Merge pull request rails#20724 from y-yagi/fix_scaffold_in_full_engine

fix NoMethodError that occurs when generating scaffold inside full mode engine

Author: senny

railties/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Fix `NoMethodError` when generating a scaffold inside a full engine.
+
+    *Yuji Yaginuma*
+
 *   Adding support for passing a block to the `add_source` action of a custom generator
 
     *Mike Dalton*, *Hirofumi Wakasugi*

railties/lib/rails/generators/named_base.rb

@@ -183,6 +183,10 @@ def pluralize_table_names?
           !defined?(ActiveRecord::Base) || ActiveRecord::Base.pluralize_table_names
         end
 
+        def mountable_engine?
+          defined?(ENGINE_ROOT) && namespaced?
+        end
+
         # Add a class collisions name to be checked on class initialization. You
         # can supply a hash with a :prefix or :suffix to be tested.
         #

railties/lib/rails/generators/test_unit/controller/templates/functional_test.rb

@@ -2,7 +2,7 @@
 
 <% module_namespacing do -%>
 class <%= class_name %>ControllerTest < ActionController::TestCase
-<% if defined?(ENGINE_ROOT) -%>
+<% if mountable_engine? -%>
   setup do
     @routes = Engine.routes
   end

railties/lib/rails/generators/test_unit/scaffold/scaffold_generator.rb

@@ -21,7 +21,7 @@ def create_test_files
 
       def fixture_name
         @fixture_name ||=
-          if defined?(ENGINE_ROOT)
+          if mountable_engine?
             "%s_%s" % [namespaced_path, table_name]
           else
             table_name

railties/lib/rails/generators/test_unit/scaffold/templates/functional_test.rb

@@ -4,7 +4,7 @@
 class <%= controller_class_name %>ControllerTest < ActionController::TestCase
   setup do
     @<%= singular_table_name %> = <%= fixture_name %>(:one)
-<% if defined?(ENGINE_ROOT) -%>
+<% if mountable_engine? -%>
     @routes = Engine.routes
 <% end -%>
   end

railties/test/generators/scaffold_controller_generator_test.rb

@@ -186,6 +186,17 @@ def test_controller_tests_pass_by_default_inside_mountable_engine
     end
   end
 
+  def test_controller_tests_pass_by_default_inside_full_engine
+    Dir.chdir(destination_root) { `bundle exec rails plugin new bukkits --full` }
+
+    engine_path = File.join(destination_root, "bukkits")
+
+    Dir.chdir(engine_path) do
+      quietly { `bin/rails g controller dashboard foo` }
+      assert_match(/2 runs, 2 assertions, 0 failures, 0 errors/, `bundle exec rake test 2>&1`)
+    end
+  end
+
   def test_api_only_generates_a_proper_api_controller
     run_generator ["User", "--api"]
 

railties/test/generators/scaffold_generator_test.rb

@@ -491,4 +491,18 @@ def test_scaffold_tests_pass_by_default_inside_mountable_engine
       assert_match(/8 runs, 13 assertions, 0 failures, 0 errors/, `bundle exec rake test 2>&1`)
     end
   end
+
+  def test_scaffold_tests_pass_by_default_inside_full_engine
+    Dir.chdir(destination_root) { `bundle exec rails plugin new bukkits --full` }
+
+    engine_path = File.join(destination_root, "bukkits")
+
+    Dir.chdir(engine_path) do
+      quietly do
+        `bin/rails g scaffold User name:string age:integer;
+        bundle exec rake db:migrate`
+      end
+      assert_match(/8 runs, 13 assertions, 0 failures, 0 errors/, `bundle exec rake test 2>&1`)
+    end
+  end
 end