Merge pull request rails#52571 from chunlea/chunlea/remove-pwa-for-api-only

Remove PWA views and routes if its an API only project

Author: byroot

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

@@ -491,6 +491,7 @@ def delete_app_views_if_api_option
             remove_dir "app/views"
           else
             remove_file "app/views/layouts/application.html.erb"
+            remove_dir  "app/views/pwa"
           end
         end
       end

railties/lib/rails/generators/rails/app/templates/config/routes.rb.tt

@@ -5,10 +5,12 @@ Rails.application.routes.draw do
   # Can be used by load balancers and uptime monitors to verify that the app is live.
   get "up", to: "rails/health#show", as: :rails_health_check
 
+  <%- unless options.api? -%>
   # Render dynamic PWA files from app/views/pwa/*
   get "service-worker", to: "rails/pwa#service_worker", as: :pwa_service_worker
   get "manifest", to: "rails/pwa#manifest", as: :pwa_manifest
 
+  <%- end -%>
   # Defines the root path route ("/")
   # root "posts#index"
 end

railties/test/generators/generators_test_helper.rb

@@ -62,9 +62,12 @@ def with_database_configuration(database_name = "secondary")
 
   def copy_routes
     routes = File.expand_path("../../lib/rails/generators/rails/app/templates/config/routes.rb.tt", __dir__)
+    routes = evaluate_template(routes, {
+      options: ActiveSupport::OrderedOptions.new
+    })
     destination = File.join(destination_root, "config")
     FileUtils.mkdir_p(destination)
-    FileUtils.cp routes, File.join(destination, "routes.rb")
+    File.write File.join(destination, "routes.rb"), routes
   end
 
   def copy_gemfile(*gemfile_entries)