From aa6741135cb7e78ab038eee798067f2df32bb6d9 Mon Sep 17 00:00:00 2001
From: aszlig <aszlig@redmoonstudios.org>
Date: Fri, 11 Dec 2015 07:34:35 +0100
Subject: modules: Make requiresTests a list of paths.

Unfortunately, we went into trouble and the Hydra machine not only
consumed lots of ram during evaluation but it also triggered a bunch of
evaluation errors because the available tests were strictly evaluated.

Using attribute paths consisting of plain strings doesn't have the same
problem, even though they look a bit uglier.

Signed-off-by: aszlig <aszlig@redmoonstudios.org>
---
 modules/profiles/common.nix                 | 14 +++++++-------
 modules/profiles/tests.nix                  |  4 ++--
 modules/user/aszlig/services/i3/default.nix |  4 ++--
 3 files changed, 11 insertions(+), 11 deletions(-)

(limited to 'modules')

diff --git a/modules/profiles/common.nix b/modules/profiles/common.nix
index 58d6c27b..486426e9 100644
--- a/modules/profiles/common.nix
+++ b/modules/profiles/common.nix
@@ -35,11 +35,15 @@ with lib;
     };
 
     requiresTests = mkOption {
-      type = types.listOf types.package;
+      type = types.listOf (types.listOf types.str);
       default = [];
+      example = [ ["nixos" "nat" "firewall"] ["vuizvui" "foo"] ];
       description = ''
-        A list of derivations which have to succeed in order to trigger a
-        channel update for the current configuration/machine.
+        A list of attribute paths to the tests which need to succeed in order to
+        trigger a channel update for the current configuration/machine.
+
+        Every attribute path itself is a list of attribute names, which are
+        queried using <function>lib.getAttrFromPath</function>.
       '';
     };
   };
@@ -93,9 +97,5 @@ with lib;
         rootChannelsPath
       ] ++ optional config.vuizvui.enableGlobalNixpkgsConfig nixpkgsConfig;
     in mkIf config.vuizvui.modifyNixPath (mkOverride 90 nixPath);
-
-    _module.args.tests = import ../../lib/get-tests.nix {
-      inherit nixpkgs system;
-    };
   };
 }
diff --git a/modules/profiles/tests.nix b/modules/profiles/tests.nix
index 66ea76e5..811d6cf8 100644
--- a/modules/profiles/tests.nix
+++ b/modules/profiles/tests.nix
@@ -1,10 +1,10 @@
-{ config, lib, tests, ... }:
+{ config, lib, ... }:
 
 with lib;
 
 let
   which = if config.networking.useNetworkd then "networkd" else "scripted";
-  networkTests = map (attr: getAttr attr tests.nixos.networking.${which}) [
+  networkTests = map (name: ["nixos" "networking" which]) [
     "static" "dhcpSimple" "dhcpOneIf" "bond" "bridge" "macvlan" "sit" "vlan"
   ];
 
diff --git a/modules/user/aszlig/services/i3/default.nix b/modules/user/aszlig/services/i3/default.nix
index b9025766..0814ac1e 100644
--- a/modules/user/aszlig/services/i3/default.nix
+++ b/modules/user/aszlig/services/i3/default.nix
@@ -1,4 +1,4 @@
-{ pkgs, tests, lib, config, ... }:
+{ pkgs, lib, config, ... }:
 
 with lib;
 
@@ -106,7 +106,7 @@ in
   config = mkIf cfg.enable {
     vuizvui.user.aszlig.services.i3.workspaces = defaultWorkspaces;
 
-    vuizvui.requiresTests = [ tests.vuizvui.aszlig.i3 ];
+    vuizvui.requiresTests = [ ["vuizvui" "aszlig" "i3"] ];
 
     services.xserver.windowManager = {
       default = "i3";
-- 
cgit 1.4.1