about summary refs log tree commit diff
path: root/tests
diff options
context:
space:
mode:
authoraszlig <aszlig@nix.build>2021-11-07 17:31:44 +0100
committeraszlig <aszlig@nix.build>2021-11-07 17:31:44 +0100
commitbba268d374dbcf1238e350f394bca0b103b16297 (patch)
tree47df53a7761a32edb6de34d0f948ea8644124be0 /tests
parente06654b3d2e6b56b1916369b56e312647e7da93d (diff)
tests: Fix running background command for PSI test
Recently, the implementation behind Machine.execute() and thus also
Machine.succeed() has been changed[1] to pipe all the command's output
into base64 on the guest machine.

Unfortunately this means that base64 is blocking until stdout is closed,
which in turn means that we now need to make sure that whenever we run a
program in background via "&" we also need to make sure to close stdout.

In the PSI test, we're doing this by simply redirecting the output to
stderr.

[1]: https://github.com/NixOS/nixpkgs/pull/142747

Signed-off-by: aszlig <aszlig@nix.build>
Diffstat (limited to 'tests')
-rw-r--r--tests/aszlig/programs/psi.nix2
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/aszlig/programs/psi.nix b/tests/aszlig/programs/psi.nix
index d42c4c21..b926f0c0 100644
--- a/tests/aszlig/programs/psi.nix
+++ b/tests/aszlig/programs/psi.nix
@@ -19,7 +19,7 @@
     machine.wait_for_x()
     machine.wait_for_file("/home/alice/.Xauthority")
     machine.succeed("xauth merge ~alice/.Xauthority")
-    machine.succeed('su -c "DISPLAY=:0.0 psi" - alice &')
+    machine.succeed('su -c "DISPLAY=:0.0 psi" - alice >&2 &')
     machine.wait_for_text('(?i)Reg.ster new account')
     machine.screenshot('psi')
   '';