Fix rails#52986

A freshly generated Rails project includes a Rubocop linter violation in this line. This edits the file to stick to the listing rules a new Rails project is shipped with by default.

Author: FrancescoK

actionmailbox/test/test_helper.rb

@@ -6,7 +6,7 @@
 ENV["RAILS_INBOUND_EMAIL_PASSWORD"] = "tbsy84uSV1Kt3ZJZELY2TmShPRs91E3yL4tzf96297vBCkDWgL"
 
 require_relative "../test/dummy/config/environment"
-ActiveRecord::Migrator.migrations_paths = [File.expand_path("../test/dummy/db/migrate", __dir__)]
+ActiveRecord::Migrator.migrations_paths = [ File.expand_path("../test/dummy/db/migrate", __dir__) ]
 require "rails/test_help"
 
 require "webmock/minitest"

actiontext/test/test_helper.rb

@@ -7,7 +7,7 @@
 
 require_relative "../test/dummy/config/environment"
 require "active_record/testing/query_assertions"
-ActiveRecord::Migrator.migrations_paths = [File.expand_path("../test/dummy/db/migrate", __dir__)]
+ActiveRecord::Migrator.migrations_paths = [ File.expand_path("../test/dummy/db/migrate", __dir__) ]
 require "rails/test_help"
 
 require "rails/test_unit/reporter"

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

@@ -109,7 +109,7 @@ Rails.application.configure do
   config.active_record.dump_schema_after_migration = false
 
   # Only use :id for inspections in production.
-  config.active_record.attributes_for_inspect = [:id]
+  config.active_record.attributes_for_inspect = [ :id ]
   <%- end -%>
 
   # Enable DNS rebinding protection and other `Host` header attacks.

railties/lib/rails/generators/rails/plugin/templates/test/test_helper.rb.tt

@@ -3,7 +3,7 @@ ENV["RAILS_ENV"] = "test"
 
 require_relative "<%= File.join("..", options[:dummy_path], "config/environment") -%>"
 <% unless options[:skip_active_record] -%>
-ActiveRecord::Migrator.migrations_paths = [File.expand_path("../<%= options[:dummy_path] -%>/db/migrate", __dir__)]
+ActiveRecord::Migrator.migrations_paths = [ File.expand_path("../<%= options[:dummy_path] -%>/db/migrate", __dir__) ]
 <% if options[:mountable] -%>
 ActiveRecord::Migrator.migrations_paths << File.expand_path("../db/migrate", __dir__)
 <% end -%>
@@ -13,7 +13,7 @@ require "rails/test_help"
 <% unless options[:skip_active_record] -%>
 # Load fixtures from the engine
 if ActiveSupport::TestCase.respond_to?(:fixture_paths=)
-  ActiveSupport::TestCase.fixture_paths = [File.expand_path("fixtures", __dir__)]
+  ActiveSupport::TestCase.fixture_paths = [ File.expand_path("fixtures", __dir__) ]
   ActionDispatch::IntegrationTest.fixture_paths = ActiveSupport::TestCase.fixture_paths
   ActiveSupport::TestCase.file_fixture_path = File.expand_path("fixtures", __dir__) + "/files"
   ActiveSupport::TestCase.fixtures :all

railties/test/generators/shared_generator_tests.rb

@@ -385,6 +385,16 @@ def test_target_rails_prerelease_with_relative_app_path
     assert_file "myproject/Gemfile", %r{^gem ["']rails["'], github: ["']rails/rails["'], branch: ["']main["']$}
   end
 
+  def test_generated_files_have_no_rubocop_warnings
+    run_generator
+
+    Dir.chdir(destination_root) do
+      output = `./bin/rubocop`
+
+      assert_predicate $?, :success?, "bin/rubocop did not exit successfully:\n#{output}"
+    end
+  end
+
   private
     def assert_load_defaults
     end