Move tests into server_test.rb

Author: andyw8

test/ruby_lsp_rails/runner_client_test.rb

@@ -16,6 +16,7 @@ class RunnerClientTest < ActiveSupport::TestCase
         assert_predicate @client, :stopped?
       end
 
+      # This is an integration test which starts the server. For the more fine-grained tests, see `server_test.rb`.
       test "#model returns information for the requested model" do
         # These columns are from the schema in the dummy app: test/dummy/db/schema.rb
         columns = [
@@ -30,18 +31,6 @@ class RunnerClientTest < ActiveSupport::TestCase
         assert_equal(columns, response.fetch(:columns))
         assert_match(%r{db/schema\.rb$}, response.fetch(:schema_file))
       end
-
-      test "returns nil if model doesn't exist" do
-        assert_nil @client.model("Foo")
-      end
-
-      test "returns nil if class is not a model" do
-        assert_nil @client.model("Time")
-      end
-
-      test "returns nil if class is an abstract model" do
-        assert_nil @client.model("ApplicationRecord")
-      end
     end
   end
 end

test/ruby_lsp_rails/server_test.rb

@@ -5,6 +5,20 @@
 require "ruby_lsp/ruby_lsp_rails/server"
 
 class ServerTest < ActiveSupport::TestCase
+  test "returns nil if model doesn't exist" do
+    response = RubyLsp::Rails::Server.new.resolve_database_info_from_model("Foo")
+    assert_nil(response.fetch(:result))
+  end
+
+  test "returns nil if class is not a model" do
+    response = RubyLsp::Rails::Server.new.resolve_database_info_from_model("Time")
+    assert_nil(response.fetch(:result))
+  end
+
+  test "returns nil if class is an abstract model" do
+    response = RubyLsp::Rails::Server.new.resolve_database_info_from_model("ApplicationRecord")
+    assert_nil(response.fetch(:result))
+  end
   test "handles older Rails version which don't have `schema_dump_path`" do
     ActiveRecord::Tasks::DatabaseTasks.send(:alias_method, :old_schema_dump_path, :schema_dump_path)
     ActiveRecord::Tasks::DatabaseTasks.undef_method(:schema_dump_path)