apply code review

Author: afucher

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

@@ -23,16 +23,17 @@
    The test ensures that the Forward Slurp action correctly modifies the code structure
    by moving the closing parenthesis forward."
   (let [project-name "clojure.core"
-        {:keys [fixture project deps-file]} (test-utils/setup-test-project project-name)
-        clj-file (.copyFileToProject ^CodeInsightTestFixture fixture "foo.clj")]
+        {: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)))
     (is deps-file)
 
-    (let [editor (test-utils/open-file-in-editor fixture clj-file)]
+    (let [editor (test-utils/open-file-in-editor fixtures clj-file)]
       (test-utils/setup-lsp-server project)
       (editor/move-caret-to-position editor 2 8)
       (test-utils/run-editor-action "ClojureLSP.ForwardSlurp" project)
       (clj4intellij.test/dispatch-all)
-      (println (test-utils/get-editor-text fixture))
-      (.checkResultByFile ^CodeInsightTestFixture fixture "foo_expected.clj")
+
+      (.checkResultByFile ^CodeInsightTestFixture fixtures "foo_expected.clj")
+
       (test-utils/teardown-test-project project))))

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

@@ -32,7 +32,6 @@
   [action-id project]
   (let [action (.getAction (ActionManager/getInstance) action-id)
         context (.getDataContext (DataManager/getInstance))]
-    (println "Running action:" action-id)
     (app-manager/write-command-action
      project
      (fn []
@@ -76,18 +75,18 @@
   ([project-name]
    (setup-test-project project-name "{}"))
   ([project-name deps-content]
-   (let [fixture (clj4intellij.test/setup project-name)
-         deps-file (.createFile fixture "deps.edn" deps-content)
-         _ (.setTestDataPath fixture "testdata")
-         project (.getProject fixture)]
-     {:fixture fixture
+   (let [fixtures (clj4intellij.test/setup project-name)
+         deps-file (.createFile fixtures "deps.edn" deps-content)
+         _ (.setTestDataPath fixtures "testdata")
+         project (.getProject fixtures)]
+     {:fixtures fixtures
       :project project
       :deps-file deps-file})))
 
 (defn setup-lsp-server
   "Sets up and waits for the LSP server to be ready."
   [project]
-  (let [my-settings (ServiceManager/getService SettingsState)]
+  (let [my-settings ^SettingsState (ServiceManager/getService SettingsState)]
     (.loadState my-settings my-settings)
     (clj4intellij.test/dispatch-all)
     (wait-lsp-start {:project project})))