Only require parts of Rails actually needed

In our app we don't need Action Cable/Text/Mailbox etc and we don't want to pull it in for this gem.

Author: bdewater

Gemfile.lock

@@ -103,7 +103,9 @@ PATH
   remote: .
   specs:
     ruby-lsp-rails (0.2.7)
-      rails (>= 6.0)
+      actionpack (>= 6.0)
+      activerecord (>= 6.0)
+      railties (>= 6.0)
       ruby-lsp (>= 0.12.0, < 0.13.0)
       sorbet-runtime (>= 0.5.9897)
 

bin/rails

@@ -11,5 +11,4 @@ APP_PATH = File.expand_path("../test/dummy/config/application", __dir__)
 ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../Gemfile", __dir__)
 require "bundler/setup" if File.exist?(ENV["BUNDLE_GEMFILE"])
 
-require "rails/all"
 require "rails/engine/commands"

ruby-lsp-rails.gemspec

@@ -21,7 +21,9 @@ Gem::Specification.new do |spec|
     Dir["{app,config,db,lib}/**/*", "LICENSE.txt", "Rakefile", "README.md"]
   end
 
-  spec.add_dependency("rails", ">= 6.0")
+  spec.add_dependency("actionpack", ">= 6.0")
+  spec.add_dependency("activerecord", ">= 6.0")
+  spec.add_dependency("railties", ">= 6.0")
   spec.add_dependency("ruby-lsp", ">= 0.12.0", "< 0.13.0")
   spec.add_dependency("sorbet-runtime", ">= 0.5.9897")
 end

test/dummy/app/channels/application_cable/channel.rb

@@ -3,7 +3,17 @@
 
 require_relative "boot"
 
-require "rails/all"
+require "active_model/railtie"
+# require "active_job/railtie"
+require "active_record/railtie"
+# require "active_storage/engine"
+require "action_controller/railtie"
+# require "action_mailer/railtie"
+# require "action_mailbox/engine"
+# require "action_text/engine"
+require "action_view/railtie"
+# require "action_cable/engine"
+require "rails/test_unit/railtie"
 
 # Require the gems listed in Gemfile, including any gems
 # you've limited to :test, :development, or :production.

test/dummy/app/channels/application_cable/connection.rb

@@ -17,7 +17,7 @@
   # Eager loading loads your whole application. When running a single test locally,
   # this probably isn't necessary. It's a good idea to do in a continuous integration
   # system, or in some way before deploying your code.
-  config.eager_load = ENV["CI"].present?
+  config.eager_load = true
 
   # Configure public file server for tests with Cache-Control for performance.
   config.public_file_server.enabled = true
@@ -36,16 +36,6 @@
   # Disable request forgery protection in test environment.
   config.action_controller.allow_forgery_protection = false
 
-  # Store uploaded files on the local file system in a temporary directory.
-  config.active_storage.service = :test
-
-  config.action_mailer.perform_caching = false
-
-  # Tell Action Mailer not to deliver emails to the real world.
-  # The :test delivery method accumulates sent emails in the
-  # ActionMailer::Base.deliveries array.
-  config.action_mailer.delivery_method = :test
-
   # Print deprecation notices to the stderr.
   config.active_support.deprecation = :stderr
 

test/dummy/app/jobs/application_job.rb

@@ -31,7 +31,7 @@ class RackAppTest < ActionDispatch::IntegrationTest
       end
 
       test "GET show returns not_found if class is not a model" do
-        get "/ruby_lsp_rails/models/ApplicationJob"
+        get "/ruby_lsp_rails/models/Time"
         assert_response(:not_found)
       end