summary refs log tree commit diff
path: root/pkgs/test/config.nix
diff options
context:
space:
mode:
authorRobert Hensing2022-05-29 11:50:48 +0200
committerRobert Hensing2022-05-29 11:50:48 +0200
commit28f9043aa92e3d1d127b18d21a7c6984eebcdf93 (patch)
tree15d23a612d6874ac2989e8b1c12bef6d4cb4ec58 /pkgs/test/config.nix
parent1a7c0eac1572f56afa970e5fe6493b85cecf51a5 (diff)
pkgs/tests/config.nix: Make test future proof
Diffstat (limited to 'pkgs/test/config.nix')
-rw-r--r--pkgs/test/config.nix15
1 files changed, 9 insertions, 6 deletions
diff --git a/pkgs/test/config.nix b/pkgs/test/config.nix
index 6047b013206a..734e1aace148 100644
--- a/pkgs/test/config.nix
+++ b/pkgs/test/config.nix
@@ -9,12 +9,15 @@ lib.recurseIntoAttrs {
               tempAllow pkgs.authy "2.1.0" [ "electron-9.4.4" ];
           };
         };
-        # Allow with forgetting
-        tempAllow = p: v: pa:
-          lib.optionals (lib.assertMsg (p.version == v) "${p.name} is no longer at version ${v}, consider removing the tempAllow") pa;
-        # For this test we don't _really_ care about the version though,
-        # only about evaluation strictness
-        tempAllowAlike = p: v: pa: builtins.seq v builtins.seq p.version pa;
+        # A simplification of `tempAllow` that doesn't check the version, but
+        # has the same strictness characteristics. Actually checking a version
+        # here would add undue maintenance.
+        #
+        # Original:
+        #     tempAllow = p: v: pa:
+        #       lib.optionals (lib.assertMsg (p.version == v) "${p.name} is no longer at version ${v}, consider removing the tempAllow") pa;
+        #
+        tempAllow = p: v: pa: builtins.seq v builtins.seq p.version pa;
 
     in pkgs.hello;