Fix slurp-action-test against newer LSP4IJ versions

The test was failing in CI because newer LSP4IJ versions broke the
test setup in two independent ways. Both are addressed here.

1. ClojureLanguageServerFactory throws UnsupportedOperationException
   on plugin instantiation:

       Caused by: java.lang.UnsupportedOperationException:
         createLanguageServerSettingsContributor
         (com.github.clojure-lsp.intellij.extension.language-server-factory/
          ClojureLanguageServerFactory_createLanguageServerSettingsContributor
          not defined?)

   LSP4IJ added `createServerInstaller` and
   `createLanguageServerSettingsContributor` as default interface
   methods. Our factory uses `def-extension` (clj4intellij), which is
   `gen-class`-backed; gen-class always overrides interface methods,
   never falling through to the Java default. Adding explicit stubs
   that return `nil` restores the upstream-default behaviour.

2. Once the factory loads, the LSP server still fails to start:

       Caused by: java.lang.UnsupportedOperationException:
         Failed to map temp:///src
         (filesystem com.intellij.openapi.vfs.ex.temp.TempFileSystem)
         into nio Path
         at LanguageServerWrapper.<init>
         <- FileSystemWatcherManager.getProjectBasePath
         <- VirtualFile.toNioPath

   LSP4IJ's `FileSystemWatcherManager` requires the project base to
   resolve to a real filesystem `Path`. The previous test used
   `clj4intellij.test/setup`, which builds a `LightProjectDescriptor`
   exposing the project under the in-memory `TempFileSystem` — for
   which `toNioPath` throws.

   Bump `clj4intellij` to 0.9.0 (which adds `setup-heavy`) and switch
   `test_utils/setup-test-project` to it. The heavy fixture uses
   `IdeaTestFixtureFactory.createFixtureBuilder` +
   `createTempDirTestFixture`, giving the project a real disk-backed
   path. Also rename the test project to `clojure-sample-project`
   (heavy fixtures use the name as the on-disk directory and IntelliJ
   sanitizes dots to underscores).

🤖 Generated with [eca](https://eca.dev)

Co-Authored-By: eca-agent <git@eca.dev>

Author: ericdallo

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## [Unreleased]
 
+- Implement `createServerInstaller` and `createLanguageServerSettingsContributor` on `ClojureLanguageServerFactory`. Newer LSP4IJ versions added these as default interface methods, but our `def-extension`/`gen-class`-backed factory always overrides interface methods, so explicit stubs are required.
+- Fix `slurp-action-test` against newer LSP4IJ versions. Bump `clj4intellij` to `0.9.0` and switch the test fixture to `setup-heavy` so the project base has a real filesystem path (LSP4IJ's `FileSystemWatcherManager` calls `VirtualFile.toNioPath()`, which throws on the in-memory `TempFileSystem` used by light fixtures).
+
 ## 3.5.2
 
 - Fix `QuoteHandler` compile error by merging `BAD_CHARACTER` into the quote-handler `TokenSet`.

build.gradle.kts

@@ -35,7 +35,7 @@ repositories {
 
 dependencies {
     implementation ("org.clojure:clojure:1.12.0")
-    implementation ("com.github.ericdallo:clj4intellij:0.8.0")
+    implementation ("com.github.ericdallo:clj4intellij:0.9.0")
     implementation ("seesaw:seesaw:1.5.0")
     implementation ("camel-snake-kebab:camel-snake-kebab:0.4.3")
     implementation ("com.rpl:proxy-plus:0.0.9")

src/main/clojure/com/github/clojure_lsp/intellij/extension/language_server_factory.clj

@@ -85,6 +85,13 @@
 
   (getServerInterface [_] com.github.clojure_lsp.intellij.ClojureLanguageServer)
 
+  ;; Newer LSP4IJ versions added these as default methods on
+  ;; LanguageServerFactory. `def-extension` is gen-class-backed and always
+  ;; overrides interface methods (no fall-through to defaults), so we must
+  ;; declare explicit stubs that return nil — matching the upstream defaults.
+  (createServerInstaller [_] nil)
+  (createLanguageServerSettingsContributor [_] nil)
+
   (createClientFeatures [_]
     (doto
      (proxy+ [] LSPClientFeatures

src/test/clojure/com/github/clojure_lsp/intellij/slurp_action_test.clj

@@ -22,7 +22,10 @@
    
    The test ensures that the Forward Slurp action correctly modifies the code structure
    by moving the closing parenthesis forward."
-  (let [project-name "clojure.sample-project"
+  ;; Avoid dots/underscores in the project name: heavy fixtures use it as
+  ;; the on-disk directory name and IntelliJ sanitizes it to a filesystem-safe
+  ;; identifier.
+  (let [project-name "clojure-sample-project"
         {:keys [fixtures project deps-file]} (test-utils/setup-test-project project-name)
         clj-file (.copyFileToProject ^CodeInsightTestFixture fixtures "foo.clj")]
     (is (= project-name (.getName ^Project project)))

src/test/clojure/com/github/clojure_lsp/intellij/test_utils.clj

@@ -75,7 +75,12 @@
   ([project-name]
    (setup-test-project project-name "{}"))
   ([project-name deps-content]
-   (let [fixtures (clj4intellij.test/setup project-name)
+   ;; setup-heavy gives us a disk-backed CodeInsightTestFixture, which is
+   ;; required by recent LSP4IJ versions: their FileSystemWatcherManager
+   ;; calls VirtualFile.toNioPath() on the project base, and that throws
+   ;; UnsupportedOperationException for the in-memory TempFileSystem used
+   ;; by light fixtures.
+   (let [fixtures (clj4intellij.test/setup-heavy project-name)
          deps-file (.createFile fixtures "deps.edn" deps-content)
          _ (.setTestDataPath fixtures "testdata")
          project (.getProject fixtures)]