Merge pull request #38269 from jorgemanrubia/cache-view-templates-in-…

…tests

Cache compiled view templates when running tests by default

railties/CHANGELOG.md

@@ -1,3 +1,16 @@
+*   Cache compiled view templates when running tests by default
+
+    When generating a new app without `--skip-spring`, caching classes is
+    disabled in `environments/test.rb`. This implicitly disables caching
+    view templates too. This change will enable view template caching by
+    adding this to the generated `environments/test.rb`:
+
+    ````ruby
+    config.action_view.cache_template_loading = true
+    ````
+
+    *Jorge Manrubia*
+
 *   `Rails::Application#eager_load!` is available again to load application code
     manually as it was possible in previous versions.
 

railties/lib/rails/generators/rails/app/templates/config/environments/test.rb.tt

@@ -9,6 +9,7 @@ Rails.application.configure do
   <%-# Spring executes the reloaders when files change. %>
   <%- if spring_install? -%>
   config.cache_classes = false
+  config.action_view.cache_template_loading = true
   <%- else -%>
   config.cache_classes = true
   <%- end -%>

railties/test/generators/app_generator_test.rb

@@ -836,6 +836,7 @@ def test_spring
     assert_gem "spring"
     assert_file("config/environments/test.rb") do |contents|
       assert_match("config.cache_classes = false", contents)
+      assert_match("config.action_view.cache_template_loading = true", contents)
     end
   end