From 78275321f813c30a6add74ccffb3797caed7d07c Mon Sep 17 00:00:00 2001
From: Matt Sturgeon <matt@sturgeon.me.uk>
Date: Sun, 30 Jun 2024 00:12:19 +0100
Subject: [PATCH] tests/test-derivation: don't invoke test modules

`def.module or lib.removeAttrs def [ "tests" ]` is actually equivalent to
`(def.module or lib.removeAttrs) def [ "tests" ]`.

This meant whenever `def` had a `module` attribute, it was invoked as
`def.module def [ "tests" ]`!
---
 tests/default.nix | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/default.nix b/tests/default.nix
index 2bbdd371..94d19cab 100644
--- a/tests/default.nix
+++ b/tests/default.nix
@@ -37,7 +37,7 @@ let
       inherit name;
       # The module can either be the actual definition,
       # or a child attr named `module`.
-      module = def.module or lib.removeAttrs def [ "tests" ];
+      module = def.module or (lib.removeAttrs def [ "tests" ]);
       dontRun = def.tests.dontRun or false;
       pkgs = pkgsUnfree;
     };