From 090c8a87b31de5c5a21eebc55adb22ddab625015 Mon Sep 17 00:00:00 2001
From: William Boman <william@redwill.se>
Date: Wed, 27 Apr 2022 21:20:45 +0200
Subject: feat: integrate with lspconfig's on_setup hook (#631)

* feat: integrate with lspconfig's on_setup hook

* fix!: don't use aliased installation directories if new .setup() fn is used

This makes it so servers are always installed in a directory name that
corresponds with the server name. The reason aliased installation directories
is supported is lost on me, but it's legacy and complicates things
unnecessarily.

This is a breaking change for users who previously were using the `.on_server_ready()`
hook, and now transitioned to setting up servers directly via lspconfig.
These users will need to reinstall the server.

* fix: block usage of the deprecated server:setup() method if new setup method is used

* fix: allow passing no arg to setup()

* docs: ok final.v3 readme
---
 tests/middleware_spec.lua | 39 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)
 create mode 100644 tests/middleware_spec.lua

(limited to 'tests/middleware_spec.lua')

diff --git a/tests/middleware_spec.lua b/tests/middleware_spec.lua
new file mode 100644
index 00000000..5c629ee6
--- /dev/null
+++ b/tests/middleware_spec.lua
@@ -0,0 +1,39 @@
+local util = require "lspconfig.util"
+local servers = require "nvim-lsp-installer.servers"
+local middleware = require "nvim-lsp-installer.middleware"
+
+describe("middleware", function()
+    it("should register on_setup hook with lspconfig", function()
+        -- 1. setup dummy server
+        local default_options = {
+            cmd = { "dummy-lsp" },
+            cmd_env = { PATH = "/keep/my/path/out/your/f/mouth" },
+        }
+        local server = ServerGenerator {
+            name = "dummy",
+            default_options = default_options,
+        }
+        servers.register(server)
+
+        -- 2. register hook
+        middleware.register_lspconfig_hook()
+
+        -- 3. call lspconfig hook
+        local config = {
+            name = "dummy",
+            cmd = { "should", "be", "overwritten" },
+            custom = "setting",
+            cmd_env = { SOME_DEFAULT_ENV = "important" },
+        }
+        util.on_setup(config)
+        assert.are.same({
+            cmd = { "dummy-lsp" },
+            name = "dummy",
+            custom = "setting",
+            cmd_env = {
+                PATH = "/keep/my/path/out/your/f/mouth",
+                SOME_DEFAULT_ENV = "important",
+            },
+        }, config)
+    end)
+end)
-- 
cgit v1.2.3-70-g09d2