Skip to content

Commit 8ed2c5f

Browse files
committed
nixos/tests/firefox: migrate to runTest
Part of #386873
1 parent 6af4687 commit 8ed2c5f

File tree

2 files changed

+35
-11
lines changed

2 files changed

+35
-11
lines changed

nixos/tests/all-tests.nix

+21-5
Original file line numberDiff line numberDiff line change
@@ -387,11 +387,27 @@ in {
387387
filesystems-overlayfs = runTest ./filesystems-overlayfs.nix;
388388
firefly-iii = handleTest ./firefly-iii.nix {};
389389
firefly-iii-data-importer = handleTest ./firefly-iii-data-importer.nix {};
390-
firefox = handleTest ./firefox.nix { firefoxPackage = pkgs.firefox; };
391-
firefox-beta = handleTest ./firefox.nix { firefoxPackage = pkgs.firefox-beta; };
392-
firefox-devedition = handleTest ./firefox.nix { firefoxPackage = pkgs.firefox-devedition; };
393-
firefox-esr = handleTest ./firefox.nix { firefoxPackage = pkgs.firefox-esr; }; # used in `tested` job
394-
firefox-esr-128 = handleTest ./firefox.nix { firefoxPackage = pkgs.firefox-esr-128; };
390+
firefox = runTest {
391+
imports = [./firefox.nix ];
392+
_module.args.firefoxPackage = pkgs.firefox;
393+
};
394+
firefox-beta = runTest {
395+
imports = [ ./firefox.nix ] ;
396+
_module.args.firefoxPackage = pkgs.firefox-beta;
397+
};
398+
firefox-devedition = runTest {
399+
imports = [ ./firefox.nix ];
400+
_module.args.firefoxPackage = pkgs.firefox-devedition;
401+
};
402+
firefox-esr = runTest {
403+
# used in `tested` job
404+
imports = [ ./firefox.nix ] ;
405+
_module.args.firefoxPackage = pkgs.firefox-esr;
406+
};
407+
firefox-esr-128 = runTest {
408+
imports = [ ./firefox.nix ] ;
409+
_module.args.firefoxPackage = pkgs.firefox-esr-128;
410+
};
395411
firefoxpwa = handleTest ./firefoxpwa.nix {};
396412
firejail = handleTest ./firejail.nix {};
397413
firewall = handleTest ./firewall.nix { nftables = false; };

nixos/tests/firefox.nix

+14-6
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,9 @@
1-
import ./make-test-python.nix ({ lib, pkgs, firefoxPackage, ... }:
1+
{
2+
lib,
3+
pkgs,
4+
firefoxPackage,
5+
...
6+
}:
27
{
38
name = firefoxPackage.pname;
49

@@ -9,7 +14,8 @@ import ./make-test-python.nix ({ lib, pkgs, firefoxPackage, ... }:
914
nodes.machine =
1015
{ pkgs, ... }:
1116

12-
{ imports = [ ./common/x11.nix ];
17+
{
18+
imports = [ ./common/x11.nix ];
1319
environment.systemPackages = [ pkgs.xdotool ];
1420

1521
programs.firefox = {
@@ -31,9 +37,11 @@ import ./make-test-python.nix ({ lib, pkgs, firefoxPackage, ... }:
3137

3238
};
3339

34-
testScript = let
35-
exe = lib.getExe firefoxPackage;
36-
in ''
40+
testScript =
41+
let
42+
exe = lib.getExe firefoxPackage;
43+
in
44+
''
3745
from contextlib import contextmanager
3846
3947
@@ -97,4 +105,4 @@ import ./make-test-python.nix ({ lib, pkgs, firefoxPackage, ... }:
97105
machine.screenshot("screen")
98106
'';
99107

100-
})
108+
}

0 commit comments

Comments
 (0)