diff options
author | 2023-08-23 11:55:02 -0300 | |
---|---|---|
committer | 2023-08-23 12:02:24 -0300 | |
commit | 72c6df3af5848f5551414f1749bd7fab86b28a60 (patch) | |
tree | 552e65cb15df685f4fd0166a0b04f9b3ff8f44cb /test | |
parent | disable-devel.inc: split packaging-related entries (diff) | |
download | firejail-72c6df3af5848f5551414f1749bd7fab86b28a60.tar.gz firejail-72c6df3af5848f5551414f1749bd7fab86b28a60.tar.zst firejail-72c6df3af5848f5551414f1749bd7fab86b28a60.zip |
tests: properly fix fs/kmsg test
It was broken likely due to `private-dev` being added to default.profile
on commit 307dad542 ("adding private-tmp and private-dev to
default.profile", 2023-08-20).
So ignore `private-dev` in the test and make sure to run the tests when
default.profile changes.
This amends commit 75cefd5b1 ("tests: fix error when /dev/kmsg is
missing", 2023-08-21).
Diffstat (limited to 'test')
-rwxr-xr-x | test/fs/kmsg.exp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/fs/kmsg.exp b/test/fs/kmsg.exp index 1e647ab8d..deab8fcf5 100755 --- a/test/fs/kmsg.exp +++ b/test/fs/kmsg.exp | |||
@@ -7,7 +7,7 @@ set timeout 10 | |||
7 | spawn $env(SHELL) | 7 | spawn $env(SHELL) |
8 | match_max 100000 | 8 | match_max 100000 |
9 | 9 | ||
10 | send -- "firejail\r" | 10 | send -- "firejail --ignore=private-dev\r" |
11 | expect { | 11 | expect { |
12 | timeout {puts "TESTING ERROR 1\n";exit} | 12 | timeout {puts "TESTING ERROR 1\n";exit} |
13 | -re "Child process initialized in \[0-9\]+.\[0-9\]+ ms" | 13 | -re "Child process initialized in \[0-9\]+.\[0-9\]+ ms" |
@@ -17,7 +17,6 @@ sleep 1 | |||
17 | send -- "cat /dev/kmsg\r" | 17 | send -- "cat /dev/kmsg\r" |
18 | expect { | 18 | expect { |
19 | timeout {puts "TESTING ERROR 2\n";exit} | 19 | timeout {puts "TESTING ERROR 2\n";exit} |
20 | "No such file or directory" # FIXME: Needed in CI | ||
21 | "Permission denied" | 20 | "Permission denied" |
22 | } | 21 | } |
23 | after 100 | 22 | after 100 |