From c30a7cdfde8b54763878123d24525c604528043c Mon Sep 17 00:00:00 2001 From: Johannes Altmanninger Date: Sun, 18 Apr 2021 11:55:54 +0200 Subject: [PATCH] Enable tmux-complete test in the CI Just add some extra sleep time so it hopefully also works when the CI system is overloaded. This succeeded >60 times in the CI, without a single failure. In case it legitimately fails again, we should provide simple steps to reproduce the failure interactively (using "tmux attach"). The uvar issue only triggered because two fish are started - one is running the tmux-complete script, the other one is running inside tmux. We could reduce the complexity of this test by writing it in a different language, like sh or python. --- tests/checks/tmux-complete.fish | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tests/checks/tmux-complete.fish b/tests/checks/tmux-complete.fish index 28cd8154b..83266351f 100644 --- a/tests/checks/tmux-complete.fish +++ b/tests/checks/tmux-complete.fish @@ -1,13 +1,12 @@ #RUN: %fish -C 'set -g fish %fish' %s #REQUIRES: command -v tmux -# Don't run this on GitHub Actions since it's flaky. -#REQUIRES: test "$CI" != true # Isolated tmux. set -g tmpdir (mktemp -d) set -g tmux tmux -S $tmpdir/.tmux-socket -f /dev/null set -g sleep sleep .1 +set -q CI && set sleep sleep 1 set fish (builtin realpath $fish) cd $tmpdir