Skip to content

Commit

Permalink
complete-run.pl: Un-set I3SOCK (i3#6139)
Browse files Browse the repository at this point in the history
This causes test breakage when the developer/tester is running the tests
in i3, as the actual active instance socket is used instead of the test
instance.

Besides breaking tests, this is quite dangerous as tests like 319-gaps.t
will replace the **actual** config of the user (i.e.
~/.config/i3/config) instead of the current test config.

Broken after i3#5987
  • Loading branch information
orestisfl authored and desmana committed Aug 6, 2024
1 parent 8d2df24 commit 9a4809c
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions testcases/complete-run.pl.in
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,8 @@ $ENV{PATH} = join(':',
'@abs_top_builddir@',
'@abs_top_srcdir@',
$ENV{PATH});
# Make sure we don't re-use the existing (developer's) i3 connection
delete $ENV{I3SOCK};

qx(Xephyr -help 2>&1);
die "Xephyr was not found in your path. Please install Xephyr (xserver-xephyr on Debian)." if $?;
Expand Down

0 comments on commit 9a4809c

Please sign in to comment.