Skip to content

Commit 03a6382

Browse files
MattSturgeonGaetanLepage
authored andcommitted
tests/modules/lsp: fix package-fallback assertion logic
The "is in" vs "isn't in" logic was accidentally inverted.
1 parent 7eb8f36 commit 03a6382

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

tests/test-sources/modules/lsp.nix

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -196,21 +196,21 @@
196196
let
197197
assertPrefix = name: pkg: [
198198
{
199-
assertion = lib.all (x: x == pkg) config.extraPackages;
199+
assertion = lib.elem pkg config.extraPackages;
200200
message = "Expected `${name}` to be in extraPackages";
201201
}
202202
{
203-
assertion = lib.any (x: x != pkg) config.extraPackagesAfter;
203+
assertion = !(lib.elem pkg config.extraPackagesAfter);
204204
message = "Expected `${name}` not to be in extraPackagesAfter";
205205
}
206206
];
207207
assertSuffix = name: pkg: [
208208
{
209-
assertion = lib.all (x: x != pkg) config.extraPackages;
209+
assertion = !(lib.elem pkg config.extraPackages);
210210
message = "Expected `${name}` not to be in extraPackages";
211211
}
212212
{
213-
assertion = lib.any (x: x == pkg) config.extraPackagesAfter;
213+
assertion = lib.elem pkg config.extraPackagesAfter;
214214
message = "Expected `${name}` to be in extraPackagesAfter";
215215
}
216216
];

tests/test-sources/plugins/lsp/_lsp.nix

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -264,11 +264,11 @@
264264
let
265265
assertAfter = name: pkg: [
266266
{
267-
assertion = lib.all (x: x != pkg) config.extraPackages;
267+
assertion = !(lib.elem pkg config.extraPackages);
268268
message = "Expected `${name}` not to be in extraPackages";
269269
}
270270
{
271-
assertion = lib.any (x: x == pkg) config.extraPackagesAfter;
271+
assertion = lib.elem pkg config.extraPackagesAfter;
272272
message = "Expected `${name}` to be in extraPackagesAfter";
273273
}
274274
];

0 commit comments

Comments
 (0)