Merge pull request rails#52403 from y-yagi/app_update_public_api

Don't update public directory during `app:update` command for API-only Applications

Author: rafaelfranca

railties/lib/rails/generators/rails/app/app_generator.rb

@@ -223,6 +223,8 @@ def log
     end
 
     def public_directory
+      return if options[:update] && options[:api]
+
       directory "public", "public", recursive: false
     end
 

railties/test/generators/api_app_generator_test.rb

@@ -125,6 +125,13 @@ def test_app_update_does_not_generate_unnecessary_bin_files
     pass
   end
 
+  def test_app_update_does_not_generate_public_files
+    run_generator
+    run_app_update
+
+    assert_no_file "public/406-unsupported-browser.html"
+  end
+
   private
     def default_files
       %w(.gitignore

railties/test/generators/app_generator_test.rb

@@ -1511,16 +1511,6 @@ def assert_no_node_files
       end
     end
 
-    def run_app_update(app_root = destination_root, flags: "--force")
-      Dir.chdir(app_root) do
-        gemfile_contents = File.read("Gemfile")
-        gemfile_contents.sub!(/^(gem "rails").*/, "\\1, path: #{File.expand_path("../../..", __dir__).inspect}")
-        File.write("Gemfile", gemfile_contents)
-
-        quietly { system({ "BUNDLE_GEMFILE" => "Gemfile" }, "bin/rails app:update #{flags}", exception: true) }
-      end
-    end
-
     def action(*args, &block)
       capture(:stdout) { generator.send(*args, &block) }
     end

railties/test/generators/generators_test_helper.rb

@@ -131,6 +131,16 @@ def assert_devcontainer_json_file
     end
   end
 
+  def run_app_update(app_root = destination_root, flags: "--force")
+    Dir.chdir(app_root) do
+      gemfile_contents = File.read("Gemfile")
+      gemfile_contents.sub!(/^(gem "rails").*/, "\\1, path: #{File.expand_path("../../..", __dir__).inspect}")
+      File.write("Gemfile", gemfile_contents)
+
+      quietly { system({ "BUNDLE_GEMFILE" => "Gemfile" }, "bin/rails app:update #{flags}", exception: true) }
+    end
+  end
+
   private
     def gemfile_locals
       {