From 03a638205b5cb04ba9c2ed6c604e137b15f07fa1 Mon Sep 17 00:00:00 2001 From: Matt Sturgeon Date: Thu, 15 Jan 2026 13:37:14 +0000 Subject: [PATCH] tests/modules/lsp: fix package-fallback assertion logic The "is in" vs "isn't in" logic was accidentally inverted. --- tests/test-sources/modules/lsp.nix | 8 ++++---- tests/test-sources/plugins/lsp/_lsp.nix | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/test-sources/modules/lsp.nix b/tests/test-sources/modules/lsp.nix index b0e839a2..58cbf118 100644 --- a/tests/test-sources/modules/lsp.nix +++ b/tests/test-sources/modules/lsp.nix @@ -196,21 +196,21 @@ let assertPrefix = name: pkg: [ { - assertion = lib.all (x: x == pkg) config.extraPackages; + assertion = lib.elem pkg config.extraPackages; message = "Expected `${name}` to be in extraPackages"; } { - assertion = lib.any (x: x != pkg) config.extraPackagesAfter; + assertion = !(lib.elem pkg config.extraPackagesAfter); message = "Expected `${name}` not to be in extraPackagesAfter"; } ]; assertSuffix = name: pkg: [ { - assertion = lib.all (x: x != pkg) config.extraPackages; + assertion = !(lib.elem pkg config.extraPackages); message = "Expected `${name}` not to be in extraPackages"; } { - assertion = lib.any (x: x == pkg) config.extraPackagesAfter; + assertion = lib.elem pkg config.extraPackagesAfter; message = "Expected `${name}` to be in extraPackagesAfter"; } ]; diff --git a/tests/test-sources/plugins/lsp/_lsp.nix b/tests/test-sources/plugins/lsp/_lsp.nix index d5a725c4..d66e50f7 100644 --- a/tests/test-sources/plugins/lsp/_lsp.nix +++ b/tests/test-sources/plugins/lsp/_lsp.nix @@ -264,11 +264,11 @@ let assertAfter = name: pkg: [ { - assertion = lib.all (x: x != pkg) config.extraPackages; + assertion = !(lib.elem pkg config.extraPackages); message = "Expected `${name}` not to be in extraPackages"; } { - assertion = lib.any (x: x == pkg) config.extraPackagesAfter; + assertion = lib.elem pkg config.extraPackagesAfter; message = "Expected `${name}` to be in extraPackagesAfter"; } ];