From 1e23f183fa0a6ea4000d8d9e4a849a997d45420e Mon Sep 17 00:00:00 2001
From: Gaetan Lepage <gaetan@glepage.com>
Date: Thu, 4 Jan 2024 23:29:41 +0100
Subject: [PATCH] tests/lsp: disable "all-servers" test for darwin because of
 sandbox limitations

---
 tests/test-sources/plugins/lsp/_lsp.nix | 161 ++++++++++++------------
 1 file changed, 83 insertions(+), 78 deletions(-)

diff --git a/tests/test-sources/plugins/lsp/_lsp.nix b/tests/test-sources/plugins/lsp/_lsp.nix
index f19aeb3fe6..df1975192a 100644
--- a/tests/test-sources/plugins/lsp/_lsp.nix
+++ b/tests/test-sources/plugins/lsp/_lsp.nix
@@ -70,87 +70,92 @@
     };
   };
 
-  all-servers = {
-    plugins.lsp = {
-      enable = true;
+  all-servers =
+    pkgs.lib.optionalAttrs
+    # This fails on darwin
+    # See https://github.com/NixOS/nix/issues/4119
+    (!pkgs.stdenv.isDarwin)
+    {
+      plugins.lsp = {
+        enable = true;
 
-      servers = {
-        ansiblels.enable = true;
-        astro.enable = true;
-        bashls.enable = true;
-        biome.enable = true;
-        ccls.enable = true;
-        clangd.enable = true;
-        clojure-lsp.enable = true;
-        cmake.enable = true;
-        # pkgs.csharp-ls only supports linux platforms
-        csharp-ls.enable = pkgs.stdenv.isLinux;
-        cssls.enable = true;
-        dartls.enable = true;
-        denols.enable = true;
-        dhall-lsp-server.enable = true;
-        digestif.enable = true;
-        dockerls.enable = true;
-        efm.enable = true;
-        elmls.enable = true;
-        emmet_ls.enable = true;
-        eslint.enable = true;
-        elixirls.enable = true;
-        # pkgs.fsautocomplete only supports linux platforms
-        fsautocomplete.enable = pkgs.stdenv.isLinux;
-        futhark-lsp.enable = true;
-        gleam.enable = true;
-        gopls.enable = true;
-        graphql.enable = true;
-        hls.enable = true;
-        html.enable = true;
-        java-language-server.enable = true;
-        jsonls.enable = true;
-        julials.enable = true;
-        kotlin-language-server.enable = true;
-        leanls.enable = true;
-        ltex.enable = true;
-        lua-ls.enable = true;
-        marksman.enable = true;
-        metals.enable = true;
-        nil_ls.enable = true;
-        nixd.enable = true;
-        ols.enable =
-          # ols is not supported on aarch64-linux
-          (pkgs.stdenv.hostPlatform.system != "aarch64-linux")
-          # As of 2024-01-04, ols is broken on darwin
+        servers = {
+          ansiblels.enable = true;
+          astro.enable = true;
+          bashls.enable = true;
+          biome.enable = true;
+          ccls.enable = true;
+          clangd.enable = true;
+          clojure-lsp.enable = true;
+          cmake.enable = true;
+          # pkgs.csharp-ls only supports linux platforms
+          csharp-ls.enable = pkgs.stdenv.isLinux;
+          cssls.enable = true;
+          dartls.enable = true;
+          denols.enable = true;
+          dhall-lsp-server.enable = true;
+          digestif.enable = true;
+          dockerls.enable = true;
+          efm.enable = true;
+          elmls.enable = true;
+          emmet_ls.enable = true;
+          eslint.enable = true;
+          elixirls.enable = true;
+          # pkgs.fsautocomplete only supports linux platforms
+          fsautocomplete.enable = pkgs.stdenv.isLinux;
+          futhark-lsp.enable = true;
+          gleam.enable = true;
+          gopls.enable = true;
+          graphql.enable = true;
+          hls.enable = true;
+          html.enable = true;
+          java-language-server.enable = true;
+          jsonls.enable = true;
+          julials.enable = true;
+          kotlin-language-server.enable = true;
+          leanls.enable = true;
+          ltex.enable = true;
+          lua-ls.enable = true;
+          marksman.enable = true;
+          metals.enable = true;
+          nil_ls.enable = true;
+          nixd.enable = true;
+          ols.enable =
+            # ols is not supported on aarch64-linux
+            (pkgs.stdenv.hostPlatform.system != "aarch64-linux")
+            # As of 2024-01-04, ols is broken on darwin
+            # TODO: re-enable this test when fixed
+            && !pkgs.stdenv.isDarwin;
+          omnisharp.enable = true;
+          perlpls.enable = true;
+          pest_ls.enable = true;
+          prismals.enable = true;
+          pylsp.enable = true;
+          pylyzer.enable = true;
+          pyright.enable = true;
+          rnix-lsp.enable = true;
+          ruff-lsp.enable = true;
+          rust-analyzer = {
+            enable = true;
+            installCargo = true;
+            installRustc = true;
+          };
+          solargraph.enable = true;
+          # As of 2023-12-13, sourcekit-lsp is broken
           # TODO: re-enable this test when fixed
-          && !pkgs.stdenv.isDarwin;
-        omnisharp.enable = true;
-        perlpls.enable = true;
-        pest_ls.enable = true;
-        prismals.enable = true;
-        pylsp.enable = true;
-        pylyzer.enable = true;
-        pyright.enable = true;
-        rnix-lsp.enable = true;
-        ruff-lsp.enable = true;
-        rust-analyzer = {
-          enable = true;
-          installCargo = true;
-          installRustc = true;
+          sourcekit.enable = false;
+          svelte.enable = true;
+          tailwindcss.enable = true;
+          taplo.enable = true;
+          terraformls.enable = true;
+          texlab.enable = true;
+          tsserver.enable = true;
+          typst-lsp.enable = true;
+          vls.enable = true;
+          vuels.enable = true;
+          yamlls.enable = true;
+          zls.enable = true;
         };
-        solargraph.enable = true;
-        # As of 2023-12-13, sourcekit-lsp is broken
-        # TODO: re-enable this test when fixed
-        sourcekit.enable = false;
-        svelte.enable = true;
-        tailwindcss.enable = true;
-        taplo.enable = true;
-        terraformls.enable = true;
-        texlab.enable = true;
-        tsserver.enable = true;
-        typst-lsp.enable = true;
-        vls.enable = true;
-        vuels.enable = true;
-        yamlls.enable = true;
-        zls.enable = true;
       };
     };
-  };
 }