Skip to content

Commit

Permalink
tests: disable broken wget tests in utils/sysutils
Browse files Browse the repository at this point in the history
They are taking longer than the 30s timeout[1] [2]:

    runner@fv-az246-621:~/work/firejail/firejail/test/sysutils$
    <ysutils$ firejail --ignore=quiet wget -q debian.org
    Reading profile /etc/firejail/wget.profile
    [...]
    Child process initialized in 115.54 ms
    TESTING ERROR 2

    runner@fv-az1234-541:~/work/firejail/firejail/test/utils$
    <irejail --build wget --output-document=~ debian.org
    [...]
    Resolving www.debian.org (www.debian.org)... 128.31.0.62
    Connecting to www.debian.org (www.debian.org)|128.31.0.62|:443... connected.
    TESTING ERROR 13

[1] https://github.com/kmk3/firejail/actions/runs/6005119423/job/16287436840
[2] https://github.com/kmk3/firejail/actions/runs/6005314148/job/16287794321
  • Loading branch information
kmk3 committed Aug 28, 2023
1 parent d5c90b7 commit 7e91a04
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 12 deletions.
4 changes: 2 additions & 2 deletions test/sysutils/sysutils.sh
Original file line number Diff line number Diff line change
Expand Up @@ -121,8 +121,8 @@ fi

if command -v wget
then
echo "TESTING: wget"
./wget.exp
echo "TESTING: FIXME: wget"
#./wget.exp # FIXME: Broken in CI
else
echo "TESTING SKIP: wget not found"
fi
Expand Down
21 changes: 11 additions & 10 deletions test/utils/build.exp
Original file line number Diff line number Diff line change
Expand Up @@ -97,15 +97,16 @@ after 100
# increase the timeout for remote services
set timeout 30

send -- "firejail --build wget --output-document=~ debian.org\r"
expect {
timeout {puts "TESTING ERROR 13\n";exit}
"protocol"
}
expect {
timeout {puts "TESTING ERROR 13.1\n";exit}
"inet"
}
after 100
# FIXME: Broken in CI
#send -- "firejail --build wget --output-document=~ debian.org\r"
#expect {
# timeout {puts "TESTING ERROR 13\n";exit}
# "protocol"
#}
#expect {
# timeout {puts "TESTING ERROR 13.1\n";exit}
# "inet"
#}
#after 100

puts "all done\n"

0 comments on commit 7e91a04

Please sign in to comment.