Move LSP tests to kernel

Author: kubukoz

CONTRIBUTING.md

@@ -28,7 +28,9 @@ You'll find these in [`modules/lsp/src/test`](modules/lsp/src/test), and they ru
 
 This doesn't exercise the standard I/O interface of the server, but it's still a decent way to test the composition of all the modules with a filesystem directory ([`modules/lsp/src/test/resources/test-workspaces`](modules/lsp/src/test/resources/test-workspaces)).
 
-To run integration tests, run `lsp/test`.
+These are working without any external LSP library like lsp4j or Langoustine.
+
+To run integration tests, run `lsp-kernel/test`.
 
 ## Extension integration tests
 

build.sbt

@@ -208,7 +208,14 @@ lazy val lspKernel = module("lsp-kernel")
       ("io.get-coursier" % "coursier_2.13" % "2.1.24")
         .exclude("org.scala-lang.modules", "scala-collection-compat_2.13"),
       "org.typelevel" %% "cats-tagless-core" % "0.16.3",
-    ).pipe(jsoniterFix)
+      "org.http4s" %% "http4s-ember-server" % "0.23.30" % Test,
+    ).pipe(jsoniterFix),
+    (Test / test) := {
+      (pluginCore / publishLocal).value
+      (pluginSample / publishLocal).value
+
+      (Test / test).value
+    },
   )
   .enablePlugins(BuildInfoPlugin)
   .settings(
@@ -220,17 +227,10 @@ lazy val lspKernel = module("lsp-kernel")
 lazy val lsp = module("lsp")
   .settings(
     libraryDependencies ++= Seq(
-      "org.eclipse.lsp4j" % "org.eclipse.lsp4j" % "0.24.0",
-      "org.http4s" %% "http4s-ember-server" % "0.23.30" % Test,
-    ),
-    (Test / test) := {
-      (pluginCore / publishLocal).value
-      (pluginSample / publishLocal).value
-
-      (Test / test).value
-    },
+      "org.eclipse.lsp4j" % "org.eclipse.lsp4j" % "0.24.0"
+    )
   )
-  .dependsOn(lspKernel % "test->test;compile->compile")
+  .dependsOn(lspKernel)
 
 lazy val e2e = module("e2e")
   .enablePlugins(BuildInfoPlugin)

modules/lsp/src/main/scala/playground/lsp/LanguageServer.scala modules/lsp-kernel/src/main/scala/playground/lsp/LanguageServer.scala

@@ -2,7 +2,6 @@ package playground.lsp
 
 import cats.effect.IO
 import cats.effect.kernel.Resource
-import org.eclipse.lsp4j.DidChangeWatchedFilesParams
 import playground.lsp.harness.LanguageServerIntegrationTests
 import playground.lsp.harness.TestClient
 import weaver.*

modules/lsp/src/main/scala/playground/lsp/MainServer.scala modules/lsp-kernel/src/main/scala/playground/lsp/MainServer.scala

@@ -3,7 +3,6 @@ package playground.lsp
 import cats.effect.IO
 import cats.effect.Resource
 import com.comcast.ip4s.*
-import org.eclipse.lsp4j
 import org.http4s.HttpRoutes
 import org.http4s.ember.server.EmberServerBuilder
 import org.http4s.implicits.*

modules/lsp/src/main/scala/playground/lsp/ServerBuilder.scala modules/lsp-kernel/src/main/scala/playground/lsp/ServerBuilder.scala

@@ -4,11 +4,6 @@ import cats.effect.IO
 import cats.syntax.all.*
 import fs2.io.file.Files
 import fs2.io.file.Path
-import org.eclipse.lsp4j.CodeLensParams
-import org.eclipse.lsp4j.DidChangeWatchedFilesParams
-import org.eclipse.lsp4j.DocumentDiagnosticParams
-import org.eclipse.lsp4j.MessageType
-import org.eclipse.lsp4j.TextDocumentIdentifier
 import playground.PlaygroundConfig
 import playground.language.Uri
 import playground.lsp.harness.LanguageServerIntegrationTests

modules/lsp/src/main/scala/playground/lsp/ServerLoader.scala modules/lsp-kernel/src/main/scala/playground/lsp/ServerLoader.scala

@@ -4,8 +4,6 @@ import cats.effect.IO
 import cats.effect.IOLocal
 import cats.effect.Resource
 import cats.syntax.all.*
-import org.eclipse.lsp4j.InitializeParams
-import org.eclipse.lsp4j.WorkspaceFolder
 import playground.language.Uri
 import playground.lsp.LanguageServer
 import playground.lsp.MainServer
@@ -58,11 +56,10 @@ trait LanguageServerIntegrationTests {
     .getEvents
     .flatMap { events =>
       val initLogs = List(
-        TestClient
-          .MessageLog(
-            MessageType.Info,
-            s"Hello from Smithy Playground v${BuildInfo.version}! Loading project...",
-          ),
+        TestClient.MessageLog(
+          MessageType.Info,
+          s"Hello from Smithy Playground v${BuildInfo.version}! Loading project...",
+        ),
         TestClient.MessageLog(
           MessageType.Info,
           "Loaded Smithy Playground server with 2 source entries, 0 imports, 2 dependencies and 0 plugins",