Merge pull request #60 from bdewater/less-deps

Only require parts of Rails actually needed

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/config/application.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/config/environments/development.rb

@@ -37,12 +37,12 @@
   end
 
   # Store uploaded files on the local file system (see config/storage.yml for options).
-  config.active_storage.service = :local
+  # config.active_storage.service = :local
 
   # Don't care if the mailer can't send.
-  config.action_mailer.raise_delivery_errors = false
+  # config.action_mailer.raise_delivery_errors = false
 
-  config.action_mailer.perform_caching = false
+  # config.action_mailer.perform_caching = false
 
   # Print deprecation notices to the Rails logger.
   config.active_support.deprecation = :log

test/dummy/config/environments/test.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/ruby_lsp_rails/rack_app_test.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